RFR (S): JDK-8038924: Test bit-instructions fails with unexpected exit value on sparc
vladimir.kozlov at oracle.com
Wed May 21 18:17:57 UTC 2014
Not surprisingly :) I missed that it runs only with -version. It is hard
to see which options are used. Non of these tests list '-version' even
in a comment.
I agree that if you fork the same VM (server or client, 32-bit or
64-bit) and run it with -version flag you don't need to pass all CL options.
On 5/21/14 6:23 AM, Filipp Zhinkin wrote:
> sorry for a late response.
> On 05/13/2014 11:21 PM, Vladimir Kozlov wrote:
>> On 5/13/14 11:49 AM, Filipp Zhinkin wrote:
>>> Hi all,
>>> please review fix forJDK-8038924.
>> Filipp, please, provide the link to the bug report in you RFR.
> Sorry for that, I'l take it in mind for future RFRs.
>>> CLI tests on BMI-related options previously took all externally passed
>>> VM options,
>>> add options that should be tested and launch new VM with such
>>> combination of options.
>>> Unfortunately, some options like IgnoreUnrecognizedVMOptions, can affect
>>> processing of tested flags and in such case tests fail.
>>> I've updated these tests so now they do not use externally passed
>> I don't think it is right solution. Some external flags could be
>> important (DeoptimizeALot for example).
>> You can add -XX:-IgnoreUnrecognizedVMOptions to tests which you expect
>> to fail (on platforms which does not have the feature). It will
>> nullify this flag passed from CL.
> In general I agree with that point - tests should be able to use as much
> externally passed options as possible.
> However, in this particular case I believe that it would be better to
> ignore such options, because these tests only launch a new VM with tests
> options and '-version' flag,
> so we'll not be able to catch issues that we can't find with other BMI
> tests. And if CLI tests will not use externally passed options it will
> be easier to maintain it.
> Note that other BMI-related tests use all externally passed options, so
> we can catch any issues related to JVM behaviour using it.
> So I think that it will be better to avoid usage of externally passed
> options in CLI-tests and update compiler/intrinsics/bmi tests to run it
> on all platforms.
>>> Also, I've fix silly issue with String::format in
>>> Testing: manual and automated.
More information about the hotspot-compiler-dev