[llvm] [MachineBasicBlock] Fix use after free in SplitCriticalEdge (PR #66188)

via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 13 03:00:52 PDT 2023


================
@@ -682,7 +683,8 @@ class TargetInstrInfo : public MCInstrInfo {
   /// If \p BytesRemoved is non-null, report the change in code size from the
   /// removed instructions.
   virtual unsigned removeBranch(MachineBasicBlock &MBB,
-                                int *BytesRemoved = nullptr) const {
+                                int *BytesRemoved = nullptr,
----------------
qcolombet wrote:

+1 on that if we decide to go with the additional parameter. Otherwise you'll have to add the related `/*BytesRemoved=*/`, `/*BytesAdded=*/` everywhere.

https://github.com/llvm/llvm-project/pull/66188


More information about the llvm-commits mailing list