[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