[PATCH] D57444: [DebugInfo] LLVM ERROR: Broken function found, while removing Debug Intrinsics

David Stenberg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 31 02:55:48 PST 2019


dstenb added inline comments.


================
Comment at: test/CodeGen/X86/bbi-23595.ll:14
+
+; CHECK-NOT: LLVM ERROR: Broken function found, compilation aborted!
+; CHECK-LABEL: _Z7test_itv()
----------------
It seems to me that the negative check is unnecessary now. The positive checks verify that the `llvm.dbg.label` is dropped. Also, if the verifier errors out, opt exits with a non-zero exit code, so the test would fail due to that.


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

https://reviews.llvm.org/D57444





More information about the llvm-commits mailing list