[PATCH] D70874: [X86] Add initialization of MXCSR in llvm-exegesis

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 1 01:59:47 PST 2019


pengfei added inline comments.


================
Comment at: llvm/tools/llvm-exegesis/lib/X86/Target.cpp:506
+  add(allocateStackSpace(4));
+  add(fillStackSpace(X86::MOV32mi, 0, 0x1f8)); // Mask all FP exceptions
+  add(MCInstBuilder(HasAVX ? X86::VLDMXCSR : X86::LDMXCSR)
----------------
craig.topper wrote:
> Isn't the default value of MXCSR 0x1f80 not 0x1f8?
Yes! I missed an 0. Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70874





More information about the llvm-commits mailing list