This is an archive of the discontinued LLVM Phabricator instance.

[TargetTransformInfo][NFCI] Add TargetTransformInfo::isZExtFree.
ClosedPublic

Authored by sanjoy on Apr 21 2015, 6:26 PM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

sanjoy updated this revision to Diff 24190.Apr 21 2015, 6:26 PM
sanjoy retitled this revision from to [TargetTransformInfo][NFCI] Add TargetTransformInfo::isZExtFree..
sanjoy updated this object.
sanjoy edited the test plan for this revision. (Show Details)
sanjoy added reviewers: mcrosier, chandlerc.
sanjoy added a subscriber: Unknown Object (MLST).
mcrosier accepted this revision.Apr 22 2015, 7:58 AM
mcrosier edited edge metadata.

LGTM. Probably doesn't need to be said, but please make sure the followup commit includes a test case that exercise this code.

This revision is now accepted and ready to land.Apr 22 2015, 7:58 AM
This revision was automatically updated to reflect the committed changes.

LGTM. Probably doesn't need to be said, but please make sure the followup commit includes a test case that exercise this code.

I reverted this in r252948 because currently it is unexercised and untested. The change to LSR it was supposed to support was reverted a while back (it was buggy), and I don't think I'll have time to check in a fixed version of that LSR change anytime soon.