Request for review (S): C2 should recognize "obj.getClass() == A.class" code pattern

Krystal Mok rednaxelafx at gmail.com
Sun May 6 19:08:29 PDT 2012


Hi all,

Updated the patch again, posted at:
https://gist.github.com/2622617#file_class_eq_ver_5.patch
Could I get some reviews for this new version, please? And, would anyone
sponsor this patch, pleae?

Per John's suggestion, in Parse::adjust_map_after_if(), I've factored the
logic from "int val_in_map" downward into a new
subroutine sharpen_type_after_if(), which handles comparison with constant
klass as well as the old cases. I tried a few times but I just can't find a
clean factoring.

Tested SPECjvm2008 with default arguments on a jvmg build, and it ran okay.

Regards,
Kris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20120507/5c99aba9/attachment.html 


More information about the hotspot-compiler-dev mailing list