[PATCH] D122832: [TableGen] Add a backend to help with target enums

Paul C. Anagnostopoulos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 8 15:35:34 PDT 2022


Paul-C-Anagnostopoulos added a comment.

Why don't you take an inventory of the number of uses of GenericEnum? Then we can see if it's practical to replace those uses.

Also, please review your Enum against GenericEnum to be sure you've covered all the features.

I don't think we can eliminate GenericEnum, because there may be users outside of the LLVM project. But we could deprecate it in favor of Enum.

And, of course, you will have to write documentation similar to that for GenericEnum.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122832/new/

https://reviews.llvm.org/D122832



More information about the llvm-commits mailing list