On Mon, 9 Mar 2026 15:42:38 GMT, Kangcheng Xu <[email protected]> wrote:
>> This PR refactors `PhaseIdealLoop::is_counted_loop()` into (mostly) >> `CountedLoopConverter::is_counted_loop()` and >> `CountedLoopConverter::convert()` to decouple the detection and conversion >> code. This enables us to try different loop configurations easily and >> finally convert once a counted loop is found. >> >> A nested `PhaseIdealLoop::CountedLoopConverter` class is created to handle >> the context, but I'm not if this is the best name or place for it. Please >> let me know what you think. >> >> Blocks [JDK-8336759](https://bugs.openjdk.org/browse/JDK-8336759). > > Kangcheng Xu has updated the pull request incrementally with one additional > commit since the last revision: > > Merge remote-tracking branch 'origin/master' into counted-loop-refactor (#4) > > * 8375359: Improve GC serviceability init staging > > Reviewed-by: ayang, tschatzl, stefank > > * 8377433: Test > com/sun/jmx/remote/NotificationMarshalVersions/TestSerializationMismatch.java > failed > > Reviewed-by: sspitsyn > > * 8377652: C2: Simplify Compile::flatten_alias_type for array > > Reviewed-by: roland, mhaessig > > * 8308637: AssertionError when using Trees.getScope in plug-in > > Reviewed-by: vromero > > * 8378397: Disable usage of system level jshell history in > test/hotspot/jtreg/runtime/os/TestWXHealing.java > > Reviewed-by: dholmes, jlahoda > > * 8378348: Obsolete ParallelRefProcEnabled > > Reviewed-by: jsikstro, tschatzl > > * 8378353: [PPC64] StringCoding.countPositives causes errors when the > length is not a proper 32 bit int > > Reviewed-by: mbaesken, dbriemann > > * 8373944: ObjectMonitor::ExitOnSuspend can call > java_lang_VirtualThread::set_onWaitingList() while in safepoint > > Reviewed-by: dholmes, fbredberg > > * 8377745: VoiceOver Identifies Hyperlink as Text > > Reviewed-by: kizune, dnguyen > > * 8376125: Out of memory in the CDS archive error with lot of classes > > Co-authored-by: Ioi Lam <[email protected]> > Reviewed-by: iklam, jiangli > > * 8377897: jpackage: make jdk.jpackage.internal.MockUtils available from > other packages > > Reviewed-by: almatvee > > * 8378115: (fs) sun.nio.fs.NativeBuffer no longer needs a cleaner > > Reviewed-by: alanb > > * 8378202: Remove AppContext from XAWT classes > > Reviewed-by: serb, azvegint > > * 8337853: Remove SunLayoutEngineKey and SunLayoutEngineFactory and its > cache. > > Reviewed-by: azvegint, dnguyen > > * 8378197: Remove AppContext from sun/swing/plaf/DesktopProperty.java > > Reviewed-by: serb, kizune > > * 8377924: Inconsistent parsing of XBM files after JDK-8361748 > 8377826: Eliminate code duplication in XbmImageDecoder > > Reviewed-by: dnguyen, prr, dmarkov > > * 8377905: gcc.md included with every build > > Reviewed-by: erikj, jsjolen, prr, dholmes > > * 8378378: Remove references to obsolete jdk.net.usePlainDatagramSocketImpl > property from tests > > Reviewed-by: alanb, vyazici, dfuchs > > * 8306591: SA and hotspot generate different hprof records for GC roots > > Reviewed-by: cjplummer, dholmes > > * 8377863: [s390x] Increase inlining thresholds to the same... Okay I'll refrain from merging in master like this in the future. Sorry for the noise. For now: ------------- PR Comment: https://git.openjdk.org/jdk/pull/24458#issuecomment-4024725775
