[llvm] r284826 - fix variable names; NFCI

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 07:58:31 PDT 2016


Author: spatel
Date: Fri Oct 21 09:58:30 2016
New Revision: 284826

URL: http://llvm.org/viewvc/llvm-project?rev=284826&view=rev
Log:
fix variable names; NFCI

Because we're just 'or-ing' these 2 variables later in the code, I
don't think there's a logical bug here, but of course the string with
"no size" is the one that should have the size suffix stripped off.

Modified:
    llvm/trunk/lib/CodeGen/TargetLoweringBase.cpp

Modified: llvm/trunk/lib/CodeGen/TargetLoweringBase.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/TargetLoweringBase.cpp?rev=284826&r1=284825&r2=284826&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/TargetLoweringBase.cpp (original)
+++ llvm/trunk/lib/CodeGen/TargetLoweringBase.cpp Fri Oct 21 09:58:30 2016
@@ -1973,7 +1973,7 @@ static int getOpEnabled(bool IsSqrt, EVT
   // The attribute string may omit the size suffix ('f'/'d').
   std::string VTName = getReciprocalOpName(IsSqrt, VT);
   std::string VTNameNoSize = VTName;
-  VTName.pop_back();
+  VTNameNoSize.pop_back();
   static const char DisabledPrefix = '!';
 
   for (StringRef RecipType : OverrideVector) {
@@ -2028,7 +2028,7 @@ static int getOpRefinementSteps(bool IsS
   // The attribute string may omit the size suffix ('f'/'d').
   std::string VTName = getReciprocalOpName(IsSqrt, VT);
   std::string VTNameNoSize = VTName;
-  VTName.pop_back();
+  VTNameNoSize.pop_back();
 
   for (StringRef RecipType : OverrideVector) {
     size_t RefPos;




More information about the llvm-commits mailing list