avogelsgesang added a comment. Thank for your comments, @adamcz and @njames93!
I agree that a "Implement interface" tweak is probably more useful, and I can imagine to evolve this tweak in that direction. The main reason, I first went for the "create a new class from scratch" approach, was because I was not sure how to merge the newly inserted methods with the potentially existing methods to get an intuitive order. How do you think we should further proceed here? - Do we see value in both an "Implement methods" and an "Add subclass" tweak? - Are you, @njames93, planning to follow-up on your existing review? - Would you like me to evolve this patch into an "Implement methods" patch, drawing inspiration from your existing review & the comments on it? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122102/new/ https://reviews.llvm.org/D122102 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits