[PATCH] D13363: [DeadStoreElimination] Add support for non-local DSE

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 17:57:17 PST 2015


I'd prefer if numbers were collected indicating which thresholds worked
best.

On Thursday, December 10, 2015, Ivan Baev <ibaev at codeaurora.org> wrote:

> ivanbaev added a comment.
>
> Hi Chad, thanks for helping with that. Is compile-time increase the
> remaining problem?
> One quick fix could be to change 100  to 10 for the max number on
> non-local attempts in a function:
>
>   static const unsigned MaxNonLocalAttempts = 100;
>
> We've chosen 100 arbitrarily. The newly added DSE tests should pass with
> 10.
>
> Ivan
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D13363
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151210/cd36a35b/attachment.html>


More information about the llvm-commits mailing list