RFR: 8328316: Finisher cannot emit if stream is sequential and integrator returned false [v3]
Viktor Klang
vklang at openjdk.org
Wed Mar 20 23:31:31 UTC 2024
> Adds differentiation between direct and transitive short circuiting which could prevent pushing downstream in the finisher for built-ins that were not `collect()`.
>
> Creating this as a draft PR for now, just need to run some benchmarks to validate no significant regressions first.
Viktor Klang has updated the pull request incrementally with 236 additional commits since the last revision:
- Adding a comma
- Updating copyright year on GathererOp
- 8289822: G1: Make concurrent mark code owner of TAMSes
Reviewed-by: ayang, iwalulya
- 8325362: Allow to create a simple in-memory input JavaFileObject
Reviewed-by: jlaskey, darcy
- 8326941: Remove StringUTF16::isBigEndian
Reviewed-by: rriggs
- 8325187: JVMTI GetThreadState says virtual thread is JVMTI_THREAD_STATE_INTERRUPTED when it no longer is
Reviewed-by: dholmes, alanb
- 8308660: C2 compilation hits 'node must be dead' assert
Reviewed-by: chagedorn, kvn
- 8323972: C2 compilation fails with assert(!x->as_Loop()->is_loop_nest_inner_loop()) failed: loop was transformed
Reviewed-by: chagedorn, epeter
- 8328275: CodeCache::print_internals should not be called in PRODUCT code
Reviewed-by: ihse, jwaters, dholmes
- 8326964: Remove Eclipse Shared Workspaces
Reviewed-by: erikj, ihse
- ... and 226 more: https://git.openjdk.org/jdk/compare/f32b1c5f...189ffaa3
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/18351/files
- new: https://git.openjdk.org/jdk/pull/18351/files/f32b1c5f..189ffaa3
Webrevs:
- full: Webrev is not available because diff is too large
- incr: Webrev is not available because diff is too large
Stats: 384826 lines in 1503 files changed: 18739 ins; 81305 del; 284782 mod
Patch: https://git.openjdk.org/jdk/pull/18351.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/18351/head:pull/18351
PR: https://git.openjdk.org/jdk/pull/18351
More information about the core-libs-dev
mailing list