[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