[PATCH] D61524: [BPF] Support for compile once and run everywhere

Yonghong Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 13 12:06:03 PDT 2019


yonghong-song added a comment.

> getAsInstruction probably doesn't do what you want here. It actually creates a new Instruction*, instead of casting the expression. Probably this is the source of the verifier error.

Yes, this is indeed the problem. Thanks @eli.friedman Changing it to directly check opcode (Instruction::GetElementPtr) and operands fixed the issue.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61524/new/

https://reviews.llvm.org/D61524





More information about the llvm-commits mailing list