From 7d78875f93a95815640606fa86a9972386cc5d10 Mon Sep 17 00:00:00 2001 From: Craig Topper Date: Fri, 8 Jan 2021 17:47:12 -0800 Subject: [PATCH] [X86] Remove IntrArgMemOnly from ldmxcsr intrinsic. 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 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/include/llvm/IR/IntrinsicsX86.td b/llvm/include/llvm/IR/IntrinsicsX86.td index 68b076c..bba1213 100644 --- a/llvm/include/llvm/IR/IntrinsicsX86.td +++ b/llvm/include/llvm/IR/IntrinsicsX86.td @@ -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. -- 2.7.4