RFR: 8236179: C1 register allocation error with T_ADDRESS

[Doerr, Martin]

Hi everybody,

thanks for fixing this issue.

I guess it's currently used on some platforms, but I think we should fix it for all platforms. Otherwise it will break when using the parts which were only fixed for x86.

Here's my proposal:
http://cr.openjdk.java.net/~mdoerr/8236179_C1_T_ADDRESS/webrev.01/

I'll run tests on more platforms.

Best regards,
Martin


> -----Original Message-----
> From: hotspot-compiler-dev <hotspot-compiler-dev-
> bounces at openjdk.java.net> On Behalf Of Roland Westrelin
> Sent: Donnerstag, 19. Dezember 2019 15:15
> To: Aditya Mandaleeka <adityam at microsoft.com>; hotspot compiler
> <hotspot-compiler-dev at openjdk.java.net>
> Cc: shenandoah-dev <shenandoah-dev at openjdk.java.net>
> Subject: Re: RFR: 8236179: C1 register allocation error with T_ADDRESS
> 
> 
> Hi Aditya,
> 
> AFAIK, it's a requirement that the patch be posted on the openjdk
> infrastructure. So here it is:
> 
> http://cr.openjdk.java.net/~roland/8236179/webrev.00/
> 
> The change looks good to me but it would be good to check whether
> architectures other than x86 need a similar change.
> 
> Roland.