Page MenuHomePhabricator

[X86] Enable BT to be used in place of TEST for single bit checks under optsize
ClosedPublic

Authored by craig.topper on Feb 15 2018, 11:18 AM.

Details

Summary

We already do this for 64-bit when it won't fit into a 64-bit AND/TEST's immediate field. This adds an additional qualifier to do it for any single bit constant under optsize

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper created this revision.Feb 15 2018, 11:18 AM

Don't do this immediates that fit in a byte, It won't save anything

RKSimon accepted this revision.Feb 15 2018, 12:16 PM

LGTM - don't suppose we have any load fold tests for BT do we?

This revision is now accepted and ready to land.Feb 15 2018, 12:16 PM

Doesn't look like we do. I'll look into that.

This revision was automatically updated to reflect the committed changes.