diff options
author | Daniel RodrÃguez Troitiño <drodriguez@users.noreply.github.com> | 2025-03-14 13:43:48 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-03-14 13:43:48 -0700 |
commit | 8413f4d837a96458104f63bab72c751b8285a458 (patch) | |
tree | fca52ab4af3275ccdec2ca5188be2efbd77c1ca7 /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | 456963de9690a62d47c7a823cbc66c5a0f3d77d1 (diff) | |
download | llvm-8413f4d837a96458104f63bab72c751b8285a458.zip llvm-8413f4d837a96458104f63bab72c751b8285a458.tar.gz llvm-8413f4d837a96458104f63bab72c751b8285a458.tar.bz2 |
[llvm-objcopy] Apply encryptable offset to first segment, not section (#130517)
Bug introduced #120995. The LLD code calculates the "size" of the Mach-O
headers, and then uses that size to place the segments, but the `__TEXT`
section stays at `fileoff` zero. When I wrote the code into llvm-objcopy
I calculated the extra space into the initial offset, which moved all
the sections back 1 page.
Besides the modified test checking for the right `fileoff` values of the
sections and the segments, I also manually checked the generated
binaries after `llvm-objcopy` using `dyld_info`, as the bug report
suggested.
Fixes #130472
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions