[PATCH] D117551: [VPlan] Add VPWidenIntOrFpInductionRecipe::isCanonical, use it.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 21 02:42:02 PST 2022
fhahn added a comment.
In D117551#3260739 <https://reviews.llvm.org/D117551#3260739>, @michaelplatings wrote:
> If LLVM_ENABLE_ASSERTIONS=OFF then isCanonical is not defined.
>
> See line 1077 of llvm/lib/Transforms/Vectorize/VPlan.cpp:
>
> #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
>
> Edit: (Thanks Florian for fixing already!)
Yep thanks for the heads up, should already be fixed by 55689904d2e5 <https://reviews.llvm.org/rG55689904d2e5afcc5309f7234d6369307ee305d0>
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117551/new/
https://reviews.llvm.org/D117551
More information about the llvm-commits
mailing list