This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Add most d16 load/store instruction definitions
ClosedPublic

Authored by arsenm on Aug 31 2017, 1:54 PM.

Details

Summary

Doesn't include the tied operand necessary for the loads,
but is enough for the assembler to work.

Diff Detail

Event Timeline

arsenm created this revision.Aug 31 2017, 1:54 PM
rampitec added inline comments.Aug 31 2017, 10:04 PM
lib/Target/AMDGPU/BUFInstructions.td
1593

Do you mind to align this block as around?

lib/Target/AMDGPU/DSInstructions.td
467

And also align here and below.

rampitec added inline comments.Sep 1 2017, 10:58 AM
lib/Target/AMDGPU/DSInstructions.td
468

Here as well...

lib/Target/AMDGPU/FLATInstructions.td
917

Here as well...

923

Here as well...

978

Here as well...

1022

Here as well...

arsenm updated this revision to Diff 113559.Sep 1 2017, 11:19 AM

Formatting

rampitec added inline comments.Sep 1 2017, 11:22 AM
lib/Target/AMDGPU/FLATInstructions.td
1023

The last place ;)

arsenm updated this revision to Diff 113564.Sep 1 2017, 11:32 AM

Format

lib/Target/AMDGPU/FLATInstructions.td
1023

The problem with these is the new instructions are longer, so it requires reformatting everything else around it (which is why I didn't want to have any of these aligned in the first place)

This revision is now accepted and ready to land.Sep 1 2017, 11:33 AM
arsenm closed this revision.Sep 1 2017, 11:37 AM

r312347