Page MenuHomePhabricator

[DAGCombiner] Add node to the worklist in topological order in scalarizeExtractedVectorLoad

Authored by deadalnix on Aug 23 2019, 8:49 AM.

Diff Detail


Event Timeline

deadalnix created this revision.Aug 23 2019, 8:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 23 2019, 8:49 AM
lebedev.ri added inline comments.Aug 23 2019, 11:10 AM
16639 ↗(On Diff #216871)

Won't this cause the node to be revisited, before AddToWorklist(EVE); ?

deadalnix marked 2 inline comments as done.Aug 23 2019, 12:06 PM
deadalnix added inline comments.
1696 ↗(On Diff #216871)

When the node itself is returned, it is not added back to the worklist.

16639 ↗(On Diff #216871)

No. The only difference between this and just returning an empty value is that it increment the counter of transform that has been done. The main loop assumes that whatever transform does this has handled the worklist properly.

This revision is now accepted and ready to land.Sep 15 2019, 12:16 PM
RKSimon accepted this revision.Sep 15 2019, 1:44 PM


This revision was automatically updated to reflect the committed changes.