Make building using LLVM/Clang easier. #1557

pull TheBlueMatt wants to merge 2 commits into bitcoin:master from TheBlueMatt:clang changing 2 files +32 −5

TheBlueMatt commented at 7:46 PM on July 4, 2012: member

This is based on #1514 and adds fixes to bitcoin-qt.pro to build with clang+lto without editing makefiles. After this, you can use: qmake bitcoin-qt.pro USE_CLANG=1 USE_LTO=1 && make or make -f makefile.unix CXX=clang++ LDFLAGS="-use-gold-plugin" CXXFLAGS=-emit-llvm Building bitcoind requires ld-gold, however bitcoin-qt does not.

Motivation here is a speedup: clang-lto: " block index 6523ms" gcc-lto: " block index 8497ms"

That said, <gmaxwell> Seriously, every time I've seen a major speedup with clang it was miscompiling the code. Still its interesting that there is a speedup that large, and making it easier to compile using clang is not bad.

Use linker-arguments only when linking

Passing linker-arguments when only compiling will cause warnings with
Clang. This change fixes those.

Signed-off-by: Giel van Schijndel <me@mortis.eu>

defcb24795

Make compiling using clang possible without editing makefiles. dbce4de426
TheBlueMatt commented at 8:27 PM on July 4, 2012: member

Unnecessary, possible with QMAKE_CXX/QMAKE_LFLAGS and friends.
TheBlueMatt closed this on Jul 4, 2012
DrahtBot locked this on Sep 8, 2021

Contributors

TheBlueMatt

Linked (view graph)

#1514 Use linker-arguments only when linking