This is an archive of the discontinued LLVM Phabricator instance.

[VPlan] Replace invariance fields from VPWidenGEPRecipe.
ClosedPublic

Authored by fhahn on Feb 21 2023, 7:11 AM.

Details

Summary

There is no need to store information about invariance in the recipe.
Replace the fields with checks of the operands using
isDefinedOutsideVectorRegions.

Diff Detail

Event Timeline

fhahn created this revision.Feb 21 2023, 7:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 21 2023, 7:11 AM
fhahn requested review of this revision.Feb 21 2023, 7:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 21 2023, 7:11 AM
Ayal accepted this revision.Mar 8 2023, 2:07 AM

Nice cleanup!

llvm/lib/Transforms/Vectorize/VPlan.h
992

nit: worth adding getPointer() to VPWidenGEPRecipe?

This revision is now accepted and ready to land.Mar 8 2023, 2:07 AM
This revision was landed with ongoing or failed builds.Mar 9 2023, 8:52 AM
This revision was automatically updated to reflect the committed changes.