[llvm] r314143 - [Hexagon] Avoid unused variable warnings in Release builds.

Benjamin Kramer via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 25 12:42:21 PDT 2017


Author: d0k
Date: Mon Sep 25 12:42:20 2017
New Revision: 314143

URL: http://llvm.org/viewvc/llvm-project?rev=314143&view=rev
Log:
[Hexagon] Avoid unused variable warnings in Release builds.

No functionality change intended.

Modified:
    llvm/trunk/lib/Target/Hexagon/HexagonBitTracker.cpp

Modified: llvm/trunk/lib/Target/Hexagon/HexagonBitTracker.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/HexagonBitTracker.cpp?rev=314143&r1=314142&r2=314143&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Hexagon/HexagonBitTracker.cpp (original)
+++ llvm/trunk/lib/Target/Hexagon/HexagonBitTracker.cpp Mon Sep 25 12:42:20 2017
@@ -125,8 +125,8 @@ uint16_t HexagonEvaluator::getPhysRegBit
   if (const TargetRegisterClass *RC = TRI.getMinimalPhysRegClass(Reg))
     return TRI.getRegSizeInBits(*RC);
 
-  StringRef E = "Unhandled physical register";
-  llvm_unreachable((Twine(E) + TRI.getName(Reg)).str().c_str());
+  llvm_unreachable(
+      (Twine("Unhandled physical register") + TRI.getName(Reg)).str().c_str());
 }
 
 const TargetRegisterClass &HexagonEvaluator::composeWithSubRegIndex(
@@ -134,10 +134,12 @@ const TargetRegisterClass &HexagonEvalua
   if (Idx == 0)
     return RC;
 
+#ifndef NDEBUG
   const auto &HRI = static_cast<const HexagonRegisterInfo&>(TRI);
   bool IsSubLo = (Idx == HRI.getHexagonSubRegIndex(RC, Hexagon::ps_sub_lo));
   bool IsSubHi = (Idx == HRI.getHexagonSubRegIndex(RC, Hexagon::ps_sub_hi));
   assert(IsSubLo != IsSubHi && "Must refer to either low or high subreg");
+#endif
 
   switch (RC.getID()) {
     case Hexagon::DoubleRegsRegClassID:




More information about the llvm-commits mailing list