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

[Adrian Kuegel via Phabricator via llvm-commits]

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";
----------------
fhahn wrote:
> akuegel wrote:
> > 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.
> Thanks, this should already be fixed by acbefbf19f6c
Ok, I see your commit now :)
Thanks for the fix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111302