[PATCH] D116645: [RISCV][llvm] Update CSRs
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 6 05:32:51 PST 2022
asb added a comment.
In D116645#3224967 <https://reviews.llvm.org/D116645#3224967>, @achieveartificialintelligence wrote:
> In D116645#3224932 <https://reviews.llvm.org/D116645#3224932>, @asb wrote:
>
>> I haven't gone through everything here, but I just spotted this is missing scountovf from the recently ratified Count Overflow and Mode-Based Filtering Extension <https://github.com/riscv/riscv-count-overflow/releases/download/v0.5.2/Sscofpmf.pdf>.
>
> I couldn't find `scountovf` in https://github.com/riscv/riscv-isa-manual/releases/tag/draft-20211229-fb7e236 . Should I add this CSR?
Oh I see, some of the CSRs from recently ratified extensions have been merged into the privileged spec but not all. Probably easiest to review if we keep this patch to just those CSRs added in the privilege spec update, and have separate patches for CSRs in other ratified extensions that aren't yet merged in.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116645/new/
https://reviews.llvm.org/D116645
More information about the llvm-commits
mailing list