This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Add outer product instructions for MMA
ClosedPublic

Authored by saghir on Sep 21 2020, 1:49 PM.

Details

Summary

This patch adds outer product instructions for MMA, including related infrastructure, and their tests.

Depends on D84968.

Diff Detail

Event Timeline

saghir created this revision.Sep 21 2020, 1:49 PM
saghir requested review of this revision.Sep 21 2020, 1:49 PM
saghir retitled this revision from [PowerPC] Add instruction definitions for MMA to [PowerPC] Add outer product instructions for MMA.Sep 21 2020, 1:52 PM
saghir edited the summary of this revision. (Show Details)
saghir edited the summary of this revision. (Show Details)
saghir edited the summary of this revision. (Show Details)
amyk added a subscriber: amyk.Sep 22 2020, 4:35 PM
amyk added inline comments.
llvm/lib/Target/PowerPC/PPCInstrPrefix.td
979

s/versioon/version

1294

s/aaccumlating/accumulating

llvm/test/MC/PowerPC/ppc64-encoding-ISA31.s
35

This is minor (and may not matter), but maybe we can align the # encoding lines together.

saghir updated this revision to Diff 294988.Sep 29 2020, 7:46 AM

Rebased after dependant patches were committed.

bsaleil accepted this revision.Sep 29 2020, 11:14 AM

LGTM, thanks. I think the patch can be committed once the typos are fixed.

This revision is now accepted and ready to land.Sep 29 2020, 11:14 AM
saghir updated this revision to Diff 295082.Sep 29 2020, 12:11 PM

Fixed typos and indentation.

amyk accepted this revision.Sep 29 2020, 1:57 PM

LGTM.

This revision was automatically updated to reflect the committed changes.