[PATCH] D30350: [LSR] Add a cap for reassociation of AllFixupsOutsideLoop type LSRUse to protect compile time

Wei Mi via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 8 11:22:02 PST 2017


On Wed, Mar 8, 2017 at 10:50 AM, Davide Italiano via Phabricator <
reviews at reviews.llvm.org> wrote:

> davide added a comment.
>
> Wei, any progress on this? We should either commit a workaround or revert
> the patchset to unblock folks.
>
>
Note the problem is always there, so reverting the patchset doesn't help
and we need a fix anyway.

The workaround is what I have for now. I am waiting reviewers' opinion to
see if it is a sensible workaround. And I havn't started to work on the
improvement of AllFixupsOutsideLoop kind of LSRUse issue. I admit it is
relatively low priority in my list.

Thanks,
Wei.


>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D30350
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170308/30f684c7/attachment.html>


More information about the llvm-commits mailing list