[PATCH] D144434: [VPlan] Handle invariant GEPs in isUniformAfterVectorization.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 30 08:17:48 PDT 2023
fhahn marked an inline comment as done.
fhahn added inline comments.
================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:2708
+ if (auto *GEP = dyn_cast<VPWidenGEPRecipe>(Def))
+ return all_of(GEP->operands(), isUniformAfterVectorization);
return false;
----------------
Ayal wrote:
> Propagating uniformity forwards on-demand from operands is fine, for GEPs or other non-phi recipes, where not cached as a result of SCEV/Divergence Analysis.
>
> Should this function still be "inline"?
>
> The "AfterVectorization" suffix should be dropped if favor of "onlyFirstLaneUsed()" above (independent of this patch).
Will move into VPlan.cpp now that this has grown and rename accordingly.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144434/new/
https://reviews.llvm.org/D144434
More information about the llvm-commits
mailing list