Conversation
|
Thanks for your pull request, @WalterBright! Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. Testing this PR locallyIf you don't have a local development environment setup, you can use Digger to test this PR: dub run digger -- build "master + dmd#22723" |
| else version (X86) | ||
| public import rt.sections_osx_x86; | ||
| else version (AArch64) | ||
| public import rt.sections_darwin_rt; |
There was a problem hiding this comment.
Yes, it does:
https://github.com/dlang/dmd/blob/master/druntime/src/rt/sections_darwin_64.d
Should it exist?
There was a problem hiding this comment.
sections_darwin_64 != sections_darwin_rt
|
And LDC basically uses a generalized |
|
src/rt/sections.d is in druntime/mak/SRCS, which is fed to Makefile. How does gdc and ldc avoid compiling it? |
|
LDC doesn't use the Makefiles. The whole |
|
Is there any essential difference between the sections for AArch64 and X68_64 on the OSX? |
|
I don't recall any OTOH, but it's been more than 5 years that LDC comes with a macOS arm64 package, so the memory is anything but fresh. Edit: I've taken a quick look, and no, I can't spot any diff at all for arm64. |
|
@kinke thank you! |
kinke
left a comment
There was a problem hiding this comment.
Should work fine. FWIW, rt.sections_darwin_64 (LDC only uses that file - I guess it's just the base helpers) uses D_LP64 as inclusion guard. So I think ideally rt.sections_osx_x86_64 would be renamed (to rt.sections_osx_64 or so), to make clear that it's the generic 64-bit implementation for Apple targets.
|
ok I renamed the file |
As gdc and ldc compile druntime for OSX AArch64, it is a bit of a mystery why this is not represented in src/rt/sections.d ??
@ibuclaw @kinke please weigh in!