[PATCH] D80106: Fix several places that were calling verifyFunction or verifyModule without checking the return value.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 18 07:29:34 PDT 2020


jdoerfert added a comment.

Are those all cases? Wasn't there on in the SLP vectorizer?



================
Comment at: llvm/lib/CodeGen/WinEHPrepare.cpp:1077
 
-    LLVM_DEBUG(verifyFunction(F));
+    assert(!verifyFunction(F, &dbgs()));
     cleanupPreparedFunclets(F);
----------------
Nit: I don't know what the convention is, `&errs()` might be better, but no strong feelings.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80106





More information about the llvm-commits mailing list