[PATCH] D33931: [ARM] Add curly braces around switch case [NFC]

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 6 03:09:14 PDT 2017


peter.smith created this revision.
Herald added subscribers: kristof.beyls, javed.absar, aemerson.

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.


https://reviews.llvm.org/D33931

Files:
  lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp


Index: lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp
===================================================================
--- lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp
+++ lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp
@@ -695,7 +695,7 @@
       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 @@
     EncValue |= (Value & 0xff);
     return swapHalfWords(EncValue, IsLittleEndian);
   }
+  }
 }
 
 void ARMAsmBackend::processFixupValue(const MCAssembler &Asm,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33931.101528.patch
Type: text/x-patch
Size: 682 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170606/8dab3f97/attachment.bin>


More information about the llvm-commits mailing list