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

[Hans Wennborg via llvm-commits]

,

On Sat, Feb 18, 2017 at 3:29 PM, Joerg Sonnenberger <joerg at bec.de> wrote:
> 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.

I'm sorry, but I really do think this kind of change is too risky so
late in the release process.

 - Hans