Skip to content

Commit d16c55d

Browse files
committedJun 6, 2017
[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 llvm-svn: 304785
1 parent 61eaa63 commit d16c55d

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed
 

‎llvm/lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp

+2-1
Original file line numberDiff line numberDiff line change
@@ -695,7 +695,7 @@ unsigned ARMAsmBackend::adjustFixupValue(const MCFixup &Fixup, uint64_t Value,
695695
return 0;
696696
}
697697
return Value;
698-
case ARM::fixup_t2_so_imm:
698+
case ARM::fixup_t2_so_imm: {
699699
Value = ARM_AM::getT2SOImmVal(Value);
700700
if ((int64_t)Value < 0) {
701701
Ctx.reportError(Fixup.getLoc(), "out of range immediate fixup value");
@@ -712,6 +712,7 @@ unsigned ARMAsmBackend::adjustFixupValue(const MCFixup &Fixup, uint64_t Value,
712712
EncValue |= (Value & 0xff);
713713
return swapHalfWords(EncValue, IsLittleEndian);
714714
}
715+
}
715716
}
716717

717718
void ARMAsmBackend::processFixupValue(const MCAssembler &Asm,

0 commit comments

Comments
 (0)
Please sign in to comment.