On 9/5/24 12:52 PM, Palmer Dabbelt wrote:
We have cheap logical ops, so let's just move this back to the default
to take advantage of the standard branch/op hueristics.
gcc/ChangeLog:
PR target/116615
* config/riscv/riscv.h (LOGICAL_OP_NON_SHORT_CIRCUIT): Remove.
So on the BPI this is a pretty clear win. Not surprisingly perlbench
and gcc are the big winners. It somewhat surprisingly regresses x264,
deepsjeng & leela, but the magnitudes are smaller. The net from a cycle
perspective is 2.4%. Every benchmark looks better from a branch count
perspective.
So in my mind it's just a matter of fixing any testsuite fallout (I
would expect some) and this is OK.
jeff