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.fedore...@linux.dev>
Signed-off-by: Hardevsinh Palaniya <hardevsinh.palan...@siliconsignals.io>
---
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-1cd2b4923...@linux.dev/T/#t
---
Reviewed-by: Vadim Fedorenko <vadim.fedore...@linux.dev>
_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
