Request for reviews (M): 6689060: Escape Analysis does not work with Compressed Oops

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Thu Apr 17 14:08:54 PDT 2008


It is initial review for the bug fix. I am still testing.

http://webrev.invokedynamic.info/kvn/6689060/index.html

Fixed 6689060: Escape Analysis does not work with Compressed Oops

Problem:
64-bits VM crashes with -XX:+AggresiveOpts (EA + CO).

Solution:
Added missing lines in escape.cpp.

Scalar Replacement: Cast oop fields type to narrow before searching
values since values has narrow type. Used DecodeN to get wide oops for
debug info (I will optimize it later) and undo cast to narrow.
Need not delayed transformation to be able scalar replace objects.

Added not_null encodeP decodeN instructions into .ad files.
Fixed verify_oop stabs included debug messages to know where
a problem happened.

VerifyOops still failing with CO (even without EA and this changes).

Reviewed by:
Fix verified (y/n): y

Other testing:
JPRT with +EA, benchmarks, CTW, nsk

Thanks,
Vladimir





More information about the hotspot-compiler-dev mailing list