[PATCH] D89301: [X86] Add user-level interrupt instructions

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 18 19:18:35 PDT 2020


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86InstrInfo.td:320
+def X86testui : SDNode<"X86ISD::TESTUI",
+                       SDTypeProfile<1, 0, [SDTCisVT<0, i32>]>, [SDNPHasChain]>;
 
----------------
I think this should have SDNPSideEffect to match the intrinsic and to prevent reordering with CLUI/STUI in MachineIR since the TESTUI instruction will inherit properties from X86testui being used in its pattern.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89301



More information about the llvm-commits mailing list