[EXTERNAL] Re: RFR(S) 8251271- C2: Compile::_for_igvn list is corrupted after RenumberLiveNodes

[Nhat Nguyen]

Thank you Christian for taking a look at the patch! I'll be sure to ask a sponsor to assign the bug for me in the future.

Nhat

-----Original Message-----
From: Christian Hagedorn <christian.hagedorn at oracle.com> 
Sent: Thursday, August 27, 2020 7:54 AM
To: Nhat Nguyen <honguye at microsoft.com>; hotspot-compiler-dev at openjdk.java.net
Subject: [EXTERNAL] Re: RFR(S) 8251271- C2: Compile::_for_igvn list is corrupted after RenumberLiveNodes

Hi Nhat

Looks good to me!

Just make sure you that next time you assign the bug to you or a sponsor and/or leave a comment that you intend to work on it to avoid the possibility of some duplicated work (was no problem in this case) ;-)

Best regards,
Christian

On 26.08.20 20:55, Nhat Nguyen wrote:
> Hi hotspot-compiler-dev,
> 
> Please review the following patch to address 
> https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugs
> .openjdk.java.net%2Fbrowse%2FJDK-8251271&data=02%7C01%7Chonguye%40
> microsoft.com%7C52cd8fdc324d4e86326b08d84a991fef%7C72f988bf86f141af91a
> b2d7cd011db47%7C1%7C0%7C637341368808595657&sdata=j3YM%2BfxaO8KK1Ie
> CbKCPRYjwmGVfCUBrNULXDCJcUxM%3D&reserved=0
> The bug is currently assigned to Christian Hagedorn, but he was supportive of me submitting the patch instead.
> I have run hotspot/tier1 and jdk/tier1 tests to make sure that the change is working as intended.
> 
> webrev: 
> https://nam06.safelinks.protection.outlook.com/?url=http:%2F%2Fcr.open
> jdk.java.net%2F~burban%2Fnhat%2FJDK-8251271%2Fwebrev.00%2F&data=02
> %7C01%7Chonguye%40microsoft.com%7C52cd8fdc324d4e86326b08d84a991fef%7C7
> 2f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637341368808595657&sdata
> =PsHUTKZf9MrvM8Et5zPXsXpj32mfsGfBRGoZATjOv0I%3D&reserved=0
> 
> Thank you,
> Nhat
>