Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
dholmes-ora created a review comment on a pull request on openjdk/jdk
What code does the unpark to wake this thread up? I can't quite see how this unparker thread operates as its logic seems dispersed.
dholmes-ora created a review comment on a pull request on openjdk/jdk
s/temporary change/temporarily change/
iklam created a comment on a pull request on openjdk/jdk
> @iklam Would it be possible to comment on this? If the system properties to configure the range of integer cache are set at runtime, does it just not use this archived object graph? I'm wondering...
openjdk-notifier[bot] commented on a commit on openjdk/jdk
<!-- COMMIT COMMENT NOTIFICATION --> ### Review - [openjdk/jdk/21541](https://git.openjdk.org/jdk/pull/21541) ### Issues - [JDK-8342387](https://bugs.openjdk.org/browse/JDK-8342387)
openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2434352287) --> @eme64 Pushed as commit e96b4cf0a81914c6a615bb4f62ea3f139a4737f3. :bulb: You may see a message that your pull request was closed with unmerged co...
openjdk[bot] closed a pull request on openjdk/jdk
8342387: C2 SuperWord: refactor and improve compiler/loopopts/superword/TestDependencyOffsets.java
I want to refactor `TestDependencyOffsets.java` using the `CompileFramework`. Reasons: - I soon need to modify the IR rules in this test soon anyway (https://github.com/openjdk/jdk/pull/21521),...openjdk[bot] pushed 1 commit to master openjdk/jdk
- 8342387: C2 SuperWord: refactor and improve compiler/loopopts/superword/TestDependencyOffsets.java Reviewed-by: thar... e96b4cf
openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- prepush e96b4cf0a81914c6a615bb4f62ea3f139a4737f3 --> Going to push as commit e96b4cf0a81914c6a615bb4f62ea3f139a4737f3. Since your change was applied there have been 15 commits pushed to the `m...
eme64 created a comment on a pull request on openjdk/jdk
@chhagedorn @TobiHartmann thanks for the reviews :blush: /integrate
rose00 created a comment on a pull request on openjdk/jdk
I took a closer look at the diffs in a file you renamed, from `classPrelinker.cpp` to `aotConstantPoolResolver.cpp`. Changes still look good, notably the filtering logic on the bootstrap methods (...
forax created a review comment on a pull request on openjdk/jdk
Ah, too slow to answer ... Anyway, the wrapper types tend to be used everywhere so there are also used in methods that are executed only a few times so only by the interpreter, in those case, acce...
hboehm created a comment on a pull request on openjdk/jdk
Let me try to belatedly correct the record here, in case I didn't overlook a later discussion somewhere else: > > Yes, that's right. Code in critical sections isn't racy, by definition. > > I...
openjdk[bot] created a comment on a pull request on openjdk/jdk
@varada1110 this pull request can not be integrated into `master` due to one or more merge conflicts. To resolve these merge conflicts and update this pull request you can run the following command...
dukbong closed a pull request on openjdk/jdk
Unnecessary Variable Removal for Optimized computeIfAbsent Implementation
Changes - The computeIfAbsent method has been optimized by removing the newValue variable and reusing the v variable, thereby eliminating unnecessary local variable declarations. Reason - In t...lmesnik created a review on a pull request on openjdk/jdk
Not actually review, just confirming that my request fulfilled. And no more issues arised during PIT
lmesnik created a review comment on a pull request on openjdk/jdk
no objection anymore
openjdk-notifier[bot] commented on a commit on openjdk/jdk
<!-- COMMIT COMMENT NOTIFICATION --> ### Review - [openjdk/jdk/21676](https://git.openjdk.org/jdk/pull/21676) ### Issues - [JDK-8342931](https://bugs.openjdk.org/browse/JDK-8342931)
AlanBateman created a comment on a pull request on openjdk/jdk
@iklam Would it be possible to comment on this? If the system properties to configure the range of integer cache are set at runtime, does it just not use this archived object graph? I'm wondering ...
openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2434306209) --> @dholmes-ora Pushed as commit f7a61fce949ea2d15ec09d295c643d0c2eabea1e. :bulb: You may see a message that your pull request was closed with unmer...
openjdk[bot] closed a pull request on openjdk/jdk
8342931: ProblemList failing tests from JDK-8335912
Trivial ProblemListing Thanks <!-- Anything below this marker will be automatically updated, please do not edit manually! --> --------- ### Progress - [x] Change must be properly reviewed (1 re...openjdk[bot] pushed 1 commit to master openjdk/jdk
- 8342931: ProblemList failing tests from JDK-8335912 Reviewed-by: henryjen, mikael f7a61fc