Ecosyste.ms: Timeline

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

openjdk/jdk

jatin-bhateja created a review on a pull request on openjdk/jdk

View on GitHub

bridgekeeper[bot] created a comment on a pull request on openjdk/jdk
<!-- PullrequestCloserBot auto close notification --> @dukbong This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any acti...

View on GitHub

openjdk-notifier[bot] pushed 2 commits to pr/21642 openjdk/jdk
  • disable test that fails with hotspot_runtime_non_cds_mode c3b2368
  • 8342907: Implement AOT testing mode for jtreg tests (authored by @katyapav) dd59b5f

View on GitHub

valeriepeng created a review comment on a pull request on openjdk/jdk
How about RFC 2560 (line 39-40)?

View on GitHub

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

View on GitHub

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

View on GitHub

sviswa7 created a comment on a pull request on openjdk/jdk
> What if you split into separate test files (.cpp) as well so that rather than one gtest that checks everything you have several smaller ones. That will reduce compilation overhead as required and...

View on GitHub

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

View on GitHub

openjdk[bot] closed a pull request on openjdk/jdk
8338544: Dedicated Array class descriptor implementation
@cl4es discovered that Stack Map generation in ClassFile API uses `componentType` and `arrayType` for `aaload` `aastore` instructions, which are currently quite slow. We can split out array class d...
openjdk[bot] pushed 1 commit to master openjdk/jdk
  • 8338544: Dedicated Array class descriptor implementation Reviewed-by: redestad, mchung, jvernee 25c2f48

View on GitHub

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

View on GitHub

liach created a comment on a pull request on openjdk/jdk
Thanks for the reviews! /integrate

View on GitHub

valeriepeng created a review comment on a pull request on openjdk/jdk
Same question applies for the java/security/interfaces/package-info.java file as well.

View on GitHub

valeriepeng created a review comment on a pull request on openjdk/jdk
How about the other 2 under `Package Specification` (from line 49-53)? Does this PR need to cover them also?

View on GitHub

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

View on GitHub

jaskarth created a comment on a pull request on openjdk/jdk
Thanks for looking at the build changes, @magicus! I've pushed a commit that removes the extra newline in the makefiles and adds newlines to the ends of files that were missing them. Thanks for ...

View on GitHub

jaskarth created a review comment on a pull request on openjdk/jdk
I think we shouldn't run `Ideal` on the graph, because there is a chance that it could undo the lowering changes that we just did. This gives lowering more freedom to change the graph in different ...

View on GitHub

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

View on GitHub

jaskarth created a review comment on a pull request on openjdk/jdk
Ah, do you mean having a method in `Node` that holds the lowering code? I was originally planning on doing it this way, but I think it would pose some problems where certain nodes' `Lower()` method...

View on GitHub

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

View on GitHub

jatin-bhateja created a review comment on a pull request on openjdk/jdk
> Because lowering is a transformation that increases the complexity of the graph. > > * A `d = ExtractD(z, 4)` expanded into `x = VectorExtract(z, 2); d = ExtractD(x, 0)` increases the number ...

View on GitHub

jatin-bhateja created a review on a pull request on openjdk/jdk

View on GitHub

dholmes-ora created a review comment on a pull request on openjdk/jdk
See the preamble in utf8.hpp. The unicode length of a utf8 string is assumed/required to fit in an int, but the raw utf8 length need not. Changing to use size_t may have a significant fanout however.

View on GitHub

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

View on GitHub

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

pr/21675 - 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=21675&range=00) ([94f596e1](https:/...

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2434049466) --> @alexmenkov <!-- solves: '8210637' 'cmFjZSBpbiBKdm10aUN1cnJlbnRCcmVha3BvaW50czo6Z2V0X2p2bXRpX2JyZWFrcG9pbnRz' --> Adding additional issue to issue...

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
@alexmenkov The following labels will be automatically applied to this pull request: - `hotspot` - `serviceability` When this pull request is ready to be reviewed, an "RFR" email will be sent to ...

View on GitHub

alexmenkov created a comment on a pull request on openjdk/jdk
/issue add 8210637

View on GitHub

dholmes-ora created a review on a pull request on openjdk/jdk
Thanks for updates!

View on GitHub

Load more