Page MenuHomePhabricator

[ARM][MVE] tail-predication: predicate new checks on force-enabled option

Authored by SjoerdMeijer on Sep 16 2020, 8:44 AM.



Additional sanity checks were added to's second argument, the loop tripcount/elementcount, in rG676febc044ecbb27f8. Like the other (overflow) checks, skip this if tail-predication is forced.

Diff Detail

Event Timeline

SjoerdMeijer created this revision.Sep 16 2020, 8:44 AM
SjoerdMeijer requested review of this revision.Sep 16 2020, 8:44 AM
dmgreen accepted this revision.Sep 16 2020, 8:46 AM

Sounds like a good idea in the short term. Thanks

This revision is now accepted and ready to land.Sep 16 2020, 8:46 AM

Thanks Dave, just for completeness, uploading a new diff with the codegen changes gone, which shouldn't have been there.

SjoerdMeijer edited the summary of this revision. (Show Details)Sep 16 2020, 9:12 AM