This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Assembler support for exp
ClosedPublic

Authored by arsenm on Nov 16 2016, 12:06 PM.

Details

Summary

compr is not currently parsed (or printed) correctly,
but that should probably be fixed along with
intrinsic changes.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 78242.Nov 16 2016, 12:06 PM
arsenm retitled this revision from to AMDGPU: Assembler support for exp.
arsenm updated this object.
arsenm added a subscriber: llvm-commits.
vpykhtin accepted this revision.Nov 18 2016, 4:41 AM
vpykhtin edited edge metadata.

LGTM.

Is "compr" bit going to be supported later?

test/MC/AMDGPU/exp.s
86

should "compr" on/off variant be added here?

This revision is now accepted and ready to land.Nov 18 2016, 4:41 AM

LGTM.

Is "compr" bit going to be supported later?

Sorry, I overlooked "compr" comment in the description, please submit.

artem.tamazov accepted this revision.Nov 18 2016, 7:02 AM
artem.tamazov edited edge metadata.

Looks good. Maybe it's worth to add tests for some gfx8 target, but this is up to you.

arsenm closed this revision.Dec 5 2016, 12:53 PM

r288698