[PATCH] D104648: [Mem2Reg] Use poison instead of undef for read of uninitialized memory

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 22 00:35:15 PDT 2021


nikic added a comment.

@aqjune I don't think performance is really relevant here: It's a necessary step to remove undef in the future. I think this is one of the main ways left in which we introduce undef in IR.

@nlopes Ugh, this is ugly. The fact that clang is using this also means that we can't just ignore bitcode autoupgrade. Adding a freeze on every load seems like a really big hammer, but I'm not sure what else could be done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104648



More information about the llvm-commits mailing list