Page MenuHomePhabricator

[VPlan] Replace invariance fields from VPWidenGEPRecipe.
ClosedPublic

Authored by fhahn on Tue, Feb 21, 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.Tue, Feb 21, 7:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Feb 21, 7:11 AM
fhahn requested review of this revision.Tue, Feb 21, 7:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Feb 21, 7:11 AM
Ayal accepted this revision.Wed, Mar 8, 2:07 AM

Nice cleanup!

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

nit: worth adding getPointer() to VPWidenGEPRecipe?

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