[PATCH] D144057: [GVN] permit GVN of non-local loads for ASAN unless undef or alloca is produced

[Evgenii Stepanov via Phabricator via llvm-commits]

eugenis added a comment.

Not convinced that this is right. The original fix is for a false postive, not a false negative - i.e., we want to prevent speculation of a memory access that is not provably safe.

See https://lists.llvm.org/pipermail/llvm-dev/2015-November/092484.html. Note that the report is on heap memory, not alloca.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144057