Page MenuHomePhabricator

[ARM][MVE] Enable tail-predication by default
ClosedPublic

Authored by SjoerdMeijer on Sep 22 2020, 6:39 AM.

Details

Summary

We have been running tests/benchmarks downstream with tail-predication enabled for some time now and this behaves as expected: we are not aware of any correctness issues, and this performs better across the board than disabling tail-predication.

So, if we can get D88086 out of the way, I think it is time to flip the switch.

Diff Detail

Event Timeline

SjoerdMeijer created this revision.Sep 22 2020, 6:39 AM
SjoerdMeijer requested review of this revision.Sep 22 2020, 6:39 AM
dmgreen accepted this revision.Sep 28 2020, 3:58 AM

Sounds fantastic. The test updates look good to me. Reductions will be back to inloop soon, hopefully. Everything else out of the way, no remaining issues?

If so, LGTM

This revision is now accepted and ready to land.Sep 28 2020, 3:58 AM

Thanks Dave. With D88086 committed now, I don't think there's anything in our way anymore.

This revision was automatically updated to reflect the committed changes.