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

[Kerry McLaughlin via Phabricator via cfe-commits]

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;



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;

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits