Page MenuHomePhabricator

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

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

Details

Summary

Additional sanity checks were added to get.active.lane.mask'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.Wed, Sep 16, 8:44 AM
SjoerdMeijer requested review of this revision.Wed, Sep 16, 8:44 AM
dmgreen accepted this revision.Wed, Sep 16, 8:46 AM

Sounds like a good idea in the short term. Thanks

This revision is now accepted and ready to land.Wed, Sep 16, 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)Wed, Sep 16, 9:12 AM