[PATCH] D62896: [X86] Add missing properties on llvm.x86.sse.{st,ld}mxcsr

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

I don't think that's sufficient. A side effecting instruction doesn't prevent other non-side effecting instructions from moving across it. It only prevents other side effecting instructions from moving relative to it. If you load mxcsr from some address A, then store mxcsr to some address B. Without having side effects on the stmxcsr, nothing would prevent it from being able to move above the ldmxcsr.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62896