[PATCH] D29866: [PDSE] Add PDSE.

[Daniel Berlin via llvm-commits]

>
>
>
> ================
> Comment at: lib/Transforms/Scalar/PDSE.cpp:920
> +    if (!PDT.getRootNode()) {
> +      DEBUG(dbgs() << "FIXME: ran into the PDT bug. nothing we can
> do.\n");
> +      return false;
> ----------------
> Can you please describe the `PDT` bug? I'm not really familiar with it (or
> I call it with another name :)
>
>
This is because the postdominator tree is *badly* broken in a myriad of
ways.

It's possible to hit this particular issue in a number of ways.

In practice, we are going to have to fix post-dominators before we ever
turn on this pass.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170618/0f8f062d/attachment-0001.html>