Page MenuHomePhabricator

[AMDGPU] Handle undef old operand in DPP combine
ClosedPublic

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

Diff Detail

Event Timeline

rampitec created this revision.Thu, Oct 10, 9:47 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Oct 10, 9:47 AM
vpykhtin accepted this revision.Thu, Oct 10, 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.Thu, Oct 10, 1:58 PM
This revision was automatically updated to reflect the committed changes.