Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

openjdk/jdk

dean-long created a comment on a pull request on openjdk/jdk
BTW, another place where a user-defined class like NoOverflowInt might be useful is compiler invocation counters. We store them as integers in various places, increment and scale them, check for s...

View on GitHub

IvaVladimir created a comment on a pull request on openjdk/jdk
The specjvm::crypto.aes workload not affected by this issue. It implemented as mix of DES and AES encryptions but AES much better optimized vs DES. The top 3 methods for this workload for the 112T...

View on GitHub

dholmes-ora created a review on a pull request on openjdk/jdk
Thanks for updates. (I need to add a Review comment so I get a checkpoint to track further updates.)

View on GitHub

dholmes-ora created a review on a pull request on openjdk/jdk
LGTM!

View on GitHub

iwanowww created a comment on a pull request on openjdk/jdk
Also, I briefly looked at #21599 in the context of this particular enhancement, but still don't see how it can improve the situation (except input rewiring part) and not simply duplicate what match...

View on GitHub

bplb created a comment on a pull request on openjdk/jdk
There is no `noreg-*` label on the issue. Is this change covered by an existing test?

View on GitHub

iwanowww created a comment on a pull request on openjdk/jdk
Thanks, @jatin-bhateja. I took a look at the latest version and still think that IGVN is not the best place for it. First of all, flags on MulVL feel too adhoc and irregular. The original IR st...

View on GitHub

jianglizhou created a comment on a pull request on openjdk/jdk
> > > When trying to sort out the LDFLAGS issues, it turned out that I could not run the linux launcher at all, not even when checking out older commits of this PR. I am almost at a loss here; I as...

View on GitHub

openjdk-notifier[bot] pushed 1 commit to pr/21507 openjdk/jdk

View on GitHub

iklam created a review on a pull request on openjdk/jdk

View on GitHub

dean-long created a review comment on a pull request on openjdk/jdk
This is also a performance optimization so the comment should probably say so, so we remember not to change it back to a loop later on. It might be nice to have NEW_RESOURCE_ARRAY do the initializ...

View on GitHub

dean-long created a review on a pull request on openjdk/jdk

View on GitHub

openjdk-notifier[bot] deleted a branch openjdk/jdk

pr/20528

openjdk-notifier[bot] commented on a commit on openjdk/jdk
<!-- COMMIT COMMENT NOTIFICATION --> ### Review - [openjdk/jdk/20528](https://git.openjdk.org/jdk/pull/20528) ### Issues - [JDK-8336665](https://bugs.openjdk.org/browse/JDK-8336665)

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
@dmlloyd The following label will be automatically applied to this pull request: - `nio` When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing l...

View on GitHub

jianglizhou created a comment on a pull request on openjdk/jdk
> > When trying to sort out the LDFLAGS issues, it turned out that I could not run the linux launcher at all, not even when checking out older commits of this PR. I am almost at a loss here; I assu...

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2436409784) --> @mcpowers Pushed as commit ca1700b8bf3b7b34616c469837748ac4c3607dfc. :bulb: You may see a message that your pull request was closed with unmerged...

View on GitHub

openjdk[bot] closed a pull request on openjdk/jdk
8336665: CCE in X509CRLImpl$TBSCertList.getCertIssuer
https://bugs.openjdk.org/browse/JDK-8336665 <!-- Anything below this marker will be automatically updated, please do not edit manually! --> --------- ### Progress - [x] Change must be properly rev...
openjdk[bot] pushed 1 commit to master openjdk/jdk
  • 8336665: CCE in X509CRLImpl$TBSCertList.getCertIssuer Reviewed-by: mullan ca1700b

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- prepush ca1700b8bf3b7b34616c469837748ac4c3607dfc --> Going to push as commit ca1700b8bf3b7b34616c469837748ac4c3607dfc. Since your change was applied there have been 1530 commits pushed to the ...

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2436409504) --> @cl4es <!-- additional required reviewers id marker (2) (reviewers) --> The total number of required reviews for this PR (including the jcheck con...

View on GitHub

dean-long created a comment on a pull request on openjdk/jdk
There's not much point in generating an incomplete replay file. How about moving the checks for this->task() to the top of the function? And we could avoid the call altogether by changing VMError...

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2436431780) --> @magicus <!-- removed label: 'build' --> The `build` label was successfully removed.

View on GitHub

luhenry created a review on a pull request on openjdk/jdk

View on GitHub

openjdk-notifier[bot] created a branch on openjdk/jdk

pr/21696 - JDK main-line development https://openjdk.org/projects/jdk

openjdk-notifier[bot] created a branch on openjdk/jdk

pr/21692 - JDK main-line development https://openjdk.org/projects/jdk

mlbridge[bot] created a comment on a pull request on openjdk/jdk
<!-- mlbridge webrev comment --> <!-- mlbridge webrev header --> ### Webrevs <!-- mlbridge webrev list --> * 00: [Full](https://webrevs.openjdk.org/?repo=jdk&pr=21692&range=00) ([21f65c3f](https:/...

View on GitHub

alexmenkov created a review on a pull request on openjdk/jdk

View on GitHub

openjdk-notifier[bot] pushed 1 commit to pr/21695 openjdk/jdk

View on GitHub

valeriepeng created a comment on a pull request on openjdk/jdk
Note the bot warning on the branch name being master. Will you change this following the bot's instruction? Also the minor issue with the convention of starting the synopsis with Upper case, i.e. "...

View on GitHub

Load more