[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