This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] performAddCombine should run after DAG is legalized.
ClosedPublic

Authored by FarhanaAleen on May 1 2018, 4:08 PM.

Details

Summary

performAddCombine should run after DAG is legalized; Otherwise generic optimization in the DAGCombiner can optimize an addcarry+trunc into an addcarry instruction with illegal types.

Diff Detail

Repository
rL LLVM

Event Timeline

FarhanaAleen created this revision.May 1 2018, 4:08 PM
rampitec accepted this revision.May 1 2018, 4:25 PM

LGTM

This revision is now accepted and ready to land.May 1 2018, 4:25 PM
This revision was automatically updated to reflect the committed changes.