[PATCH] D156497: [BPF] Emit UNDEF rather than constant 0

[Yonghong Song via Phabricator via llvm-commits]

yonghong-song added a comment.

In D156497#4583451 <https://reviews.llvm.org/D156497#4583451>, @tamird wrote:

>> With current clang, could you show me the error message with 'constant 0' vs. 'undef'? We can then decide to pick the one which is more user friendly.
>
> You're referring to verifier errors, right? With "constant 0" there is no verifier error - the program will just do the wrong thing (not what the programmer intended). I don't have the "undef" verifier error on hand, I can get that to you tomorrow.

Not about verifier errors but errors at compilation time. The compilation should fail in two instances in this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156497