-
Notifications
You must be signed in to change notification settings - Fork 6.1k
8342382: Implement JEP 522: G1 GC: Improve Throughput by Reducing Synchronization #23739
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
Changes from 1 commit
0372026
7782295
9e26abb
3004a96
b8100b9
0100d8e
9ef9c5f
e51eec8
7d361fc
d87935a
810bf2d
b3dd008
8f46dc9
9e2ee54
442d9ea
fc674f0
b4d19d9
4a97811
a457e6e
350a4fa
c994000
93b884f
758fac0
aec9505
3766b76
7861117
51a9eed
b073017
447fe39
4d0afd5
ff9eb26
c833bc8
f419556
5e76a51
d931104
6d574da
51fb6e6
27b3dd6
1c5a669
c5d5f3a
9481821
855ec8d
d4649ed
63b1de8
39aa903
fcf96a2
fd77531
068d2a3
e683152
a3b2386
e4bf1ac
51dfbe5
8b56880
1def83a
c07a73d
750ed2d
441c234
5ab928e
4b21868
cea0e1b
dd83638
23aa2c8
188fc81
7fe518e
6c88f1d
e8a8282
cc4b7a0
4a41b40
b3873d6
104d506
2a614a2
4601bf8
87b4136
e7c3a06
de1469d
d0ca906
b47c7b0
c469c13
74e9240
1ced9f9
bf8cab3
b5d22d5
53ef008
6e37f8d
311bb3e
d80d690
3c889e9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1405,7 +1405,7 @@ public void opto2() { | |
|
|
||
| @Test | ||
| @IR(failOn = IRNode.ALLOC) | ||
| @IR(counts = {IRNode.COUNTED_LOOP, "1"}) // not fail | ||
| @IR(counts = {IRNode.COUNTED_LOOP, ">1"}) // not fail | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you explain what led to the difference? Can you also set an upper bound? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. With the decreased complexity of the barrier, C2 started unrolling that loop. I do not know how to determine a bound here. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this going to be GC independent? What if the VM swaps to another GC ergonomically? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The test runs with If we still want to test the improved loop unrolling opportunities, I suggest to create a separate IR test for it, possibly in a separate RFE. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I suppose that's probably not possible, as far as I know we always run with G1GC, so it should be fine :) |
||
| public void opto3() { | ||
| for (int i = 0; i < 100; i++) { | ||
| obj = new Object(); | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.