On 13/11/2024 13:41, Hardevsinh Palaniya wrote:
The original code checks 'if (ARC_CC_AL)', which is always true since ARC_CC_AL is a constant. This makes the check redundant and likely obscures the intention of verifying whether the jump is conditional. Updates the code to check cond == ARC_CC_AL instead, reflecting the intent to differentiate conditional from unconditional jumps. Suggested-by: Vadim Fedorenko <vadim.fedorenko@xxxxxxxxx> Signed-off-by: Hardevsinh Palaniya <hardevsinh.palaniya@xxxxxxxxxxxxxxxxx> --- Changelog in V2: - Changed subject line - Updated condition check to 'if (cond == ARC_CC_AL)' instead of removing it Link for v1: https://lore.kernel.org/bpf/e6d27adb-151c-46c1-9668-1cd2b492321b@xxxxxxxxx/T/#t ---
Reviewed-by: Vadim Fedorenko <vadim.fedorenko@xxxxxxxxx>
