sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land.
The modifier seems reasonable to me. The most important cases are callsites (where `virtual`) isn't written. Can you extend a test to cover some such case? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107145/new/ https://reviews.llvm.org/D107145 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
