[PATCH] D90562: [VPlan] Use VPDef for VPInterleaveRecipe.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.



In D90562#2477665 <https://reviews.llvm.org/D90562#2477665>, @john.brawn wrote:

> This commit is causing the following
>
>   int example(int n, int *arr)
>   {
>    int ret = 0;
>   
>    for( int i = 0; i < n; i += 2 ) {
>     ret = arr[i+1] + arr[i] + ret;
>    }
>   
>    return ret;
>   }
>
> to hit an assertion failure when compiled with --target=arm-none-eabi  -march=armv8.1-m.main+mve -O2:
>
>   llvm-project/llvm/lib/Transforms/Vectorize/VPlan.h:1770: llvm::VPValue* llvm::VPlan::getVPValue(llvm::Value*): Assertion `Value2VPValue.count(V) && "Value does not exist in VPlan"' failed.
>
> with the backtrace showing that the assert was hit in llvm::LoopVectorizationPlanner::adjustRecipesForInLoopReductions.

Thanks for the report! Should be fixed in 8a47e6252ad43a2eb3238f9b36394571ba13f4a9 <https://reviews.llvm.org/rG8a47e6252ad43a2eb3238f9b36394571ba13f4a9>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90562