[PATCH] D130290: [MachineVerifier] add checks for INLINEASM_BR

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 17 15:15:24 PDT 2022


efriedma added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineVerifier.cpp:904
+        report("INLINEASM_BR indirect target predecessor list missing parent",
+               &MO, i);
+    }
----------------
I don't think you need to check both isSuccessor and isPredecessor; we should have other checks that verify the predecessor lists are consistent with the successor lists.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130290/new/

https://reviews.llvm.org/D130290



More information about the llvm-commits mailing list