[PATCH] D57149: [StackProtector] Skip analysing dead users in HasAddressTaken, PR40436

[Matt Davis via Phabricator via llvm-commits]

mattd added a comment.

Hi @uabelho

The test case in your patch definitely causes a problem, but that's because the -start-before/-stop-after is used.  If I run that same test through llc, the UnreachableBlockElim pass kicks-in and drops the dead code.
I took a look at the code in your Janurary 24th comment.  I was unable to get the same dead block to surface, but most likely I had the wrong flag set.  How did you build the sample in that comment?


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

https://reviews.llvm.org/D57149