revane wrote: Indeed. I can add some tests to make sure nested macros aren't affected but my thought was since this change allows renaming _arguments_ (i.e. not the macro parameters) then it was safe since the code is not really "inside a macro". There are already other means to filter out system headers and other user-defined headers. I couldn't come up with other classes of errors where this replacement would be invalid.
https://github.com/llvm/llvm-project/pull/87792 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits