r176133 - Don't set the -target-cpu and -target-features attributes just now.

Bill Wendling isanbard at gmail.com
Tue Feb 26 15:01:34 PST 2013


Author: void
Date: Tue Feb 26 17:01:33 2013
New Revision: 176133

URL: http://llvm.org/viewvc/llvm-project?rev=176133&view=rev
Log:
Don't set the -target-cpu and -target-features attributes just now.

This is causing some problems with some of the builders. It's non-trivial to
reset the target's features.

Modified:
    cfe/trunk/lib/CodeGen/CGCall.cpp

Modified: cfe/trunk/lib/CodeGen/CGCall.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGCall.cpp?rev=176133&r1=176132&r2=176133&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/CGCall.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGCall.cpp Tue Feb 26 17:01:33 2013
@@ -1021,19 +1021,6 @@ void CodeGenModule::ConstructAttributeLi
     // Attributes that should go on the call site only.
     if (!CodeGenOpts.SimplifyLibCalls)
       FuncAttrs.addAttribute(llvm::Attribute::NoBuiltin);
-  } else {
-    // Attributes that should go on the function, but not the call site.
-    if (!TargetOpts.CPU.empty())
-      FuncAttrs.addAttribute("target-cpu", TargetOpts.CPU);
-
-    if (TargetOpts.Features.size()) {
-      llvm::SubtargetFeatures Features;
-      for (std::vector<std::string>::const_iterator
-             it = TargetOpts.Features.begin(),
-             ie = TargetOpts.Features.end(); it != ie; ++it)
-        Features.AddFeature(*it);
-      FuncAttrs.addAttribute("target-features", Features.getString());
-    }
   }
 
   QualType RetTy = FI.getReturnType();





More information about the cfe-commits mailing list