Re: [PATCH] D12817: [CMake] Add cache scripts for Apple-style clang builds.

2015-09-15 Thread Eric Christopher via cfe-commits
echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. Interesting. I'm down with putting this in - would it make more sense to put this alongside the buildit bits in the llvm/utils repo? -eric http://reviews.llvm.org/D12817

Re: [PATCH] D12817: [CMake] Add cache scripts for Apple-style clang builds.

2015-09-15 Thread Chris Bieneman via cfe-commits
beanz added a comment. I had considered putting it in the LLVM repo, but this stuff actually works on standalone clang builds too, so I felt it might be better here. -Chris http://reviews.llvm.org/D12817 ___ cfe-commits mailing list

Re: [PATCH] D12817: [CMake] Add cache scripts for Apple-style clang builds.

2015-09-15 Thread Eric Christopher via cfe-commits
Ah sure. Make it so then. On Tue, Sep 15, 2015, 1:04 PM Chris Bieneman wrote: > beanz added a comment. > > Standalone as in using the base system's LLVM installation. So you don't > actually have to have an llvm checkout. > > There's goop in the root clang CMakeLists file to