[8u] 8023697: failed class resolution reports different class name in detail message for the first and subsequent times

[Andrew Hughes]

On 20/07/2020 16:31, Zhengyu Gu wrote:
> Hi,
> 
> I would like to backport this to 8u for parity with Oracle 8u271.
> 
> The original patch does not apply cleanly. The major conflicts were due
> to the renaming of "this_oop" to "this_cp" in constantPool.[hpp|cpp]
> 

Seems to be a typographical change with no change in semantics from that
alone:

https://bugs.openjdk.java.net/browse/JDK-8031820

Keeping 'this_oop' in the new functions for consistency seems the right
thing to do.

> Replacing "this_cp" with "this_oop" in original patch, only resulted one
> minor conflict on copyright line of constantTag.cpp
> 

constantTag.cpp already has 2014, so this is fine.

> 
> Original bug: https://bugs.openjdk.java.net/browse/JDK-8023697
> Original patch:
> http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/003c36658224
> 
> 8u webrev: http://cr.openjdk.java.net/~zgu/JDK-8023697-8u/webrev.00/
> 
> Test:
>   hotspot_tier1
> 
> Thanks,
> 
> -Zhengyu
> 

Looks ok. Please flag for approval.

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222