RFR: 8256486: Linux/Windows-x86 builds broken after JDK-8254231 [v7]
Jorn Vernee
jvernee at openjdk.java.net
Wed Nov 25 12:31:59 UTC 2020
On Wed, 25 Nov 2020 12:16:01 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Pass in thread instead of rematerializing it.
>
> src/hotspot/cpu/x86/universalNativeInvoker_x86.cpp line 33:
>
>> 31:
>> 32: void ProgrammableInvoker::Generator::generate() {
>> 33: #ifdef _LP64
>
> Instead of introducing ifdefs, I'd prefer to see `universalNativeInvoker_x86.cpp` split into `universalNativeInvoker_x86_64.cpp` (current code) and `universalNativeInvoker_x86_32.cpp` (populated with dummy impelmentations).
Yeah, makes sense. The old code had ifdefs, but splitting seems cleaner to me as well.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1266
More information about the core-libs-dev
mailing list