Looks good (not a Reviewer).
Leonid
On 4/5/20 11:49 PM, Chris Plummer wrote:
[Sorry about the resend. Subject wasn't quite right the first time.]
Hello,
Please help review the following:
https://bugs.openjdk.java.net/browse/JDK-8242168
http://cr.openjdk.java.net/~cjplummer/8242168/webrev.00/
ClhsdbFindPC needs to be disabled when using -XX:+DeoptimizeALot
because it means the expected compiled frame may not be compiled. I
also fixed ClhsdbJstackXcompStress since it also relies on frames
being compiled, although I never got it to fail since it managed to
always find at least one compiled frame.
I confirmed that with this fix these two tests are not run when
-XX:+DeoptimizeALot is used. If not used, behavior is unchanged.
ClhsdbFindPC.java#id0 runs whether or not using -Xcomp and #id1 still
only runs when not using -Xcomp. ClhsdbJstackXcompStress gets run
whether or not -Xcomp is used.
thanks,
Chris