h was the prefix of multi-letter extension name, but it become a
extension name in later RISC-V isa spec.
Fortunately we don't have any extension really defined is prefixed
with h, so we can just change that.
Differential D136817
[RISCV] Add H extension kito-cheng on Oct 26 2022, 11:27 PM. Authored by
Details
h was the prefix of multi-letter extension name, but it become a Fortunately we don't have any extension really defined is prefixed
Diff Detail
Event TimelineComment Actions Thanks for this patch Kito - I'm thinking that perhaps we erred in not treating the hypervisor instructions (e.g. HLV* and HSV*) as being gated on the H extension, so a sensible follow-on to this patch would be to mark those instructions as requiring FeatureStdExtH - what do you think?
Comment Actions 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. |