Page MenuHomePhabricator

[AMDGPU] Handle undef old operand in DPP combine
ClosedPublic

Authored by rampitec on Oct 10 2019, 9:47 AM.

Diff Detail

Event Timeline

rampitec created this revision.Oct 10 2019, 9:47 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 10 2019, 9:47 AM
vpykhtin accepted this revision.Oct 10 2019, 1:58 PM

LGTM.

llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp
181–183

ok, I missed the case when there is no defining instruction. It is not very good that the def is searched again, but lets submit this for the fix.

This revision is now accepted and ready to land.Oct 10 2019, 1:58 PM
This revision was automatically updated to reflect the committed changes.