[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