[PATCH] D153865: [LoongArch] Add back SDNPSideEffect properties to CSR and IOCSR read ops

WÁNG Xuěruì via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 27 06:32:01 PDT 2023


xen0n added a comment.

In D153865#4452321 <https://reviews.llvm.org/D153865#4452321>, @SixWeining wrote:

> Interesting. Does cpucfg have the same issue?

In theory CPUCFG data could be altered by writing to certain model-specific CSRs, but I don't think it's worthwhile to make it volatile too, because no kernel should be doing that under the hood (at most do this once before starting even PID1; in fact no known LoongArch OSes do this).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153865



More information about the llvm-commits mailing list