[PATCH] D97007: [lld-macho] Define __mh_*_header synthetic symbols.
Shoaib Meenai via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 30 17:02:13 PDT 2021
smeenai added a comment.
In D97007#2636289 <https://reviews.llvm.org/D97007#2636289>, @int3 wrote:
> 2. We could make MachHeaderSection inherit from an InputSection rather than a OutputSection (SyntheticSection), and then have it be part of the `__TEXT` MergedOutputSection. This is more architecturally consistent, but would probably require a good deal of refactoring.
I don't have the full context here, but does this tie into the larger questions of whether SyntheticSections should be InputSections or OutputSections? https://bugs.llvm.org/show_bug.cgi?id=49780
To be clear, I'm not saying we're at the point where we should consider that refactoring yet; I'm just wondering if it's another data point that should be included when we eventually make that decision :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97007/new/
https://reviews.llvm.org/D97007
More information about the llvm-commits
mailing list