[clang] [Clang] Allow parsing arbitrary order of attributes for declarations (PR #133107)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Thu May 15 04:46:51 PDT 2025
AaronBallman wrote:
> Yeah, I think I can live with this. I think not supporting the arbitrary order is more annoying to users than a slight degradation in source location reporting.
The point @kadircet brings up about tooling is a good one, though. I'm not certain we need to revert the patch, but breaking a bunch of tools and making them cope with the source location change is pretty disruptive. I think we may need to consider refactoring source location handling for attributes more broadly to solve the underlying concerns. Thoughts @erichkeane?
https://github.com/llvm/llvm-project/pull/133107
More information about the cfe-commits
mailing list