Request for reviews (XS): 7048030: is_scavengable changes causing compiler to embed more constants (original) (raw)

Tom Rodriguez tom.rodriguez at oracle.com
Tue May 24 18:17:23 PDT 2011

Looks good. Thanks for taking this.

tom

On May 24, 2011, at 5:19 PM, Vladimir Kozlov wrote:

http://cr.openjdk.java.net/~kvn/7048030/webrev

Fixed 7048030: isscavengable changes causing compiler to embed more constants This is the cause of 7047300 problem to appear. The fix for 7041789 corrected the meaning of isscavengable to really mean what it says. Unfortunately there were some places that were really using it as a proxy for !isperm. In particular in ciObject::canbeconstant and shouldbeconstant we will now embed constants that we didn't used as long as they are in tenured. The code should be changed from !isscavengable to isperm. Verified with failing test from 7047300.

More information about the hotspot-compiler-dev mailing list