This is an archive of the discontinued LLVM Phabricator instance.

InstCombine: Add fma + fabs/fneg transforms
ClosedPublic

Authored by arsenm on Dec 18 2016, 3:17 PM.

Details

Summary

fma (fneg x), (fneg y), z -> fma x, y, z
fma (fabs x), (fabs x), z -> fma x, x, z

Diff Detail

Event Timeline

arsenm updated this revision to Diff 81897.Dec 18 2016, 3:17 PM
arsenm retitled this revision from to InstCombine: Add fma + fabs/fneg transforms.
arsenm updated this object.
arsenm added a subscriber: llvm-commits.
arsenm updated this revision to Diff 81898.Dec 18 2016, 3:17 PM
arsenm edited edge metadata.

Remove unrelated test change

majnemer added inline comments.
lib/Transforms/InstCombine/InstCombineCalls.cpp
1568–1569

Formatting looks weird.

1579–1580

Formatting looks weird.

arsenm updated this revision to Diff 81900.Dec 18 2016, 3:46 PM

clang-format

majnemer accepted this revision.Jan 2 2017, 2:26 PM
majnemer added a reviewer: majnemer.

LGTM

This revision is now accepted and ready to land.Jan 2 2017, 2:26 PM
arsenm closed this revision.Jan 2 2017, 8:43 PM

r290859