RFR: 8255565: [Vector API] Add missing format strings for extract instructs in x86.ad
Jatin Bhateja
jbhateja at openjdk.java.net
Thu Oct 29 07:42:44 UTC 2020
On Thu, 29 Oct 2020 02:07:03 GMT, Jie Fu <jiefu at openjdk.org> wrote:
> Hi all,
>
> Extract instructs in x86.ad like extractI/vextractI/extractL missed the format strings (format %{ ... %}).
> When analyzing or debugging with PrintOptoAssembly, it's hard to map the generated assembly code to the MachNode instructs without the format strings.
> So it would be better to fix it.
>
> Thanks a lot.
> Best regards,
> Jie
Hi @DamonFool , thanks for fixing these, format string gets emitted for debug builds with +PrintOptoAssembly, few of them are muti-match patterns, may be introducing a new format specifier like %t which is replaced by BasicType of ideal node through ADLC extension could make it more clear.
LGTM
-------------
PR: https://git.openjdk.java.net/jdk/pull/920
More information about the hotspot-compiler-dev
mailing list