Hi Claes!
Not a thorough review, but a minor comment.
209 .replaceAll("DMH.invoke", "DMH/xxxxxxxx.invoke")
The dot in the regexp needs to be escaped here.
With kind regards,
Ivan
On 09.08.2016 15:26, Claes Redestad wrote:
<resending due to missing RFR in subject>
On 2016-08-09 13:48, Claes Redestad wrote:
Hi,
change to enable DMH pre-generation caused a subtle change in the
stacktrace, which caused this test to fail.
Please review this patch which reverts the stackframe to look like
before for non-pregenerated DMHs, while also updating the test to
accept pregenerated DMHs.
To verify this works I added invokeVirtual_LL_V (used in the test) to
the list of pre-generated DMHs in GenerateJLIClassesPlugin and
verified that the test pass both on a jlinked image and an exploded
image.
Webrev: http://cr.openjdk.java.net/~redestad/8163476/webrev.01
Bug: https://bugs.openjdk.java.net/browse/JDK-8163476
Thanks!
/Claes
