This is an archive of the discontinued LLVM Phabricator instance.

Add support for writing binary ELF doc nodes
ClosedPublic

Authored by tpr on Apr 17 2023, 11:47 PM.

Details

Summary

Will be used for a front-end compiler
https://github.com/GPUOpen-Drivers/llpc
to insert a binary blob value into AMDGPU PAL metadata.

Change-Id: I90685883d1478335937218e6b1a569b1b26dc05a

Diff Detail

Event Timeline

tpr created this revision.Apr 17 2023, 11:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2023, 11:47 PM
tpr requested review of this revision.Apr 17 2023, 11:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2023, 11:47 PM
tpr added a reviewer: jmnash.
scott.linder accepted this revision.Apr 18 2023, 11:59 AM

LGTM with 1 small nit

llvm/include/llvm/BinaryFormat/MsgPackDocument.h
380

Local should be CamelCase

This revision is now accepted and ready to land.Apr 18 2023, 11:59 AM
This revision was landed with ongoing or failed builds.Apr 20 2023, 12:25 AM
This revision was automatically updated to reflect the committed changes.