This is an archive of the discontinued LLVM Phabricator instance.

[TTI] Add generic SADDSAT/SSUBSAT costs
ClosedPublic

Authored by RKSimon on Jan 25 2019, 6:51 AM.

Details

Summary

Add generic costs calculation for SADDSAT/SSUBSAT intrinsics, this uses generic costs for sadd_with_overflow/ssub_with_overflow, an extra sign comparison + a selects based on the sign/overflow.

This completes PR40316

Diff Detail

Repository
rL LLVM

Event Timeline

RKSimon created this revision.Jan 25 2019, 6:51 AM
nikic accepted this revision.Jan 26 2019, 12:25 PM

LGTM, assuming that we're going to drop the scalarization for vector sadd.add/sub in the near feature.

This revision is now accepted and ready to land.Jan 26 2019, 12:25 PM
This revision was automatically updated to reflect the committed changes.