Re: [PATCH] ARC: bpf_jit_arcv2: Remove redundant condition check

From: Shahab Vahedi
Date: Tue Nov 12 2024 - 21:12:15 EST


Vadim Fedorenko wrote:

> The original code is obviously optimized out, but the intention, I
> believe, was to check if the jump is conditional or not.
> So the proper fix should change the code to check cond:
>
> - if (ARC_CC_AL)
> + if (cond == ARC_CC_AL)

That is absolutely correct. If a new patch is not submitted soon
I'll try to fix it myself.

Cheers,
Shahab