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

Repository
rL LLVM

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 ↗(On Diff #117071)
One of the following AMD GPU architecture specific OS ABIs:
412–413 ↗(On Diff #117071)

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 ↗(On Diff #117071)

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 ↗(On Diff #117571)

or -> ,

370 ↗(On Diff #117571)

or -> ,

This revision was automatically updated to reflect the committed changes.