Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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...
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...
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
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...
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.
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?
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 ...
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 ...
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...
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 ...
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.
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:/...
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...
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 ...
openjdk[bot] created a comment on a pull request on openjdk/jdk
❗ This change is not yet ready to be integrated. See the **Progress** checklist in the description for automated requirements. <!-- PullRequestBot merge is ready comment --> <!-- PullRequestBot pla...
bridgekeeper[bot] created a comment on a pull request on openjdk/jdk
<!-- Welcome message --> :wave: Welcome back amenkov! A progress list of the required criteria for merging this PR into `master` will be added to the body of your pull request. There are additiona...
alexmenkov opened a pull request on openjdk/jdk
8342577: Clean up JVMTI breakpoint support
The fix cleans up code to support list of JVMTI breakpoints. - classes required to supports cache of byte code pointers (GrowableElement, GrowableCache, JvmtiBreakpointCache) are dropped; - class...
