int3 added a comment.
I'm trying to add similar support to lld-macho, hence the question :)
Comment at: lld/COFF/LTO.cpp:183
+ [&](size_t task, const Twine &moduleName) {
+buf[task].first = moduleName.str();
return std::make_unique(
Any re
int3 added a comment.
lld/MachO changes lgtm
Comment at: lld/MachO/Driver.cpp:1386
// ld64 does something really weird. It attempts to realign the value to
the
-// page size, but assumes the the page size is 4K. This doesn't work with
+// page size, but assumes th
int3 requested changes to this revision.
int3 added a comment.
This revision now requires changes to proceed.
Clearing queue
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122568/new/
https://reviews.llvm.org/D122568
___
int3 added a comment.
The underlying test is still using just ld64 though. I think a proper fix be to
use LLD if it's available as the host linker?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122568/new/
https://reviews.llvm.org/D122568
___