Tidy checking for the soft float attribute.
authorEric Christopher <echristo@gmail.com>
Sat, 15 Apr 2017 06:14:52 +0000 (06:14 +0000)
committerEric Christopher <echristo@gmail.com>
Sat, 15 Apr 2017 06:14:52 +0000 (06:14 +0000)
llvm-svn: 300394

llvm/lib/Analysis/InlineCost.cpp

index 813f5d72c8a4b28280e92e74b2722cf0a5409c60..1f8dec2aed80639464d1edd58fa98666aa411a6a 100644 (file)
@@ -1101,19 +1101,10 @@ bool CallAnalyzer::analyzeBlock(BasicBlock *BB,
     // is expensive or the function has the "use-soft-float" attribute, this may
     // eventually become a library call. Treat the cost as such.
     if (I->getType()->isFloatingPointTy()) {
-      bool hasSoftFloatAttr = false;
-
       // If the function has the "use-soft-float" attribute, mark it as
       // expensive.
-      if (F.hasFnAttribute("use-soft-float")) {
-        Attribute Attr = F.getFnAttribute("use-soft-float");
-        StringRef Val = Attr.getValueAsString();
-        if (Val == "true")
-          hasSoftFloatAttr = true;
-      }
-
       if (TTI.getFPOpCost(I->getType()) == TargetTransformInfo::TCC_Expensive ||
-          hasSoftFloatAttr)
+          (F.getFnAttribute("use-soft-float").getValueAsString() == "true"))
         Cost += InlineConstants::CallPenalty;
     }