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