This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Pull fneg/fabs out of a select if free
ClosedPublic

Authored by arsenm on Dec 19 2016, 6:58 PM.

Details

Summary

Allows better source modifier usage.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 82056.Dec 19 2016, 6:58 PM
arsenm retitled this revision from to DAGCombiner: Pull fneg/fabs out of a select if free.
arsenm updated this object.
arsenm added a reviewer: escha.
arsenm added a subscriber: llvm-commits.
arsenm updated this revision to Diff 82346.Dec 22 2016, 9:10 AM
arsenm edited edge metadata.

Don't require splat constants

arsenm updated this revision to Diff 82391.Dec 22 2016, 5:37 PM
arsenm edited edge metadata.

Fix fabs

arsenm updated this revision to Diff 82392.Dec 22 2016, 6:01 PM
arsenm edited edge metadata.

Remove parts that only will help after more combines are added

arsenm updated this revision to Diff 82953.Jan 3 2017, 2:34 PM
arsenm retitled this revision from DAGCombiner: Pull fneg/fabs out of a select if free to AMDGPU: Pull fneg/fabs out of a select if free.
arsenm edited edge metadata.

Move to target code

nhaehnle accepted this revision.Jan 11 2017, 2:10 PM
nhaehnle added a reviewer: nhaehnle.

LGTM

This revision is now accepted and ready to land.Jan 11 2017, 2:10 PM
arsenm closed this revision.Jan 11 2017, 4:08 PM

r291729