Page MenuHomePhabricator

[DAG][PowerPC] Enable initial ISD::BITCAST SimplifyDemandedBits/SimplifyMultipleUseDemandedBits big-endian handling
ClosedPublic

Authored by RKSimon on Nov 28 2021, 9:20 AM.

Details

Summary

This patch begins extending handling for peeking through bitcast nodes to big-endian targets as well as the existing little-endian case.

Diff Detail

Event Timeline

RKSimon created this revision.Nov 28 2021, 9:20 AM
RKSimon requested review of this revision.Nov 28 2021, 9:20 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 28 2021, 9:20 AM

It might be better to address this before D114589

any comments?

lebedev.ri retitled this revision from [DAG] Enable initial ISD::BITCAST SimplifyDemandedBits/SimplifyMultipleUseDemandedBits big-endian handling to [DAG][PowerPC] Enable initial ISD::BITCAST SimplifyDemandedBits/SimplifyMultipleUseDemandedBits big-endian handling.Dec 1 2021, 3:01 AM
lebedev.ri added a subscriber: lebedev.ri.

Not really familiar with PPC, but the TargetLowering.cpp part makes sense to me.

nemanjai accepted this revision.Dec 1 2021, 2:59 PM

LGTM. The PPC changes are fine AFAICT.

This revision is now accepted and ready to land.Dec 1 2021, 2:59 PM
This revision was automatically updated to reflect the committed changes.