[PATCH] D19795: Add new flag and intrinsic support for MWAITX and MONITORX instructions.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Mon May 9 05:52:50 PDT 2016


RKSimon added a subscriber: RKSimon.
RKSimon added a reviewer: RKSimon.

================
Comment at: include/llvm/IR/IntrinsicsX86.td:8722
@@ +8721,2 @@
+        Intrinsic<[], [ llvm_i32_ty, llvm_i32_ty, llvm_i32_ty ], []>;
+}
----------------
Why is _sse3_ in the intrinsic name? I realise the original monitor/mwait came out at the same time as SSE3 (and are included in the same header) but these certainly didn't.

================
Comment at: lib/Support/Host.cpp:806
@@ -804,2 +805,3 @@
   Features["tbm"]    = HasExtLeaf1 && ((ECX >> 21) & 1);
+  Features["mwaitx"]    = HasExtLeaf1 && ((ECX >> 29) & 1);
 
----------------
align the '='


Repository:
  rL LLVM

http://reviews.llvm.org/D19795





More information about the llvm-commits mailing list