[llvm] [BPF] Report warning for some insn imm requiring valid range in inline asm (PR #142989)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 12 11:53:06 PDT 2025
eddyz87 wrote:
Latest changes lgtm.
However, on a second thought, maybe `Imm < INT_MIN || Imm > UINT_MAX` makes more sense in the end.
E.g. corresponding selftest change is an unnecessary obfuscation:
```
--- a/tools/testing/selftests/bpf/progs/verifier_and.c
+++ b/tools/testing/selftests/bpf/progs/verifier_and.c
@@ -99,7 +99,7 @@ __naked void known_subreg_with_unknown_reg(void)
call %[bpf_get_prandom_u32]; \
r0 <<= 32; \
r0 += 1; \
- r0 &= 0xFFFF1234; \
+ r0 &= -60876; \
```
https://github.com/llvm/llvm-project/pull/142989
More information about the llvm-commits
mailing list