This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add simplification/combines for llvm.amdgcn.fma.legacy
ClosedPublic

Authored by foad on Oct 23 2020, 4:57 AM.

Details

Summary

This follows on from D89558 which added the new intrinsic and D88955
which added similar combines for llvm.amdgcn.fmul.legacy.

Diff Detail

Event Timeline

foad created this revision.Oct 23 2020, 4:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 23 2020, 4:57 AM
foad requested review of this revision.Oct 23 2020, 4:57 AM
arsenm added inline comments.Oct 23 2020, 7:54 AM
llvm/test/Transforms/InstCombine/AMDGPU/fma_legacy.ll
7

Test preserving flags?

48

Should also test that flags are preserved

foad updated this revision to Diff 300305.Oct 23 2020, 8:28 AM

Add tests for preserving fast math flags.

arsenm accepted this revision.Oct 23 2020, 8:29 AM
This revision is now accepted and ready to land.Oct 23 2020, 8:29 AM
This revision was landed with ongoing or failed builds.Oct 23 2020, 8:31 AM
This revision was automatically updated to reflect the committed changes.