[PATCH] D149119: [CMake] Use LLVM own tools in extract_symbols.py

[James Henderson via Phabricator via cfe-commits]

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