llvm-config with shared libraries in cmake builds broken (since r257003?)

Hi Andrew,

since today, I get:

$ llvm-config --link-shared --libs engine
llvm-config: error: libLLVM-3.8svn.so is missing

Looking at the log, this is most likely caused by your recent change. cmake shared library builds generate separate .so files analogous to the static library builds, e.g. libLLVMCodeGen.so (no version suffix, curiously enough). It would be nice if that wasn't broken :slight_smile:

Cheers,
Nicolai

since today, I get:

$ llvm-config --link-shared --libs engine
llvm-config: error: libLLVM-3.8svn.so is missing

Looking at the log, this is most likely caused by your recent change.
cmake shared library builds generate separate .so files analogous to the
static library builds, e.g. libLLVMCodeGen.so (no version suffix,
curiously enough). It would be nice if that wasn't broken :slight_smile:

After more investigation and help from Tom Stellard, it turns out that setting BUILD_SHARED_LIBS=on is what isn't supported with that change to llvm-config.

since today, I get:

$ llvm-config --link-shared --libs engine
llvm-config: error: libLLVM-3.8svn.so is missing

Looking at the log, this is most likely caused by your recent change.
cmake shared library builds generate separate .so files analogous to the
static library builds, e.g. libLLVMCodeGen.so (no version suffix,
curiously enough). It would be nice if that wasn’t broken :slight_smile:

After more investigation and help from Tom Stellard, it turns out that
setting BUILD_SHARED_LIBS=on is what isn’t supported with that change to
llvm-config.

Nicolai,

Sorry, I monitored the buildbots but clearly they don’t test that configuration. I’ll get onto it right away.

Cheers,
Andrew

since today, I get:

$ llvm-config --link-shared --libs engine
llvm-config: error: libLLVM-3.8svn.so is missing

Looking at the log, this is most likely caused by your recent change.
cmake shared library builds generate separate .so files analogous to the
static library builds, e.g. libLLVMCodeGen.so (no version suffix,
curiously enough). It would be nice if that wasn’t broken :slight_smile:

After more investigation and help from Tom Stellard, it turns out that
setting BUILD_SHARED_LIBS=on is what isn’t supported with that change to
llvm-config.

Nicolai,

Sorry, I monitored the buildbots but clearly they don’t test that configuration. I’ll get onto it right away.

Nicolai,

My proposed fix is here: http://reviews.llvm.org/D15986. If you could test it out, that would be great.

Cheers,
Andrew