[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