[llvm] r304785 - [ARM] Add curly braces around switch case [NFC]

Peter Smith via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 6 03:22:49 PDT 2017


Author: psmith
Date: Tue Jun  6 05:22:49 2017
New Revision: 304785

URL: http://llvm.org/viewvc/llvm-project?rev=304785&view=rev
Log:
[ARM] Add curly braces around switch case [NFC]    

My previous commit r304702 introduced a new case into a switch statement.
This case defined a variable but I forgot to add the curly brackets around the
case to limit the scope.

This change puts the curly braces back in so that the next person that adds a
case doesn't get a build failure. Thanks to avieira for the spot.

Differential Revision: https://reviews.llvm.org/D33931


Modified:
    llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp

Modified: llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp?rev=304785&r1=304784&r2=304785&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp (original)
+++ llvm/trunk/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp Tue Jun  6 05:22:49 2017
@@ -695,7 +695,7 @@ unsigned ARMAsmBackend::adjustFixupValue
       return 0;
     }
     return Value;
-  case ARM::fixup_t2_so_imm:
+  case ARM::fixup_t2_so_imm: {
     Value = ARM_AM::getT2SOImmVal(Value);
     if ((int64_t)Value < 0) {
       Ctx.reportError(Fixup.getLoc(), "out of range immediate fixup value");
@@ -712,6 +712,7 @@ unsigned ARMAsmBackend::adjustFixupValue
     EncValue |= (Value & 0xff);
     return swapHalfWords(EncValue, IsLittleEndian);
   }
+  }
 }
 
 void ARMAsmBackend::processFixupValue(const MCAssembler &Asm,




More information about the llvm-commits mailing list