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

[Xi Ruoyao via Phabricator via llvm-commits]

xry111 added a comment.

Ah, in the C standard "reading a volatile variable" is indeed a side effect.  I must have been thinking about another definition of "side effect".

But should we do it for movfcsr2gr as well?  It cannot be reordered with any floating-point arithmetic instructions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153865