This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU/SI: Add a helper for creating aliases for the _e32 instructions
ClosedPublic

Authored by tstellarAMD on Oct 2 2015, 12:53 PM.

Details

Summary

We are currently only using these aliases for VOPC instructions,
but this helper will make it easier to use them everywhere.

These aliases allow for the automatic matching of instructions
with forced 32-bit encoding. Eventually, we should be able to remove
the custom C++ logic we have for this in the assembler.

Diff Detail

Repository
rL LLVM

Event Timeline

tstellarAMD retitled this revision from to AMDGPU/SI: Add a helper for creating aliases for the _e32 instructions.
tstellarAMD updated this object.
tstellarAMD added a reviewer: arsenm.
tstellarAMD added a subscriber: llvm-commits.
arsenm added inline comments.Oct 2 2015, 1:02 PM
lib/Target/AMDGPU/SIInstrInfo.td
1247 ↗(On Diff #36392)

I don't see where this is used anywhere

tstellarAMD added inline comments.Oct 2 2015, 1:23 PM
lib/Target/AMDGPU/SIInstrInfo.td
1247 ↗(On Diff #36392)

ResultInst is a field defined by the parent class: InstAlias. The AsmMatcher backend uses it for creating aliases.

arsenm accepted this revision.Oct 2 2015, 1:28 PM
arsenm edited edge metadata.

LGTM

This revision is now accepted and ready to land.Oct 2 2015, 1:28 PM
This revision was automatically updated to reflect the committed changes.