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

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 10:52:08 PDT 2015


craig.topper added inline comments.

================
Comment at: lib/Target/X86/X86.td:40
@@ -39,1 +39,3 @@
 
+def FeatureXSAVE   : SubtargetFeature<"xsave", "HasXSAVE", "true",
+                                       "Support xsave instructions">;
----------------
These probably need to be added to their respective CPUs as well. But that can be done as a followup.

================
Comment at: test/CodeGen/X86/system-intrinsics-64.ll:1
@@ -1,2 +1,2 @@
 ; RUN: llc < %s -mtriple=x86_64-unknown-unknown | FileCheck %s
 
----------------
The command line for this test doesn't enable xsaves or xsaveopt. Looks like its up to feature detection on the processor it runs on.


Repository:
  rL LLVM

http://reviews.llvm.org/D13012





More information about the llvm-commits mailing list