On Fri, 15 Nov 2024 06:20:53 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:
>> Yes, I think it's an acceptable workaround to disable attribute-warning >> (It's one of the solutions I mentioned earlier, but I'm guessing you mean >> more aggressively). LTO is not the typical way of compiling HotSpot, so this >> shouldn't be a problem, until LTO becomes viable enough that it becomes the >> standard, but that's probably going to be a long time in the future > > As mentioned in the JBS issue, some of my problems are a result of the devkit > I'm using not putting > the lto-plugin in the necessary spot. So I think I'm now where @MBaesken is > at. > > I worry that LTO could become the default fairly quickly if it really does > show good results, once the > kinds of issues @TheShermanTanker mentioned are fixed. That would make the > poisoning of various > functions effectively cease to work if we disable that feature in LTO builds. > > Also, does disabling of LTO for that one file actually work? The gcc docs > say that mixing LTO and > non-LTO objects isn't possible with GNU binutils, and it just falls back to > nolto-rel: > https://gcc.gnu.org/onlinedocs/gcc-13.2.0/gcc/Link-Options.html#:~:text=During%20the%20incremental,link%2Dtime%20optimization > Maybe it works with the gold linker? Which seems to be the default in the > devkit I'm using. I misspoke a bit earlier, I do not think LTO will become the Standard way of compiling HotSpot, even if all the issues with it are fixed. It simply takes too long, which I think is the main reason Erik mentioned why it is not used for the product today. I intend to make LTO viable, for those who want the extra performance at the expense of link times, but it is unlikely to become the default The documentation linked above is for incremental linking, it is indeed possible to link LTO and non LTO object files together if linking normally and not incrementally, otherwise LTO would already fail horribly on assembly files (Which are always assembled directly into non LTO object files), and HotSpot contains many assembly source files (This might be an issue for Magnus and his static Java, though, since I think that relies on incremental linking). However, that said, I am not fond of disabling LTO for any specific file, since that will cause a decline in the resulting assembly quality I've been looking at alternative ways to implement poisoning of methods we don't want people to be using that don't rely on the warning attribute, however that hasn't been successful ([8313396](https://bugs.openjdk.org/browse/JDK-8313396)). I've checked the gcc bug tracker, and the current warnings are apparently a hard to fix bug in gcc, since pragmas are not saved into the GIMPLE when LTO is enabled. I'll list the poisoning thing as a blocker for making LTO viable, so the issues with LTO won't be fixed until the poisoning works with LTO. I'll look into poisoning again once my Windows fixes make it in As a workaround that doesn't rely on disabling poisoning whatsoever, you could NOINLINE code that contains an ALLOW_C_FUNCTION, but that would likewise also inhibit optimizations for the JVM ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22069#discussion_r1843341779
