This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Add use check in v_fma combine.
ClosedPublic

Authored by tsymalla on Sep 29 2022, 12:59 AM.

Details

Summary

In D132837, an existing v_fma combine was extended to regard nested
fma instructions. Originally, the inner FMA was checked for being used
only once. In its current state, this check is missing, which causes
some regressions.

In this patch, this check was added.

Diff Detail

Event Timeline

tsymalla created this revision.Sep 29 2022, 12:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 29 2022, 12:59 AM
tsymalla requested review of this revision.Sep 29 2022, 12:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 29 2022, 12:59 AM
foad accepted this revision.Sep 29 2022, 1:29 AM
This revision is now accepted and ready to land.Sep 29 2022, 1:29 AM
This revision was landed with ongoing or failed builds.Sep 29 2022, 3:25 AM
This revision was automatically updated to reflect the committed changes.