This is an archive of the discontinued LLVM Phabricator instance.

[SystemZ] Adjust cost functions for targets that use LI + LOC instead of IPM
ClosedPublic

Authored by jonpa on Aug 28 2018, 1:07 AM.

Details

Reviewers
uweigand
Summary

After recent improvements which makes better use of LOC instead of IPM, the TTI cost functions also needs to be updated to reflect this.

This involves extensions of and xor of i1.

The tests were updated so that for z13 the new costs are expected, while the old costs are still checked for on zEC12.

Diff Detail

Event Timeline

jonpa created this revision.Aug 28 2018, 1:07 AM
uweigand accepted this revision.Sep 12 2018, 6:28 AM

I agree, we need to update the cost model. Patch LGTM.

This revision is now accepted and ready to land.Sep 12 2018, 6:28 AM
jonpa closed this revision.Sep 13 2018, 11:53 PM

r342207.

test/Analysis/CostModel/SystemZ/cmp-ext-02.ll