This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Add curly braces around switch case [NFC]
ClosedPublic

Authored by peter.smith on Jun 6 2017, 3:09 AM.

Details

Summary

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.

Diff Detail

Repository
rL LLVM

Event Timeline

peter.smith created this revision.Jun 6 2017, 3:09 AM
avieira accepted this revision.Jun 6 2017, 3:13 AM

LGTM, thanks for the patch!

This revision is now accepted and ready to land.Jun 6 2017, 3:13 AM
This revision was automatically updated to reflect the committed changes.