[llvm] BPF: Emit an error for illegal LD_imm64 insn when LLVM_ENABLE_ASSERTI… (PR #74035)

via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 7 10:09:21 PST 2023


yonghong-song wrote:

No problem, it is still a mystery why old build interferes with new build, I suppose cmake should take care of this. But anyway, glad now we are on the same page.

I think the current change is okay and I don't plan to check other asserts at this moment. Those asserts are mostly for BPF llvm backend developers. I assume most (if not all) users should be able to write inline asm with correct syntax if they intend to embed inline asm in their bpf programs. So please stamp if you get a chance. Thanks!

https://github.com/llvm/llvm-project/pull/74035


More information about the llvm-commits mailing list