[PATCH] D58157: Stop enabling clang-tools-extra automatically when clang is in LLVM_ENABLE_PROJECTS

Mehdi AMINI via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 13 15:02:27 PST 2019


mehdi_amini added a comment.

In D58157#1396824 <https://reviews.llvm.org/D58157#1396824>, @rnk wrote:

>   And, this change really just keeps us at parity with what we had with svn. We can always revisit the decision to merge the clang tools into clang. This particular change just gives us more options, today.


Sure, looks fine. The cost of reversing it isn't that high either (updating a bunch of bot configurations).
It'd still be nice to identify the end goal with most cfe people (i.e. even if you land this right now, discussing the desired layout in the monorepo)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58157/new/

https://reviews.llvm.org/D58157





More information about the cfe-commits mailing list