[PATCH] D68121: [X86] Model MXCSR for all SSE instructions

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 26 23:50:08 PDT 2019


craig.topper added a comment.

Does this really not affect any tests? I would have thought some test would print MIR output that would show the implicit use that wasn't there before.



================
Comment at: llvm/lib/Target/X86/X86InstrFormats.td:230
 class NOTRACK { bit hasNoTrackPrefix = 1; }
+class SIMD_EXP { list<Register> Uses = [MXCSR]; bit mayRaiseFPException = 1; }
 
----------------
Is EXP an abbreviation for Exception? If so, I think EXC is probably a better abbreviation


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68121





More information about the llvm-commits mailing list