yonghong-song wrote: @efriedma-quic Thanks for your comments. BPF llvm backend and kernel BPF jit try to make BPF<->other_architecture compatible. riscv started to support BPF starting in 2020 and probably BPF has limited usage so the bug is exposed until now. Yes, BPFTargetLowering::LowerCall does do sign-extend arguments marked "signext", and this can help BPF<->other_architecture too. Previously only for char/short (to cover architectures which only have 32bit subregister). Now we found int type also needs support. BPF community will continuously monitor any potential inter-arch incompatibility issue and once found will address them promptly.
https://github.com/llvm/llvm-project/pull/84874 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
