[9] RFR(S) 8175097: [TESTBUG] 8174164 fix missed the test

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[9] RFR(S) 8175097: [TESTBUG] 8174164 fix missed the test

Vladimir Kozlov
Original 8174164 changes had new test which was missing in the final push:

http://cr.openjdk.java.net/~roland/8174164/webrev.00/

test/compiler/c2/TestReplacedNodesOSR.java

I had to fix the test since it had trailing spaces and forgot to 'hg add' it.

Thanks,
Vladmir
Reply | Threaded
Open this post in threaded view
|

Re: [9] RFR(S) 8175097: [TESTBUG] 8174164 fix missed the test

Vladimir Kozlov
I will list Roland as Author and me as Reviewer since it was already reviewed for 8174164.

Vladimir

On 2/16/17 12:01 PM, Vladimir Kozlov wrote:

> Original 8174164 changes had new test which was missing in the final push:
>
> http://cr.openjdk.java.net/~roland/8174164/webrev.00/
>
> test/compiler/c2/TestReplacedNodesOSR.java
>
> I had to fix the test since it had trailing spaces and forgot to 'hg add' it.
>
> Thanks,
> Vladmir
Reply | Threaded
Open this post in threaded view
|

Re: [9] RFR(S) 8175097: [TESTBUG] 8174164 fix missed the test

Roland Westrelin-3

Thanks for pushing the test case.

Roland.