[PATCH] D62896: [X86] Add missing properties on llvm.x86.sse.{st,ld}mxcsr
Clement Courbet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 5 08:59:53 PDT 2019
courbet added a comment.
In D62896#1530895 <https://reviews.llvm.org/D62896#1530895>, @craig.topper wrote:
> Don’t we need stmxcsr to have side effects to keep it from reordering relative to ldmxcsr?
Shouldn't it be sufficient that it writes to memory and `ldmxcsr` has side effects ? Essentially `stmxcsr` is nothing more than a fancy `MOV32mr`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62896/new/
https://reviews.llvm.org/D62896
More information about the llvm-commits
mailing list