RFR: 8301628: RISC-V: c2 fix pipeline class for several instructions [v2]
Gui Cao
gcao at openjdk.org
Thu Feb 2 02:58:51 UTC 2023
> HI,
>
> The current C2 instructions can use some more accurate ins_pipe, eg.:
>
> instruct regI_not_reg(iRegINoSp dst, iRegI src1, immI_M1 m1) %{
> match(Set dst (XorI src1 m1));
> ins_cost(ALU_COST);
> format %{ "xori $dst, $src1, -1\t#@regI_not_reg" %}
>
> ins_encode %{
> __ xori(as_Register($dst$$reg), as_Register($src1$$reg), -1);
> %}
>
> ins_pipe(ialu_reg);
> %}
>
>
> We can use the more accurate pipe_class `ialu_reg_imm` instead.
>
> Please take a look and have some reviews. Thanks a lot.
>
> ## Testing:
> - hotspot and jdk tier1 on unmatched board without new failures
Gui Cao has updated the pull request incrementally with one additional commit since the last revision:
Fix typo error
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/12379/files
- new: https://git.openjdk.org/jdk/pull/12379/files/64666538..317cb90e
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=12379&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=12379&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/12379.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/12379/head:pull/12379
PR: https://git.openjdk.org/jdk/pull/12379
More information about the hotspot-compiler-dev
mailing list