[release/8.0-staging] Always keep global symbols on ApplePlatforms

[github-actions]

Backport of #99504 to release/8.0-staging

/cc @mikem8361 @am11

Customer Impact

• Customer reported
• Found internally

The Native AOT SOS support will not work on MacOS without this fix.

Regression

• Yes
• No

Testing

Manual testing by creating a Native AOT MacOS core dump with this change and running the Native AOT SOS support.

Risk

Low. It only changes the symbol stripping logic to not strip export symbols.

[dotnet-policy-service]

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

[jeffschwMSFT]

approved. we will treat this one as tell mode. please get a code review

[agocke]

LGTM

[jeffschwMSFT]

quick question before merging - what, if any, is the expected impact on output size? Marginal increase?

[jkotas]

quick question before merging - what, if any, is the expected impact on output size? Marginal increase?

@am11 Did you happen to check the binary size impact of this change?

[am11]

The problem was that on macOS, we weren't exporting DotNetRuntimeDebugHeader with the default StripSymbols=true. It was an unintentional behavior difference compared to linux and Windows.

@am11 Did you happen to check the binary size impact of this change?

Yes #99504 (comment) (32-48 bytes increase depending on alignment?). We are only exporting one symbol by default for executables:

runtime/src/coreclr/nativeaot/BuildIntegration/Microsoft.NETCore.Native.targets

Line 246 in e4fceb3

<IlcArg Condition="'$(_targetOS)' != 'win' and '$(DebuggerSupport)' != 'false'" Include="--export-dynamic-symbol:DotNetRuntimeDebugHeader" />

If user is setting IlcExportUnmanagedEntrypoints=true without OutputType=Exe, and the app has lots of exported symbols, then it will add 32-48 bytes for each. Those preconditions are unlikely and the size overhead is negligible.

[jeffschwMSFT]

failures are known issues