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

Reply via email to