[PATCH] D146197: [RISCV] Remove ExtName from RISCVExtensionInfo.

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 16 02:00:56 PDT 2023


asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.

LGTM. I think I can see the potential reasoning for the extension name being in `RISCVExtensionInfo` in the first place - it makes it more plausible to use it separated from the OrderedExtensionMap. But as you note, in reality we don't have any such uses and I can't really foresee any either.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146197



More information about the llvm-commits mailing list