[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