[PATCH] D41833: [lli] Make lli support -mcpu=native for CPU autodetection

[Phabricator via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL322100: [lli] Make lli support -mcpu=native for CPU autodetection (authored by ctopper, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D41833?vs=128969&id=129117#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D41833

Files:
  llvm/trunk/include/llvm/CodeGen/CommandFlags.def
  llvm/trunk/tools/lli/lli.cpp


Index: llvm/trunk/tools/lli/lli.cpp
===================================================================
--- llvm/trunk/tools/lli/lli.cpp
+++ llvm/trunk/tools/lli/lli.cpp
@@ -378,8 +378,8 @@
   std::string ErrorMsg;
   EngineBuilder builder(std::move(Owner));
   builder.setMArch(MArch);
-  builder.setMCPU(MCPU);
-  builder.setMAttrs(MAttrs);
+  builder.setMCPU(getCPUStr());
+  builder.setMAttrs(getFeatureList());
   if (RelocModel.getNumOccurrences())
     builder.setRelocationModel(RelocModel);
   if (CMModel.getNumOccurrences())
Index: llvm/trunk/include/llvm/CodeGen/CommandFlags.def
===================================================================
--- llvm/trunk/include/llvm/CodeGen/CommandFlags.def
+++ llvm/trunk/include/llvm/CodeGen/CommandFlags.def
@@ -326,6 +326,26 @@
   return Features.getString();
 }
 
+LLVM_ATTRIBUTE_UNUSED static std::vector<std::string> getFeatureList() {
+  SubtargetFeatures Features;
+
+  // If user asked for the 'native' CPU, we need to autodetect features.
+  // This is necessary for x86 where the CPU might not support all the
+  // features the autodetected CPU name lists in the target. For example,
+  // not all Sandybridge processors support AVX.
+  if (MCPU == "native") {
+    StringMap<bool> HostFeatures;
+    if (sys::getHostCPUFeatures(HostFeatures))
+      for (auto &F : HostFeatures)
+        Features.AddFeature(F.first(), F.second);
+  }
+
+  for (unsigned i = 0; i != MAttrs.size(); ++i)
+    Features.AddFeature(MAttrs[i]);
+
+  return Features.getFeatures();
+}
+
 /// \brief Set function attributes of functions in Module M based on CPU,
 /// Features, and command line flags.
 LLVM_ATTRIBUTE_UNUSED static void


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41833.129117.patch
Type: text/x-patch
Size: 1682 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180109/655f90ea/attachment.bin>