[PATCH] D11963: Create a __config_site file to capture configuration decisions.

Jonathan Roelofs via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 19 09:31:01 PDT 2015


jroelofs added a comment.

In http://reviews.llvm.org/D11963#227441, @EricWF wrote:

> @jroelofs What do you think of an approach like this?


Having two copies of the __config_site file makes me uncomfortable, but I could put up with that given that they're effectively the same for 99% of people who will want to build this library.

That being said, @mclow.lists raised a few concerns with the overall strategy... I don't want to keep pushing on this patch if his plan is to pocket-veto it.


http://reviews.llvm.org/D11963





More information about the cfe-commits mailing list