RFR (S) 8131782: C1 Class.cast optimization breaks when Class is loaded from static final (original) (raw)

Aleksey Shipilev aleksey.shipilev at oracle.com
Wed Jul 29 09:00:25 UTC 2015


Ah. That would save me half a day digging in HotSpot. Thanks should go to John Rose who suggested the factory fix, not the Class.cast intrinsic one.

-Aleksey

On 07/28/2015 11:43 PM, Krystal Mok wrote:

Hi Aleksey,

Thanks for fixing this in OpenJDK! I actually noticed the same issue a few weeks ago [1], but somehow I had missed the reply that Roland sent me, so I didn't send out a request for review for my version of the change. But now that it's fixed, everythings all right ;-) Thanks, Kris [1]: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2015-June/018179.html On Mon, Jul 27, 2015 at 1:51 AM, Aleksey Shipilev <aleksey.shipilev at oracle.com <mailto:aleksey.shipilev at oracle.com>> wrote: Thanks for pushing the change, Dean! -Aleksey On 07/25/2015 05:32 AM, John Rose wrote: > You are fine. > Aleksey is an Author for JDK 9: http://openjdk.java.net/census#shade > Committers who sponsor changes are expected to use the correct Author > (not themselves). > It's a syntax error to push a non-Author changeset to an OpenJDK repo. > BTW, jcheck does not consult the OJN census AFAIK. > > — John > > On Jul 24, 2015, at 6:35 PM, Dean Long <dean.long at oracle.com_ _<mailto:dean.long at oracle.com> > <mailto:dean.long at oracle.com <mailto:dean.long at oracle.com>>> wrote: >> >> OK I will push it now. I did 'hg ci -u shade' so that Aleksey gets >> credit for it, and jcheck isn't complaining. Does anyone know if JPRT >> does more checks for Committer status? If so I'll have to redo it and >> add a Contributed-by line. >

-------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20150729/e4a606e0/signature.asc>



More information about the hotspot-compiler-dev mailing list