This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Add ELFOSABI_AMDGPU_MESA3D
ClosedPublic

Authored by kzhuravl on Sep 28 2017, 3:53 PM.

Diff Detail

Event Timeline

kzhuravl created this revision.Sep 28 2017, 3:53 PM
t-tye added inline comments.Sep 28 2017, 9:08 PM
docs/AMDGPUUsage.rst
404
One of the following AMD GPU architecture specific OS ABIs:
412–413

Is it necessary to mention "for AMD GCN GPUs (GFX6+)" as the other ones above do not, and presumably it would be ok for the R600 architecture to use this ABI too if it wanted to?

425–426

Same comment as above.

include/llvm/Object/ELFObjectFile.h
1071–1076 ↗(On Diff #117071)

Same comment as in other patch.

kzhuravl updated this revision to Diff 117185.Sep 29 2017, 10:56 AM
kzhuravl marked 4 inline comments as done.

Address review feedback.

t-tye accepted this revision.Sep 29 2017, 3:08 PM

LGTM

This revision is now accepted and ready to land.Sep 29 2017, 3:08 PM
t-tye accepted this revision.Oct 3 2017, 2:15 PM

LGTM (couple minor documentation suggestions)

docs/AMDGPUUsage.rst
367

or -> ,

370

or -> ,

This revision was automatically updated to reflect the committed changes.