On Thu, 21 Sep 2023 16:44:52 GMT, Paul Sandoz <psan...@openjdk.org> wrote:
>> Srinivas Vamsi Parasa has updated the pull request incrementally with one >> additional commit since the last revision: >> >> change variable names of indexPivot* to pivotIndex* > > test/jdk/java/util/Arrays/Sorting.java line 30: > >> 28: * @build Sorting >> 29: * @run main/othervm -XX:+UnlockDiagnosticVMOptions >> -XX:DisableIntrinsic=_arraySort,_arrayPartition, Sorting -shortrun >> 30: * @run main/othervm -XX:CompileThreshold=1 -XX:-TieredCompilation >> Sorting -shortrun > > It would be useful to target the compilation threshold only for the intrinsic > methods, but i suspect that is not possible? Hi Paul, Did some further study and found that it's possible to scale the compilation threshold only for the intrinsic methods (`sort `and `partition`) using the `CompileCommand` and `CompileThersholdScaling` ``` -XX:-TieredCompilation -XX:CompileCommand=CompileThresholdScaling,java.util.DualPivotQuicksort::sort,0.0001 -XX:CompileCommand=CompileThresholdScaling,java.util.DualPivotQuicksort::partition,0.0001 Will push this change along with build script change suggested by Magnus. Thanks, Vamsi ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14227#discussion_r1333797193