8038098: test/compiler/codegen/LoadWithMask2.java tags changes to also test C1 inlining

[Vladimir Kozlov]

Hi Evgeniya

What about LoadWithMask.java test?

Please, simple remove '-server' option. We always do testing with server JVM so we can't miss the bug if it returns. We 
did go through such cleanup in tests before and did not have any problems.

Flag -XX:-TieredCompilation may affect bug reproduction. That is why we specify it sometimes.
In general we can remove it too since we run Nightly in both configurations with and without Tiered. But since it does 
not affect what JVM is used we keep it.

In compiler tests we should not specify neither -server or -client to make sure the correct JVM is tested. This is one 
of our rules when we add tests. Recently we added tests which have -server specified:

compiler/IntegerArithmetic/TestIntegerComparison.java
compiler/testlibrary/rtm/RTMTestBase.java
compiler/uncommontrap/UncommonTrapStackBang.java

compiler/stable/* test have  @run command with both -server and -client configurations.

All these tests should be cleaned up.

Regards,
Vladimir

On 9/15/14 7:11 AM, Evgeniya Stepanova wrote:
> Hi Filipp,
>
> I want to keep first string "as is" because it is a regression test and if we just remove -server option, it is possible
> that bug have returned, but we won't find it before release because options will not compose in the right way.



> The test is very fast, so the second run will not take significant time, but we will be sure that there is no regression.
>
> As for C2-specific options, I agree. I've removed -client and TieredCompilation options to use jtreg submitted.
>
> Please see updated diff:
> http://cr.openjdk.java.net/~iignatyev/eistepan/8038098/webrev.00/
>
> Thanks,
> Jane
> On 15.09.2014 16:01, Filipp Zhinkin wrote:
>> Hi Jane,
>>
>> mail's summary does not match the bug's name.
>>
>> As about your fix: I thought that such configs as client/server
>> and tired/non-tired should be rotated in nightly, aren't it?
>>
>> Maybe it does more sense to just remove '-server' option?
>> With '-client' the test may fail during test runs that were started
>> with some C2-specific options, like UseRTMLocking.
>>
>> Thanks,
>> Filipp.
>>
>> On 09/15/2014 03:42 PM, Evgeniya Stepanova wrote:
>>> Hi all!
>>>
>>> Please review fix for 8038098
>>>
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8038098
>>>
>>> Problem: Since we have inlining in C1 now, we should check that issue 8031743 will not appear in C1.
>>> Solution: Added one more run in test for C1 inlining checking
>>>
>>> A am also need someone to push changes.
>>>
>>> Diff is in the attachments.
>>> I've also included diff to this letter since it is very small.
>>>
>>>
>>> Thanks,
>>> Jane
>>>
>>> diff -r 087443edb74a test/compiler/codegen/LoadWithMask2.java
>>> --- a/test/compiler/codegen/LoadWithMask2.java    Mon Sep 15 08:08:22 2014 +0200
>>> +++ b/test/compiler/codegen/LoadWithMask2.java    Mon Sep 15 14:35:50 2014 +0400
>>> @@ -26,6 +26,7 @@
>>>   * @bug 8031743
>>>   * @summary loadI2L_immI broken for negative memory values
>>>   * @run main/othervm -server -Xbatch -XX:-TieredCompilation -XX:CompileCommand=compileonly,*.foo* LoadWithMask2
>>> + * @run main/othervm -client -Xbatch -XX:+TieredCompilation -XX:CompileCommand=compileonly,*.foo* LoadWithMask2
>>>   *
>>>   */
>>>  public class LoadWithMask2 {
>>
>
> --
> /Evgeniya Stepanova/