This is an archive of the discontinued LLVM Phabricator instance.

[RISCV][docs] Describe status of zicsr and zifencei
ClosedPublic

Authored by reames on Feb 13 2023, 8:48 AM.

Details

Summary

This has been a repeated point of recent confusion. There are a couple attempts in flight to change the way we handle the extensions, but for the moment, let's just document what's going on.

Diff Detail

Event Timeline

reames created this revision.Feb 13 2023, 8:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 13 2023, 8:48 AM
reames requested review of this revision.Feb 13 2023, 8:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 13 2023, 8:48 AM
asb accepted this revision.Feb 13 2023, 9:01 AM

LGTM, thanks!

This revision is now accepted and ready to land.Feb 13 2023, 9:01 AM
jrtc27 accepted this revision.Feb 13 2023, 9:04 AM
jrtc27 added inline comments.
llvm/docs/RISCVUsage.rst
133

Minor tweak to make it more obvious

reames closed this revision.Feb 13 2023, 1:26 PM

commit 0c412fed360f491e2c33373ffae12e10574a2f0e
Author: Philip Reames <preames@rivosinc.com>
Date: Mon Feb 13 09:07:28 2023 -0800

[RISCV][docs] Describe status of zicsr and zifencei

This has been a repeated point of recent confusion. There are a couple attempts in flight to change the way we handle the extensions, but for the moment, let's just document what's going on.

Differential Revision: https://reviews.llvm.org/D14392

(Typo in the differential revision line prevented the auto-close)