RFR: 8374622: StressIncrementalInlining should also randomize the processing order [v2]

Marc Chevalier mchevalier at openjdk.org
Tue Jan 20 15:54:53 UTC 2026


> As it says: randomize the order the late inlines are processed, and slightly factor it with macro nodes.
> 
> I didn't add shuffle to the `GrowableArray` class since it seems a bit method specialized method (and it seems it'd be a controversial change to a widely use class), it would make `GrowableArray` depends on `Compile` for random number generation (or require a callback, for instance, giving a non-trivial signature and usage), I couldn't find other shuffling of such an object.
> There is also shuffling of `UniqueNodeList` (for `StressIGVN`), but it seems hard to unify: access and length are not written quite the same, it would probably be not simpler than duplicating the implem (which is simple, so it's fine in my opinion).
> 
> Thanks,
> Marc

Marc Chevalier has updated the pull request incrementally with one additional commit since the last revision:

  Randomize insertion

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/29110/files
  - new: https://git.openjdk.org/jdk/pull/29110/files/38569db4..f45e368e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=29110&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29110&range=00-01

  Stats: 7 lines in 1 file changed: 7 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/29110.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/29110/head:pull/29110

PR: https://git.openjdk.org/jdk/pull/29110


More information about the hotspot-compiler-dev mailing list