Page MenuHomePhabricator

[llvm][TextAPI] adding inlining reexported libraries support

Authored by cishida on Sep 16 2019, 7:31 PM.



[llvm][TextAPI] adding inlining reexported libraries support

  • this patch adds reader/writer support for MachO tbd files.

The usecase is to represent reexported libraries in top level library
that won't need to exist for linker indirection because all of the
needed content will be inlined in the same document.

Diff Detail

Event Timeline

cishida created this revision.Sep 16 2019, 7:31 PM

Does nm print now all symbols - including the inlined ones? That would be an unexpected change, because we don't do the same for MachOs that have re-exported frameworks. I think this feature should be guarded by an option.

cishida updated this revision to Diff 254028.Mar 31 2020, 3:51 PM

Updating this patch to just create inlining framework support, and I will attach lib/Object & nm support in a seperate patch

cishida retitled this revision from [TextAPI] Add Multiple Document Support to TBDv3 to [llvm][TextAPI] adding inlining reexported libraries support.Mar 31 2020, 11:47 PM
cishida edited the summary of this revision. (Show Details)
ributzka accepted this revision.Apr 1 2020, 8:49 AM


This revision is now accepted and ready to land.Apr 1 2020, 8:49 AM
jhenderson added inline comments.Apr 2 2020, 12:02 AM

Nit here and below in comments - missing trailing full stops.


Nit: too many blank lines


There's a lot of usage of auto in this and the other files, and I'm pretty sure this is stretching LLVM's coding standard on the topic - the type isn't obvious. See

cishida marked 4 inline comments as done.Apr 2 2020, 12:13 PM

the lint corrections will be/have been made in the revision that lands.


got it! I will update these files to update these usages in a separate patch to conform to the standard.

This revision was automatically updated to reflect the committed changes.
cishida marked an inline comment as done.