[PATCH] D21948: [DSE] fix - missing store to runtime stack in thunk with tail call bvval arg
Gerolf Hoflehner via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 2 18:35:50 PDT 2016
Sent from my iPhone
> On Jul 2, 2016, at 5:15 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
>
> 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.
Why?
> 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