[PATCH] D116854: bpf: Report an error properly for unsupported opcodes in lowering operations

Yonghong Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 10 19:50:25 PST 2022


yonghong-song added a comment.

@dsanders Thanks for pointing out llvm_unreachable is only available if assertions are enabled. I am not aware of this since I always have assertions on with my build. So I think the correct fix should be chaning llvm_unreachable() to report_fatal_error(). The current patch almost works except some minor issues I pointed in the above.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116854



More information about the llvm-commits mailing list