[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