Timm =?utf-8?q?Bäder?= <tbae...@redhat.com> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/117...@github.com>
AaronBallman wrote: > but this is not a bug introduced by the changes, rather, the newly supplied > (valid!) source ranges now get taken into account in > `checkRangesForMacroArgExpansion()` in `Frontend/DiagnosticRenderer.cpp`. Yeah, I think we need to solve that before moving forward with the changes, which is why this keeps not being improved. (It always seems to come back to bite us when we decide that invalid source locations are an in-band indicator of anything.) https://github.com/llvm/llvm-project/pull/117671 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
