[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