[PATCH] D13012: [X86] Add XSAVE intrinsics (LLVM part)

Amjad Aboud via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 11:49:57 PDT 2015


aaboud added inline comments.

================
Comment at: lib/Target/X86/X86Subtarget.h:97
@@ -89,1 +96,3 @@
+  bool HasXSAVES;
+
   /// Target has carry-less multiplication
----------------
craig.topper wrote:
> I really wanted to approve this, but then I realized that these aren't set to false in the X86SubTarget constructor. Which I think is bad.
No Problem :)
Will set them to false.
Is there anything else?


Repository:
  rL LLVM

http://reviews.llvm.org/D13012





More information about the llvm-commits mailing list