[llvm] [PAuthLR] Fix Unannoted Fallthrough (PR #113883)

Jack Styles via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 02:04:57 PDT 2024


https://github.com/Stylie777 created https://github.com/llvm/llvm-project/pull/113883

There was a missing break, which led to an unannotated fallthrough when merging #112171. This has caused sanitizer builds to fail.

This adds the missing break in the switch statement to ensure that the fallthrough does not occur.

>From c2873d728d81575ef0ed4e9de05f84ef1fa83833 Mon Sep 17 00:00:00 2001
From: Jack Styles <jack.styles at arm.com>
Date: Mon, 28 Oct 2024 08:56:00 +0000
Subject: [PATCH] [PAuthLR] Fix unannoted fallthrough

There was a missing break, which led to an unannotated fallthrough
when merging #112171. This has caused sanitizer builds to fail.

This adds the missing break in the switch statement to ensure that
the fallthrough does not occur.
---
 llvm/lib/CodeGen/MachineOperand.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/llvm/lib/CodeGen/MachineOperand.cpp b/llvm/lib/CodeGen/MachineOperand.cpp
index 620779e2dec708..d9e5e9d9d1e41f 100644
--- a/llvm/lib/CodeGen/MachineOperand.cpp
+++ b/llvm/lib/CodeGen/MachineOperand.cpp
@@ -772,6 +772,7 @@ static void printCFI(raw_ostream &OS, const MCCFIInstruction &CFI,
     OS << "negate_ra_sign_state_with_pc ";
     if (MCSymbol *Label = CFI.getLabel())
       MachineOperand::printSymbol(OS, *Label);
+    break;
   default:
     // TODO: Print the other CFI Operations.
     OS << "<unserializable cfi directive>";



More information about the llvm-commits mailing list