[PATCH] D67095: [SVE][Inline-Asm] Fix -Wimplicit-fallthrough in AArch64ISelLowering.cpp

Kerry McLaughlin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 3 06:32:17 PDT 2019


kmclaughlin created this revision.
kmclaughlin added reviewers: sdesmalen, rovka, cameron.mcinally, greened, gribozavr, ruiu.
Herald added subscribers: psnobl, rkruppe, kristof.beyls, tschuett, javed.absar.
Herald added a project: LLVM.

Adds break to 'x' case in getRegForInlineAsmConstraint added by D66302 <https://reviews.llvm.org/D66302>, fixing the unintentional fallthrough.


Repository:
  rL LLVM

https://reviews.llvm.org/D67095

Files:
  lib/Target/AArch64/AArch64ISelLowering.cpp


Index: lib/Target/AArch64/AArch64ISelLowering.cpp
===================================================================
--- lib/Target/AArch64/AArch64ISelLowering.cpp
+++ lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -5834,6 +5834,7 @@
         return std::make_pair(0U, &AArch64::ZPR_4bRegClass);
       if (VT.getSizeInBits() == 128)
         return std::make_pair(0U, &AArch64::FPR128_loRegClass);
+      break;
     case 'y':
       if (!Subtarget->hasFPARMv8())
         break;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67095.218443.patch
Type: text/x-patch
Size: 481 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190903/59e59c25/attachment.bin>


More information about the cfe-commits mailing list