Page MenuHomePhabricator

[InstCombine] Remove hasOneUse check for pow(C,x) -> exp2(log2(C)*x)
ClosedPublic

Authored by foad on Tue, May 5, 4:48 AM.

Details

Summary

I don't think there's any good reason not to do this transformation when
the pow has multiple uses.

Diff Detail

Event Timeline

foad created this revision.Tue, May 5, 4:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, May 5, 4:48 AM
lebedev.ri accepted this revision.Tue, May 5, 5:05 AM
lebedev.ri added a subscriber: lebedev.ri.

Seems reasonable.

This revision is now accepted and ready to land.Tue, May 5, 5:05 AM
xbolva00 accepted this revision.Tue, May 5, 5:59 AM

Yeah

This revision was automatically updated to reflect the committed changes.