This is an archive of the discontinued LLVM Phabricator instance.

[SelectionDAG] Teach simplifyDemandedBits to handle shifts by constant splat vectors
ClosedPublic

Authored by craig.topper on Sep 9 2017, 11:41 AM.

Details

Summary

This teach simplifyDemandedBits to handle constant splat vector shifts.

This required changing all the uses of getZExtValue to getLimitedValue since we can't rely on legalization using getShiftAmountTy for the shift amount.

I believe there may have been a bug in the ((X << C1) >>u ShAmt) handling where we didn't check if the inner shift was too large. I've fixed that here, but maybe I should split that out.

I had to add new patterns to ARM because the zext/sext the patterns were trying to look for got turned into an any_extend with this patch. Happy to split that out too, but not sure how to test without this change.

Diff Detail

Event Timeline

craig.topper created this revision.Sep 9 2017, 11:41 AM

Missed one ARM test change.

RKSimon edited edge metadata.

Adding ARM specialists

RKSimon added inline comments.Sep 9 2017, 1:06 PM
lib/CodeGen/SelectionDAG/TargetLowering.cpp
784

Which do you think is better - getLimitedValue or using APInt::uge() which we do in other places?

Change to use APInt::uge in most places. There was one place where getLimitedValue was still convenient for a compound if condition.

LGTM - @t.p.northover @rengolin are you alright with the ARMInstrNEON.td fixes?

Any ARM specialist wanting to comment on the ARMInstrNEON.td change?

efriedma accepted this revision.Sep 25 2017, 11:55 AM

LGTM.

lib/Target/ARM/ARMInstrNEON.td
5537

I guess the zext/sext patterns will still trigger if the value has multiple uses...?

This looks fine.

This revision is now accepted and ready to land.Sep 25 2017, 11:55 AM
This revision was automatically updated to reflect the committed changes.