On Sat, 6 May 2023 04:08:42 GMT, Quan Anh Mai <[email protected]> wrote:
>> Stefan Karlsson has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 917 commits: >> >> - ZGC: Generational >> >> Co-authored-by: Stefan Karlsson <[email protected]> >> Co-authored-by: Per Liden <[email protected]> >> Co-authored-by: Albert Mingkun Yang <[email protected]> >> Co-authored-by: Erik Österlund <[email protected]> >> Co-authored-by: Axel Boldt-Christmas <[email protected]> >> Co-authored-by: Stefan Johansson <[email protected]> >> - UPSTREAM: Introduce check_oop infrastructure to check oops in the oop >> class >> - UPSTREAM: RISCV tmp reg cleanup resolve_jobject >> - CLEANUP: barrierSetNMethod_aarch64.cpp >> - UPSTREAM: Add relaxed add&fetch for aarch64 atomics >> - UPSTREAM: assembler_ppc CMPLI >> >> Co-authored-by: TheRealMDoerr <[email protected]> >> - UPSTREAM: assembler_ppc ANDI >> >> Co-authored-by: TheRealMDoerr <[email protected]> >> - UPSTREAM: Add VMErrorCallback infrastructure >> - Merge branch 'zgc_generational' into zgc_generational_rebase_target >> - Whitespace nit >> - ... and 907 more: https://git.openjdk.org/jdk/compare/705ad7d8...349cf9ae > > src/hotspot/cpu/x86/gc/z/zBarrierSetAssembler_x86.cpp line 310: > >> 308: // A not relocatable object could have spurious raw null pointers >> in its fields after >> 309: // getting promoted to the old generation. >> 310: __ cmpw(ref_addr, barrier_Relocation::unpatched); > > `cmpw` with immediates stalls the predecoder, it may be better to `movzwl` to > a spare register and `cmpl` there. I think we use the flag `UseStoreImmI16` for these kinds of situations. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13771#discussion_r1186662246
