Ecosyste.ms: Timeline

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

openjdk/jdk

liach created a review comment on a pull request on openjdk/jdk
Another example of such a trick is available at [test/jdk/jdk/classfile/ClassBuildingTest.java](https://github.com/openjdk/jdk/blob/master/test/jdk/jdk/classfile/ClassBuildingTest.java)

View on GitHub

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

View on GitHub

liach created a review comment on a pull request on openjdk/jdk
To reduce test file noise, I recommend adding this as a package-private class to the end of your test file (so not a nested class) after it is renamed, like: ```java class EmptyLoop { publ...

View on GitHub

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

View on GitHub

liach created a review comment on a pull request on openjdk/jdk
How about `CodeWriterIndentTest`? We prefer to use prefix modifiers, and this tests the indent for general code writer output.

View on GitHub

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

View on GitHub

IvaVladimir created a comment on a pull request on openjdk/jdk
Without the 'volatile' the performance improved for ~6% (22293 -> 23638) for the 224 threads run. Without the 'static volatile' performance improved ~1.9x (22293 -> 42045). It may be 'transient' ...

View on GitHub

alexeysemenyukoracle created a comment on a pull request on openjdk/jdk
The test does nothing on OSX, right? We just run jpackage and don't verify the output. What is the point then?

View on GitHub

cetinkaya-xues starred openjdk/jdk
jatin-bhateja created a comment on a pull request on openjdk/jdk
Hi @eme64 , Let me know if there are other comments, else looking forward to you approval :-)

View on GitHub

jatin-bhateja created a review comment on a pull request on openjdk/jdk
Hi @eme64 , @PaulSandoz , in general bounds overrun problem is only pertinent to a very small portion of vector ISA which supports memory destination flavor e.g. [VCVTPS2PH](https://www.felixclouti...

View on GitHub

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

View on GitHub

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=21698&range=00) ([d964968c](https:/...

View on GitHub

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

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

bridgekeeper[bot] created a comment on a pull request on openjdk/jdk
<!-- Welcome message --> :wave: Welcome back almatvee! 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 addition...

View on GitHub

sashamatveev opened a pull request on openjdk/jdk
8342576: [macos] AppContentTest still fails after JDK-8341443 for same reason on older macOS versions
- It is not clear on which macOS versions codesign fails if application bundle contains additional content. - As a result test was modified to generate only application image, since PKG or DMG can...
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=21697&range=00) ([f94f3085](https:/...

View on GitHub

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

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

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...

View on GitHub

bridgekeeper[bot] created a comment on a pull request on openjdk/jdk
<!-- Welcome message --> :wave: Welcome back henryjen! 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 addition...

View on GitHub

slowhog opened a pull request on openjdk/jdk
8342930: New tests from JDK-8335912 are failing
Check output for including values rather than exact match. Also include the manpage change and revert the problem list.
openjdk-notifier[bot] deleted a branch openjdk/jdk

pr/21580

wenshao closed a pull request on openjdk/jdk
8342644: Optimize InvokerBytecodeGenerator for using ClassFile API
Cache commonly used classDesc to avoid creating ReferenceClassDescImpl every time <!-- Anything below this marker will be automatically updated, please do not edit manually! --> --------- ### Prog...
bplb created a comment on a pull request on openjdk/jdk
I'm seeing a build failure on macosx-x64 on both my own machine and in the CI: ``` Error occurred during initialization of boot layer java.lang.InternalError: java.base not found ``` The platf...

View on GitHub

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

Load more