Request for reviews (XS): 7047069: Array can dynamically change size when assigned to an object field

John Rose john.r.rose at oracle.com
Fri May 27 12:41:53 PDT 2011


On May 27, 2011, at 12:34 PM, Vladimir Kozlov wrote:

> I agree, I changed the code to check result of find_int_con(offset, BIG_NEG). Webrev is updated.

Better.  As the inventor of find_int_con, I reluctantly agree with Tom that it encourages too-clever code.  Uses of find_int_con should directly check the result for the sentinel value, rather than using it cleverly in further computation.

-- John


More information about the hotspot-compiler-dev mailing list