This is an archive of the discontinued LLVM Phabricator instance.

Relax trivial cast requirements in CallPromotionUtils
ClosedPublic

Authored by scott.linder on Oct 2 2018, 11:22 AM.

Details

Summary

Broken out of https://reviews.llvm.org/D52741

Once that is committed there will be some tests exercising the change, but I can also update the PGO tests to prove the change if needed.

Diff Detail

Event Timeline

scott.linder created this revision.Oct 2 2018, 11:22 AM

Sorry, moved a little fast and grabbed the wrong patch at first.

arsenm accepted this revision.Oct 2 2018, 6:42 PM

LGTM. I think it should be possible to come up with a test using IndirectCallPromotion

This revision is now accepted and ready to land.Oct 2 2018, 6:42 PM

Add test using IndirectCallPromotion

Remove -debug flag from test

This revision was automatically updated to reflect the committed changes.