[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