This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Update B extension version to 0.93.
ClosedPublic

Authored by craig.topper on Jan 19 2021, 2:16 PM.

Diff Detail

Event Timeline

craig.topper created this revision.Jan 19 2021, 2:16 PM
craig.topper requested review of this revision.Jan 19 2021, 2:16 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 19 2021, 2:16 PM
asb added a comment.Jan 21 2021, 5:51 AM

I don't think any of the other patches in the stack update the comment at the top of RISCVInstrInfoB.td to say "version 0.92" rather than "version 0.93", and this is probably a reasonable patch to do it in.

Update header on RISCVInstrInfoB.td

Herald added a project: Restricted Project. · View Herald TranscriptJan 21 2021, 8:04 AM
Herald added a subscriber: hiraditya. · View Herald Transcript

Rebase after clang-format in Zba patch

Remove conflict marker that got left behind

frasercrmck added inline comments.Jan 22 2021, 6:56 AM
llvm/lib/Target/RISCV/RISCVInstrInfoB.td
732 ↗(On Diff #318219)

This might need updated. The statement is still true in 0.92, but it's not the "current version"

Remove reference to version from the rori comment. The version doesn't really matter, I doubt the encoding will ever be added.

frasercrmck accepted this revision.Jan 22 2021, 10:15 AM
This revision is now accepted and ready to land.Jan 22 2021, 10:15 AM
asb accepted this revision.Jan 22 2021, 10:37 AM
This revision was landed with ongoing or failed builds.Jan 22 2021, 12:51 PM
This revision was automatically updated to reflect the committed changes.