[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