RFR: 8344306: RISC-V: Add zicond
Robbin Ehn
rehn at openjdk.org
Tue Nov 26 13:21:38 UTC 2024
On Tue, 26 Nov 2024 10:46:24 GMT, Robbin Ehn <rehn at openjdk.org> wrote:
>> Seems the critical path when `+UseZicond` is longer than `-UseZicond`, I'm not sure how much impact the branch code will bring when `-UseZicond`.
>
> Not, as we are only removing one in a thousand branches, the different would be so little that I don't think I can measure it in a reasoanble amount of iterations. I think you need something very synthetic, e.g. write a hot loop where we can remove a branch in it to measure it.
> Seems the critical path when `+UseZicond` is longer than `-UseZicond`, I'm not sure how much impact the branch code will bring when `-UseZicond`.
As I said, those 4 instructions saves ~1 cycle compared branch in out-order cpu models.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22386#discussion_r1858529983
More information about the hotspot-dev
mailing list