New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test: Randomly enable AggressiveOpts in builds #7533
Conversation
@mrsolo can you take a look? |
@@ -50,7 +50,8 @@ | |||
'tests.jvm.argline' => [ | |||
{:choices => ['-server'], :method => 'get_random_one'}, | |||
{:choices => ['-XX:+UseConcMarkSweepGC', '-XX:+UseParallelGC', '-XX:+UseSerialGC', '-XX:+UseG1GC'], :method => 'get_random_one'}, | |||
{:choices => ['-XX:+UseCompressedOops', '-XX:-UseCompressedOops'], :method => 'get_random_one'} | |||
{:choices => ['-XX:+UseCompressedOops', '-XX:-UseCompressedOops'], :method => 'get_random_one'}, | |||
{:choices => ['-XX:+AggressiveOpts'], :method => 'get_random_one_or_none'} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New method is not needed.
Use this instead
{:choices => ['-XX:+AggressiveOpts'], :method => 'get_50_percent'}
The change exposes the bug in line 293 s[k] = v.join(' ') #this should be dependent on class of v[0] and perform reduce operation instead... good enough for now. tests.javm.argline will be quoted if AgressiveOpts isn't picked tests.jvm.argline: -server -XX:+UseSerialGC -XX:+UseCompressedOops -XX:+AggressiveOpts tests.jvm.argline: '-server -XX:+UseG1GC -XX:-UseCompressedOops ' changing the line to s[k] = v.compact.join(' ') will address the issue |
@mrsolo thanks for the review I pushed a new commit |
LGTM.. pushing code change through CI |
2cc3052
to
8938726
Compare
8938726
to
64dce61
Compare
to catch JVM bugs earlier I'd love to run builds randomly with
-XX:+AggressiveOpt