This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Recover the implication between Zve* extensions and the V extension.
ClosedPublic

Authored by jacquesguan on Feb 7 2022, 7:36 PM.

Details

Summary

This revision recover the implication between Zve* extensions and the V extension.

Diff Detail

Event Timeline

jacquesguan created this revision.Feb 7 2022, 7:36 PM
jacquesguan requested review of this revision.Feb 7 2022, 7:36 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 7 2022, 7:36 PM
eopXD accepted this revision.Feb 11 2022, 3:30 AM

LGTM.
For the record the implication in this patch correspond to the note under v-spec:

As is the case with other RISC-V extensions, it is valid to include overlapping extensions in the same ISA string. For example, RV64GCV and RV64GCV_Zve64f are both valid and equivalent ISA strings, as is RV64GCV_Zve64f_Zve32x_Zvl128b.

This revision is now accepted and ready to land.Feb 11 2022, 3:30 AM

Kindly reverse ping.
May you land this change?

Kindly reverse ping.
May you land this change?

OK, right now.