Page MenuHomePhabricator

[LV][VPlan] Fix printing TripCount liveins. NFC
ClosedPublic

Authored by dmgreen on Tue, Mar 7, 7:32 AM.

Details

Summary

The TripCount liveins would currently be printed as badref in the vplan as they are not allocated slots in the VPSlotTracker. This patch allocates them a slot and adds them to the printed Live-Ins when printing the vplan. It also makes a minor adjustment to printing of Live-ins to reduce the empty lines when multiple Live-ins are present.

Diff Detail

Event Timeline

dmgreen created this revision.Tue, Mar 7, 7:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Mar 7, 7:32 AM
dmgreen requested review of this revision.Tue, Mar 7, 7:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Mar 7, 7:32 AM
sdesmalen accepted this revision.Fri, Mar 10, 8:23 AM

Nice improvement.

This revision is now accepted and ready to land.Fri, Mar 10, 8:23 AM
fhahn accepted this revision.Fri, Mar 10, 12:25 PM

LGTM, thanks!

llvm/test/Transforms/LoopVectorize/AArch64/sve-tail-folding-forced.ll
13

It would probably be good to use a variable here, to avoid the test needing updating if the numbering changes.

This revision was landed with ongoing or failed builds.Mon, Mar 13, 12:44 PM
This revision was automatically updated to reflect the committed changes.