[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:08:10 PDT 2023


xen0n added a comment.

In D153865#4452222 <https://reviews.llvm.org/D153865#4452222>, @xry111 wrote:

> But it's puzzling.  Is reading a CSR really a side effect?

I didn't look deeper regarding how to best model this (effectively I'm just reverting some changes), but the crux is that (IO)CSR reads should in general be treated as volatile.


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