[clang] [Clang][Diagnostics] Update select uses in DiagnosticXKinds.td to use enum_select (PR #130868)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 19 06:10:34 PDT 2025


erichkeane wrote:

> I was wondering if it was okay for a replacement since the names of the CommandTraits options are the same. Also would a select with only 1 use (with a magic number) be up for replacement?

I'll check out the review.  As far as this CommandTraits one, it probably is not a good candidate.  There is already an enum for it that is stored in the AST.  

As far as 1 use: Yes, if it has 1 use with a magic number, that is a perfectly reasonable thing to replace with an enum_select.

https://github.com/llvm/llvm-project/pull/130868


More information about the cfe-commits mailing list