Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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 on a pull request on openjdk/jdk
Thanks! This looks better.
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).
TheRealMDoerr created a review on a pull request on openjdk/jdk
Thanks this looks better.
TheRealMDoerr created a review on a pull request on openjdk/jdk
Thanks this looks better.
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 ...