[PATCH] D57367: [DAGCombine] Do several rounds of combine for nodes using SimplifyDemandedBits.

Nirav Davé via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 21 13:55:58 PST 2019


 I believe I only have one patch missing (anyextend of and with a constant
doesn't apply when it could) to make the dangling node removal patches
strictly improvements and thus landable. At that point we should be
converging to a local maxima in the DAG space quickly like one would hope
and we could potentially loop until no work is done while we figure out
where we fail to add nodes to the worklist in our first pass.

On Thu, Feb 21, 2019, 16:34 Amaury SECHET via Phabricator <
reviews at reviews.llvm.org> wrote:

> deadalnix added a comment.
> Herald added a subscriber: jdoerfert.
>
> @niravd Any progress ?
>
>
> Repository:
>   rL LLVM
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D57367/new/
>
> https://reviews.llvm.org/D57367
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190221/03289032/attachment.html>


More information about the llvm-commits mailing list