<div class="gmail_quote">On Wed, Nov 2, 2011 at 10:18 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I think this line is still necessary to cause llvm-config to be built at all; otherwise nothing depends on it. I'm going to add it back to address problems reported with CMake install.</blockquote><div><br></div><div>
Yea, testing showed that we still need the target, and also still need the libs -> target dependency edge to ensure proper ordering. Removing the modification of COMMON_DEPENDS is all good though. See r143548 for the changes I made. </div>
</div>