hintonda added a comment.

In https://reviews.llvm.org/D41660#966023, @phosek wrote:

> @hintonda I think this should be a platform file in 
> https://github.com/llvm-mirror/llvm/tree/master/cmake/platforms rather than 
> Clang cache file. Platform files are concerned with the host platform 
> (including cross-compilation), while cache files are  related to the 
> distribution setup. What you're trying to do is the former rather than the 
> latter. Some of the aspects of your setup like the bootstrap tool setup is 
> already handled by the 2-stage build so you should use that rather than 
> reimplementing your own solution.


Thanks for the pointer.  I'll rework the patch along the lines you suggest.


Repository:
  rC Clang

https://reviews.llvm.org/D41660



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to