[llvm] [BPF] expand mem intrinsics (memcpy, memmove, memset) (PR #97648)
Yingchi Long via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 4 19:13:07 PDT 2024
inclyc wrote:
Hi @eddyz87,
Thanks for testing this PR in the kernel! :heart:
> Because of the way kernel BPF verifier works, the unrolled version is preferable to the loop (verifier traces execution, so processing a loop would take more instruction budget). Is it possible to make `llvm::expandMemMoveAsLoop()` respect the limits set by `MaxStoresPerMemset`?
I'd like to figure out how to, and I kindly ask if "memset" is unrolled then it is friendly to the verifier, what should we do if the memset is called with dynamic length? (Expand it, or report error?)
https://github.com/llvm/llvm-project/pull/97648
More information about the llvm-commits
mailing list