This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Refactor VOP1 and VOP2 instruction TD definitions
ClosedPublic

Authored by vpykhtin on Sep 19 2016, 11:35 AM.

Diff Detail

Event Timeline

vpykhtin updated this revision to Diff 71857.Sep 19 2016, 11:35 AM
vpykhtin retitled this revision from to [AMDGPU] Refactor VOP1 instruction TD definitions.
vpykhtin updated this object.
vpykhtin set the repository for this revision to rL LLVM.
vpykhtin added a subscriber: nhaustov.
artem.tamazov accepted this revision.Sep 19 2016, 11:46 AM
artem.tamazov edited edge metadata.

I do not see anything bad))

This revision is now accepted and ready to land.Sep 19 2016, 11:46 AM
vpykhtin updated this revision to Diff 72170.Sep 22 2016, 7:38 AM
vpykhtin retitled this revision from [AMDGPU] Refactor VOP1 instruction TD definitions to [AMDGPU] Refactor VOP1 and VOP2 instruction TD definitions.
vpykhtin updated this object.
vpykhtin edited edge metadata.
vpykhtin removed rL LLVM as the repository for this revision.

Tom, can you please run additional testing on this patch, its located at

https://github.com/vpykhtin/llvm/tree/vop12_td

for convinience.

Thanks,
Valery

tstellarAMD accepted this revision.Sep 22 2016, 6:50 PM
tstellarAMD edited edge metadata.

LGTM. OCL/OGL tests pass.

This revision was automatically updated to reflect the committed changes.