[Lldb-commits] [PATCH] D29333: [CMake] Add accurate dependency specifications

[Chris Bieneman via Phabricator via lldb-commits]

beanz added a comment.

In https://reviews.llvm.org/D29333#661979, @labath wrote:

> I was thinking about that as well. I am not sure if it will work if we actually need multiple iterations of the loop to get all the dependencies converging, but it may be worth trying out.


The way it is supposed to work, CMake will duplicate the libraries on the command line in the right order to avoid the need for looping at all.

> I am hoping we will be able to reduce the loops in the future. My plan is that after finishing with the Log class,  to move it to a new module, with clean and well defined dependencies (I need to remove all ConstStrings and LLDB Streams from it first), After that I want start moving other low level functionalities there as well. That got a bit delayed now, but I should be back to that in a week or two.

This would be great!


https://reviews.llvm.org/D29333