RFR: 8017234: Hotspot should stop using mapfiles
Magnus Ihse Bursie
ihse at openjdk.org
Thu Feb 22 09:39:53 UTC 2024
On Thu, 22 Feb 2024 07:50:34 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
> Thanks for doing this! This is a good improvement IMO.
Thanks!
> If the `SUNWprivate_1.1` part of the symbols is still needed, it can be added using `--default-symver` ld parameter.
Ah, good to know. In the spirit of making as minimal changes as possible in this PR, I will add it.
> You might want to run tier1-5 tests; compilation-related changes are often detected by higher-level builds and tests only.
I can run tier 4 and 5 also. But once again, the entire point of this PR being so convoluted is that there should be a minimum of differences in compilation.
On macOS and Windows, the binaries before and after this patch are almost identical; just a few hundred assembler instructions have been reordered slightly. (Why this is even so I don't know, but in the end I did not care to dig into the details.)
On Linux, more code is shuffled around, making for a greater bit-by-bit difference. Still, the symbol tables are (almost) identical, and these changes should only affect symbol visibility, so I believe this is mostly an artifact of how the linker organises stuff.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17955#issuecomment-1959051714
More information about the build-dev
mailing list