Ecosyste.ms: Timeline

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

openjdk/jdk

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

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
⚠️ @larry-cable This pull request contains merges that bring in commits not present in the target repository. Since this is not a "merge style" pull request, these changes will be squashed when th...

View on GitHub

magicus created a comment on a pull request on openjdk/jdk
> I'm not sure that error was caused by this issue. That would imply that I had strange file attributes on files in my build directory, and I don't think I do. I can't see what else did. I thin...

View on GitHub

valeriepeng created a comment on a pull request on openjdk/jdk
Is "https://tools.ietf.org/html/rfc8103" considered external spec? It is mentioned in com.sun.crypto.provider.ChaCha20Poly1305Parameters class but not covered in this PR. Is there any additional co...

View on GitHub

openjdk-notifier[bot] pushed 1 commit to pr/21574 openjdk/jdk
  • Improve specification per code review feedback. 96c05b5

View on GitHub

jddarcy created a review comment on a pull request on openjdk/jdk
Reasonable suggestion; I'll put the same kind of pointers into java.lang.Double sections as java.lang.Float has.

View on GitHub

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

View on GitHub

jddarcy created a review comment on a pull request on openjdk/jdk
Hmm. I'm trying to keep parity between Float16 and java.lang.{Float, Double}. Float/Double don't include the class-level statement so I wouldn't include on in Float16 until they do. Writing Float16...

View on GitHub

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

View on GitHub

pchilano created a review comment on a pull request on openjdk/jdk
It is `thread->vthread()->thread_id()` but it will match `thread->threadObj()->thread_id()` when there is no virtual thread mounted.

View on GitHub

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

View on GitHub

pchilano created a review comment on a pull request on openjdk/jdk
Done.

View on GitHub

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

View on GitHub

pchilano created a review comment on a pull request on openjdk/jdk
I can remove the check. I don't think it hurts either though. Also we can actually just use rscratch1 in the ASSERT case.

View on GitHub

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

View on GitHub

pchilano created a review comment on a pull request on openjdk/jdk
Added.

View on GitHub

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

View on GitHub

pchilano created a review comment on a pull request on openjdk/jdk
Added comment and fixed dst definition.

View on GitHub

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

View on GitHub

pchilano created a review comment on a pull request on openjdk/jdk
I see we are already using rscratch1 in these locking macros so I could change it to use that instead. But looking at all other macros in this file we are already using rscratch1 and rscratch2 too,...

View on GitHub

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

View on GitHub

pchilano created a review comment on a pull request on openjdk/jdk
> Also JavaThread::_lock_id in the VM means "the java.lang.Thread thread-id to use for locking" - correct? > Yes.

View on GitHub

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

View on GitHub

pchilano created a review comment on a pull request on openjdk/jdk
Sorry, I should add context on why this is needed. The problem is that inside this temporal transition we could try to acquire some monitor. If the monitor is not inflated we will try to use the Lo...

View on GitHub

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

View on GitHub

openjdk-notifier[bot] pushed 48 commits to pr/21574 openjdk/jdk
  • 8342579: RISC-V: C2: Cleanup effect of killing flag register for call instructs Reviewed-by: rehn, fjiang 76ae072
  • 8342287: C2 fails with "assert(is_IfTrue()) failed: invalid node class: IfFalse" due to Template Assertion Predicate ... d61f56a
  • 8342612: Increase memory usage of compiler/c2/TestScalarReplacementMaxLiveNodes.java Reviewed-by: kvn 21682bc
  • 8341407: C2: assert(main_limit == cl->limit() || get_ctrl(main_limit) == new_limit_ctrl) failed: wrong control for ad... 8f2b23b
  • 8342496: C2/Shenandoah: SEGV in compiled code when running jcstress Reviewed-by: shade, rkennke 680dc5d
  • 8342334: CDS: Scratch mirrors should not point to dead klasses Reviewed-by: adinn, coleenp, iklam aa060f2
  • 8342578: GHA: RISC-V: Bootstrap using Debian snapshot is still failing Reviewed-by: shade, erikj 239d84a
  • 8339570: Add Tidy build support for JDK tests Co-authored-by: Magnus Ihse Bursie <[email protected]> Reviewed-by: eri... 5d5d88a
  • 8341470: BigDecimal.stripTrailingZeros() optimization Reviewed-by: rgiulietti 27ef6c9
  • 8340818: Add a new jtreg test root to test the generated documentation Reviewed-by: erikj 07f550b
  • 8340241: RISC-V: Returns mispredicted Reviewed-by: fyang, luhenry 66ddaaa
  • 8342102: ZGC: Optimize copy constructors in ZPhysicalMemory Reviewed-by: stefank, kbarrett, aboldtch 1f35748
  • 8342295: compiler/jvmci/TestJVMCISavedProperties.java fails due to garbage in output Reviewed-by: dnsimon 330f2b5
  • 8328528: C2 should optimize long-typed parallel iv in an int counted loop Reviewed-by: roland, chagedorn, thartmann 80ec552
  • 8338126: C2 SuperWord: VectorCastF2HF / vcvtps2ph produces wrong results for vector length 2 Reviewed-by: thartmann,... 153ad91
  • 8342653: Fix minor doc issues in AnnotatedElement Reviewed-by: jpai, iris, liach 18b55ce
  • 8340553: ZipEntry field validation does not take into account the size of a CEN header Reviewed-by: jpai, redestad, ... 54a744b
  • 8341052: SHA-512 implementation using SHA-NI Reviewed-by: jbhateja, ascarpino, sviswanathan, sparasa 18bcbf7
  • 8342376: More reliable OOM handling in ExceptionDuringDumpAtObjectsInitPhase test Reviewed-by: iklam, phh 37aa320
  • 8339507: Test generation tool and gtest for testing APX encoding of extended gpr instructions Reviewed-by: jbhateja,... 52d752c
  • and 28 more ...

View on GitHub

openjdk-notifier[bot] pushed 1 commit to pr/21565 openjdk/jdk
  • Minor fixes in inc/dec_held_monitor_count on aarch64 and riscv baf7ffa

View on GitHub

openjdk-notifier[bot] pushed 1 commit to pr/21364 openjdk/jdk
  • Added several commends and made stylistic changes to make code more readable 0ecef8b

View on GitHub

honkar-jdk created a review comment on a pull request on openjdk/jdk
No changes required for this test, it works. The test was failing due to IDE config issue of tmp dir.

View on GitHub

honkar-jdk created a review on a pull request on openjdk/jdk

View on GitHub

Load more