[PATCH] D136817: [RISCV] Add H extension

Alex Bradbury via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 8 02:38:06 PST 2022


asb added a comment.

Thanks Kito, I think my only remaining request would be to add at least some test coverage for using a H extensions when h isn't included in the ISA string. I don't think such tests are handled very cleanly or consistently right now, but adding something to rv32i-invalid.s alongside similar checks would be better than nothing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136817



More information about the cfe-commits mailing list