[PATCH] D21948: [DSE] fix - missing store to runtime stack in thunk with tail call bvval arg

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 2 17:15:44 PDT 2016


eli.friedman added a subscriber: eli.friedman.
eli.friedman added a comment.

Special-casing this in DSE makes no sense... alias analysis should always return the correct result.

In this case, alias analysis is correct; the definition of "tail" doesn't allow marking this call.  If clang is generating code like this, it's a bug in clang.


http://reviews.llvm.org/D21948





More information about the llvm-commits mailing list