[PATCH] D111302: [VPlan] Add initial VPlan verification.

Adrian Kuegel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 9 02:34:55 PST 2021


akuegel added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlanVerifier.cpp:147
+        errs() << "Found phi-like recipe after non-phi recipe:";
+        RecipeI->dump();
+        errs() << "after\n";
----------------
The dump() method is not necessarily available, it is guarded by:
#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)

You should guard the calls to it in the same way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111302



More information about the llvm-commits mailing list