[PATCH] D85067: [RISCV] Add mucounteren CSR
Jessica Clarke via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 5 06:06:18 PDT 2020
jrtc27 accepted this revision.
jrtc27 added a comment.
This revision is now accepted and ready to land.
Looks fine, other than please mention something about it being an alias in the commit subject when landing as currently it reads as if it's a new CSR (and I'd also personally change "This patch enables the use of mucounteren." to "This patch enables the use of the old mucounteren name." to make it completely clear, but if the subject is fixed then I don't think it's _necessary_).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85067/new/
https://reviews.llvm.org/D85067
More information about the llvm-commits
mailing list