Skip to content

Commit 7d78875

Browse files
committed
[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.
1 parent 2f499b9 commit 7d78875

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

llvm/include/llvm/IR/IntrinsicsX86.td

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -286,7 +286,7 @@ let TargetPrefix = "x86" in { // All intrinsics start with "llvm.x86.".
286286
// FIXME: LDMXCSR does not actually write to memory,
287287
// but intrinsic properties are generated incorrectly
288288
// for IntrReadMem+IntrHasSideEffects.
289-
[/*IntrReadMem,*/ IntrArgMemOnly, IntrHasSideEffects]>;
289+
[/*IntrReadMem, IntrArgMemOnly,*/ IntrHasSideEffects]>;
290290
}
291291

292292
// Misc.

0 commit comments

Comments
 (0)