Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
chhagedorn created a review comment on a pull request on openjdk/jdk
Now needs to be called from `AssertionPredicatesForLoop`.
chhagedorn created a review comment on a pull request on openjdk/jdk
We can do something similar for the main and post loop which will be proposed in the next PR.
chhagedorn created a review comment on a pull request on openjdk/jdk
Noticed that after JDK-8342043, `count_opaque_loop_nodes()` can also be made debug-build only.
chhagedorn created a review comment on a pull request on openjdk/jdk
We can infer many values from other variables.
offamitkumar created a review comment on a pull request on openjdk/jdk
Oh, we don't even need `tmp` and it defaults to `noreg`. Even better.
offamitkumar created a review comment on a pull request on openjdk/jdk
why these store instruction left behind ? Can't we just pass address and give it scratch register to play with ?
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 chagedorn! 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 additio...
TheRealMDoerr created a review comment on a pull request on openjdk/jdk
The load instructions need the same adaptation as the store instructions (just without the tmp Register). ``` inline void Assembler::lwz( Register d, Address &a) { lwz(d, a.index() != noreg ?...
TheRealMDoerr created a review comment on a pull request on openjdk/jdk
The line is very long. Better break it to: ``` inline void Assembler::stw( Register d, Address &a, Register tmp) { stw(d, a.index() != noreg ? RegisterOrConstant(a.index()) : RegisterOrConsta...
TheRealMDoerr created a review comment on a pull request on openjdk/jdk
Spaces before "Register d" are uncommon. Please remove them (you can keep the one for ld to align it with the other ones).
chhagedorn opened a draft pull request on openjdk/jdk
8341977: Replace predicate walking and cloning code for Loop Peeling with a predicate visitor
#### Replacing the Remaining Predicate Walking and Cloning Code In the next series of patches (this, [JDK-8342943](https://bugs.openjdk.org/browse/JDK-8342943), [JDK-8342945](https://bugs.openjdk...
openjdk[bot] commented on a commit on openjdk/jdk
<!-- Jmerge command reply message (148310377) --> @GoeLin the [backport](https://github.com/openjdk-bots/jdk17u-dev/commit/8a73654722daea5e5781018a0d35e232be137b39) was successfully created on the ...
shipilev created a review comment on a pull request on openjdk/jdk
Yes. I think we go for a more comprehensive fix: https://github.com/openjdk/jdk/pull/20328#issuecomment-2434920739
openjdk-notifier[bot] pushed 348 commits to pr/20328 openjdk/jdk
- 8340988: Update jdk/jfr/event/gc/collection tests to accept "CodeCache GC Threshold" as valid GC reason Reviewed-by:... 9a7817b
- 8340458: Open source additional Component tests (part 2) Reviewed-by: psadhukhan 021bf63
- 8341060: Cleanup statics in HeapDumper Reviewed-by: shade, sspitsyn 0314973
- 8341367: Problemlist ShapeNotSetSometimes.java on macOS Reviewed-by: aivanov d2e7708
- 8340079: Modify rearrange/selectFrom Vector API methods to perform wrapIndexes instead of checkIndexes Reviewed-by: ... 83dcb02
- 8320308: C2 compilation crashes in LibraryCallKit::inline_unsafe_access Reviewed-by: thartmann, kvn, vlivanov, epete... 8d6d37f
- 8341243: Use ArraySupport.SOFT_MAX_ARRAY_LENGTH for max array size in java.base Reviewed-by: jpai, smarks 0f38113
- 8341277: Validate slot argument for instruction factories Reviewed-by: asotona 39c17b3
- 8340824: C2: Memory for TypeInterfaces not reclaimed by hashcons() Reviewed-by: vlivanov, qamai 90c944f
- 8340732: RISC-V: Refactor crc32 scalar version Reviewed-by: fyang 52c2ea6
- 8340109: Ubsan: ciEnv.cpp:1660:65: runtime error: member call on null pointer of type 'struct CompileTask' Reviewed-... efe3573
- 8341135: Incorrect format string after JDK-8339475 Reviewed-by: shade, clanger, alanb c8c4ff2
- 8341146: RISC-V: Unnecessary fences used for load-acquire in template interpreter Reviewed-by: fyang, rehn a4ca626
- 8336274: MutableBigInteger.leftShift(int) optimization Reviewed-by: rgiulietti d6820d1
- 8334305: Remove all code for nsk.share.Log verbose mode Reviewed-by: mli, cjplummer, lmesnik 855c8a7
- 8307532: Implement LM_LIGHTWEIGHT for Zero Reviewed-by: aboldtch, jwaters 5e98007
- 8324259: Classes used by CDS at runtime should be archived Reviewed-by: iklam, ccheung 8838048
- 8341412: Various test failures after JDK-8334305 Reviewed-by: lmesnik, dcubed 49501fe
- 8317116: Provide layouts for multiple test UI in PassFailJFrame Reviewed-by: azvegint, prr 85f0442
- 8340785: Update description of PassFailJFrame and samples Reviewed-by: prr 5063494
- and 328 more ...
shipilev created a comment on a pull request on openjdk/jdk
I redid the PR to summarily disable Leak Profiler with Shenandoah. This patch makes sure enabling JFR with Shenandoah does not break the VM. Moving forward, we would probably rewrite Leak Profiler ...
TheRealMDoerr created a comment on a pull request on openjdk/jdk
Thanks for figuring this out! So, there's a bit more to repair :-) I'll take a look.
varada1110 created a comment on a pull request on openjdk/jdk
Tier1 testing on linux-ppc64le has completed successfully. I am now running tests for aix-ppc
magicus created a comment on a pull request on openjdk/jdk
Just to clarify, the problem: ``` chmod: /Users/erik/dev/jdk/build/macosx-aarch64/jdk/lib/server/libjsig.dylib: No such file or directory ``` mentioned in https://github.com/openjdk/jdk/pull/20...
