[clang] [clang] Refactor attr diagnostics to use %select (PR #122473)
Maksim Ivanov via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 10 07:29:06 PST 2025
emaxx-google wrote:
@ilya-biryukov You suggested previously (https://github.com/llvm/llvm-project/pull/118567/files#r1868729303) to introduce a separate enum, but I didn't manage to do it so far. Instead, I added new strings to `AttributeDeclKind`. My thinking has been:
1. If we create a enum, I'm not clear on how it'd be different from `AttributeDeclKind`.
* This `AttributeDeclKind` is already used both by Parser and Sema - same is for our callsites in question.
* Not sure about how to name the items differently - they do seem very close.
2. Another idea was to include the old enum's items into the new one.
* But I don't know how, technically, could we do it without copy-pasting the enum items and `%select` arguments.
But I'm open to any suggestions; thanks!
https://github.com/llvm/llvm-project/pull/122473
More information about the cfe-commits
mailing list