[PATCH] D100751: [VPlan] Properly handle sinking of replicate regions.

[Florian Hahn via Phabricator via llvm-commits]

fhahn marked 3 inline comments as done.
fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:8966
     VPRecipeBase *Sink = RecipeBuilder.getRecipe(Entry.first);
     VPRecipeBase *Target = RecipeBuilder.getRecipe(Entry.second);
     // If the target is in a replication region, make sure to move Sink to the
----------------
Ayal wrote:
> > I think we also need to deal with the case when the target is a
> > replicate region also. I'll put up a patch once I constructed a test
> > for that scenario.
> 
> Agreed. Worth adding an assert in the meanwhile?
That's a great idea, I added an assert.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100751