[DAG] SimplifyDemandedBits OR(Op0,Op1) - attempt to simplify Op1 again once we have the known bits of Op0
AbandonedPublic
Actions

Authored by RKSimon on Jun 12 2022, 7:05 AM.

Details

Reviewers

foad
spatel
craig.topper

Summary

Noticed while investigating the regressions in D125836

@foad You tried something similar on D87464 but weren't accounting for the already demanded bits of Op0 - the AND(Op0,Op1) case might need something similar.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

RKSimon created this revision.Jun 12 2022, 7:05 AM

Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2022, 7:05 AM

Herald added subscribers: kosarev, StephenFan, kerbowa and 2 others. · View Herald Transcript

RKSimon requested review of this revision.Jun 12 2022, 7:05 AM

Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2022, 7:05 AM

Harbormaster completed remote builds in B169310: Diff 436222.Jun 12 2022, 8:20 AM

Is this the same as D126117?

Yes it is - I've been trawling through my old branches and didn't realise I'd proposed this last month!

Revision Contents
Changeset List

Path

Size

llvm/

lib/

CodeGen/

SelectionDAG/

TargetLowering.cpp

14 lines

test/

CodeGen/

AMDGPU/

permute.ll

18 lines

Diff 436222

View Options

llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp