Maurizio Cimadamore
Maurizio Cimadamore
> Do we have any performance tests available to see if there are any potential impacts? I've run all micro benchmarks whose name match `LoopOver*`. No regression was found. Few...
> > Do we have any performance tests available to see if there are any potential impacts? > > I've run all micro benchmarks whose name match `LoopOver*`. No regression...
> Separately, we might be missing a few long argument accepting guard methods for simpler cases as I suspect they are still focused more on int index types. Not sure...
> > Separately, we might be missing a few long argument accepting guard methods for simpler cases as I suspect they are still focused more on int index types. >...
> some additional overhead e.g., > > ``` > if (derefAdapters.length == 0) { > // insert align check for the root layout on the initial MS + offset >...
> > > Do we have any performance tests available to see if there are any potential impacts? > > > > > > I've run all micro benchmarks whose...
See this comment: https://github.com/openjdk/jdk/pull/19251#issuecomment-2115361171
Many thanks for getting back to me. I've verified that indeed there are errors on the browser console, and raised the issue to our team in charge of the server...
> So > `https://cr.openjdk.org/~mcimadamore/jdk/8331865/loop_over_00_baseline.json` > will become > `https://corsproxy.io/?https://cr.openjdk.org/~mcimadamore/jdk/8331865/loop_over_00_baseline.json` and that will be loadable with the visualizer > https://jmh.morethan.io/?source=https://corsproxy.io/?https://cr.openjdk.org/~mcimadamore/jdk/8331865/loop_over_00_baseline.json Nice trick/workaround - thanks!