RFR(T): 8214206: Fix for JDK-8213419 is broken on 32-bit

[Doerr, Martin]

+1

I guess we only need to use log2_jint in c1_LIRGenerator_s390.cpp to fix build on your new companies most expensive platform ��

Best regards,
Martin


-----Original Message-----
From: hotspot-dev <hotspot-dev-bounces at openjdk.java.net> On Behalf Of Aleksey Shipilev
Sent: Donnerstag, 22. November 2018 16:00
To: Roland Westrelin <rwestrel at redhat.com>; hotspot-dev Source Developers <hotspot-dev at openjdk.java.net>
Subject: Re: RFR(T): 8214206: Fix for JDK-8213419 is broken on 32-bit

On 11/22/18 3:14 PM, Roland Westrelin wrote:
> http://cr.openjdk.java.net/~roland/8214206/webrev.00/
> 
> The log2_intptr(int x) and log2_intptr(intptr_t x) conflict on 32 bit.

I don't think conditional compilation is the answer here. It seems better to have declarations that
are guaranteed not to clash. For example:
  http://cr.openjdk.java.net/~shade/8214206/webrev.00/

This builds on x86_32 and x86_64 without problems. STATIC_ASSERTS help to find the jlong downcast
issue Goetz mentioned in his note, and I believe the answer is "unsigned long long". Roland, feel
free to use this patch either partially or in full.

-Aleksey