This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Increase the Scalarized cost of masked gather/scatter operations
ClosedPublic

Authored by dmgreen on Mar 31 2023, 7:51 AM.

Details

Summary

If a gather/scatter is masked and will need to be scalarized then the cost should be higher than we currently produce. An additional cost for scalarizing the mask, extracting i1s and branching on the result needs to be added, which I gave a cost of 5 in this patch.

Diff Detail

Event Timeline

dmgreen created this revision.Mar 31 2023, 7:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 31 2023, 7:51 AM
dmgreen requested review of this revision.Mar 31 2023, 7:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 31 2023, 7:51 AM
samtebbs accepted this revision.Apr 11 2023, 5:58 AM

Sorry for the delay. This looks good to me.

This revision is now accepted and ready to land.Apr 11 2023, 5:58 AM
This revision was landed with ongoing or failed builds.Apr 11 2023, 6:49 AM
This revision was automatically updated to reflect the committed changes.