jhenderson added a comment.

I've not really looked into this patch significantly, so this may well be 
addressed in the patch, given I see you have modified stuff to do with the 
NATIVE build, but in the past I have seen LLVM using its own tools to build 
other parts of its system. I believe it was using llvm-nm to extract the list 
of symbols needed for export, possibly to do with part of the clang build, 
possibly even using this script, I don't remember. The problem was that it was 
using the just-built version of llvm-nm, rather than specifically one from a 
release build. On a debug build this caused particularly slow builds for me, so 
much so that I stopped building the relevant parts of LLVM. Please don't 
introduce a similar situation/make the situation worse (it's quite possible 
this was fixed some time ago, but I haven't tried recently, nor do I remember 
the exact thing causing the issue): much like tablegen, any parts of the LLVM 
build that use just-built tools should make use of release builds, even in 
debug configuration, at least if an appropriate cmake option is specified.


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

https://reviews.llvm.org/D149119

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

Reply via email to