[llvm] 933a566 - [PAuthLR] Add Missing Break Statement for MachineOperand Switch Statement (#113883)

via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 02:08:52 PDT 2024


Author: Jack Styles
Date: 2024-10-28T09:08:48Z
New Revision: 933a56674e75ed372e000758378b7981e5b4d387

URL: https://github.com/llvm/llvm-project/commit/933a56674e75ed372e000758378b7981e5b4d387
DIFF: https://github.com/llvm/llvm-project/commit/933a56674e75ed372e000758378b7981e5b4d387.diff

LOG: [PAuthLR] Add Missing Break Statement for MachineOperand Switch Statement (#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.

Added: 
    

Modified: 
    llvm/lib/CodeGen/MachineOperand.cpp

Removed: 
    


################################################################################
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