[PATCH] D50717: [SimplifyCFG] Fix BasicBlock use-after-free

Chijun Sima via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 14 09:50:38 PDT 2018


NutshellySima added a comment.

In https://reviews.llvm.org/D50717#1199198, @dmgreen wrote:

> LGTM
>
> I guess because it's only erasing it from a SmallPtrSet, it's not actually _using_ it, but this looks like a very sensible change.


Thanks for pointing it out. I didn't notice that. :)


Repository:
  rL LLVM

https://reviews.llvm.org/D50717





More information about the llvm-commits mailing list