RFR: CODETOOLS-7903056: perfasm can report wrong stub name

[Aleksey Shipilev]

On Thu, 11 Nov 2021 03:14:36 GMT, Nick Gasson <ngasson at openjdk.org> wrote:

> This patch changes `IntervalMap::get` to check that the value falls within an interval's high/low bounds. Otherwise it returns the closest entry < value which can be misleading.
> 
> Example of the new output, where previously the `<unknown>` line reported the wrong stub routine:
> 
> 
>  18.19%           libjvm.so  ProgrammableUpcallHandler::on_entry
>  13.94%                      <unknown>
>   9.39%           libjvm.so  ProgrammableUpcallHandler::on_exit
>   8.36%         c2, level 4  java.lang.invoke.LambdaForm$MH.0x00000008001b8400::invoke, version 1595

Changes requested by shade (Committer).

jmh-core/src/main/java/org/openjdk/jmh/util/IntervalMap.java line 42:

> 40:         @Override
> 41:         public int compareTo(Interval other) {
> 42:             return (int)(this.from - other.from);

Suggestion:

            return Long.compare(this.from, other.from);

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

PR: https://git.openjdk.java.net/jmh/pull/54