[X86] Remove IntrArgMemOnly from ldmxcsr intrinsic.
authorCraig Topper <craig.topper@sifive.com>
Sat, 9 Jan 2021 01:47:12 +0000 (17:47 -0800)
committerCraig Topper <craig.topper@sifive.com>
Sat, 9 Jan 2021 01:50:28 +0000 (17:50 -0800)
Since we're leaving this as ReadWrite with the "write" reflecting
the update to MXCSR, we shouldn't say it only writes arg memory.

Hopefully this fixes the issue reported in post-commit in D93571.

llvm/include/llvm/IR/IntrinsicsX86.td

index 68b076c..bba1213 100644 (file)
@@ -286,7 +286,7 @@ let TargetPrefix = "x86" in {  // All intrinsics start with "llvm.x86.".
                          // FIXME: LDMXCSR does not actually write to memory,
                          // but intrinsic properties are generated incorrectly
                          // for IntrReadMem+IntrHasSideEffects.
-                        [/*IntrReadMem,*/ IntrArgMemOnly, IntrHasSideEffects]>;
+                        [/*IntrReadMem, IntrArgMemOnly,*/ IntrHasSideEffects]>;
 }
 
 // Misc.