Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

[LV] Adjust cost model to use uniform store lowering for unpredicated uniform stores

Authored by reames on Sep 22 2022, 10:50 AM.



Follow up to D133580; adjust the cost model to prefer uniform store lowering for scalable stores which are unpredicated.

The impact here isn't in the uniform store lowering quality itself. InstCombine happily converts the scatter form into the single store form. The main impact is in letting the rest of the cost model make choices based on the knowledge that the vector will be scalarized on use.

Diff Detail

Event Timeline

reames created this revision.Sep 22 2022, 10:50 AM
reames requested review of this revision.Sep 22 2022, 10:50 AM
fhahn accepted this revision.Sep 26 2022, 8:48 AM

LGTM, thanks!

This revision is now accepted and ready to land.Sep 26 2022, 8:48 AM