Reviewed.
Best regards,
Vladimir Ivanov
On 11/17/15 5:21 PM, Claes Redestad wrote:
Hi,
it seems I've been testing assertions with the wrong flag.
8142334 added the same assertion check to MethodHandleImpl that was
there for DirectMethodHandle, which
due to some bootstrapping dependency fails
Hi,
it seems I've been testing assertions with the wrong flag.
8142334 added the same assertion check to MethodHandleImpl that was
there for DirectMethodHandle, which
due to some bootstrapping dependency fails for the MethodHandleImpl
case. This patch removes that assertion:
webrev: http://c
Vladimir, Paul,
thanks for looking at this so quickly!
Pushed.
/Claes
On 2015-11-17 15:30, Paul Sandoz wrote:
On 17 Nov 2015, at 15:21, Claes Redestad wrote:
Hi,
it seems I've been testing assertions with the wrong flag.
To share some knowledge: when running jtreg locally it is important
> On 17 Nov 2015, at 15:21, Claes Redestad wrote:
>
> Hi,
>
> it seems I've been testing assertions with the wrong flag.
>
To share some knowledge: when running jtreg locally it is important to use the
same parameters that are used when running the automated tests. To the best of
my knowled