[llvm] r294264 - [LVI] Switch from BFS to DFS exploration order

Joerg Sonnenberger via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 18 15:29:31 PST 2017


On Mon, Feb 06, 2017 at 04:49:57PM -0800, Hans Wennborg wrote:
> On Mon, Feb 6, 2017 at 4:42 PM, Joerg Sonnenberger via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
> > On Tue, Feb 07, 2017 at 12:25:24AM -0000, Philip Reames via llvm-commits wrote:
> >> Author: reames
> >> Date: Mon Feb  6 18:25:24 2017
> >> New Revision: 294264
> >>
> >> URL: http://llvm.org/viewvc/llvm-project?rev=294264&view=rev
> >> Log:
> >> [LVI] Switch from BFS to DFS exploration order
> >
> > Thanks! Hans, any comment on merging this into 4.0? It fixes one of the
> > two remaining cases in pkgsrc where clang with normal optimiser settings
> > hits 2GB VA.
> 
> Well.. This just landed, and I don't think it's fixing a regression
> from 3.9. While the change sounds great, I think we should just let it
> go into the next release.

It's been a while and I still would like to see this and the follow-up
r294463 merged. That was discussed with Daniel Berlin already.

Joerg


More information about the llvm-commits mailing list