[PATCH] D132345: [JITLink] Add support for symbol aliases.

[Lang Hames via Phabricator via llvm-commits]

lhames added a comment.

A couple of quick notes:

1. This still needs some more unit tests.

2. All the manual work to keep aliases consistently located is pretty gross. The main motivation for all this is that it would allow plugin writers to manipulate aliases separately. E.g. Even given "foo is an alias for bar", a plugin writer could choose to redirect those symbols to different places (e.g. to count entries via the "foo" and "bar" entrypoint differently). If we're willing to give that ability up we could potentially put //all// of the alias info (the names, sizes, linkages, scopes, etc.) into the AliasInfos side table and only keep the root Symbol in the graph, eliminating the consistency-maintenance work.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132345/new/

https://reviews.llvm.org/D132345