[PATCH] D144215: [WIP][RISCV] Accept zicntr and zihpm command line options

[Alex Bradbury via Phabricator via llvm-commits]

asb added a comment.

One additional point to make is that we don't currently try to gate access to CSR names on extensions. Partly as when we started the vocabulary to do so wasn't always there. Partly as nobody was motivated to implement it and add all the necessary testing. I'm personally not sure it has a huge amount of value.

So I'm incredibly relaxed about accepting zicntr/zihpm in the kind of way this patch suggests once their status is clarified. Looks like the versioning questions was raised on the riscv-isa-manual repo too https://github.com/riscv/riscv-isa-manual/issues/976. The merged PR https://github.com/riscv/riscv-isa-manual/pull/859 includes clarifications to zicntr/zihpm "for ratification" which suggests the process at least started.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144215/new/

https://reviews.llvm.org/D144215