[Lldb-commits] [compiler-rt] [libcxx] [libcxxabi] [libunwind] [lldb] [llvm] [compiler-rt] Disable LLVM_ENABLE_PER_TARGET_RUNTIME_DIR=ON on AIX. (PR #131200)

[Daniel Chen via lldb-commits]

DanielCChen wrote:

> > > Wouldn't it be more maintainable to just force the value to OFF for AIX where the `option()` is defined?
> > 
> > 
> > Unfortunately, the cmake command line option `-DLLVM_ENABLE_PER_TARGET_RUNTIME_DIR=ON` overrides the `option()` in the cmake file.
> 
> Sorry if this was not clear, I didn't mean changing the default value of the option, but instead forcing it explicitly in the one central place where the option is defined.

Right. What I observed is that no matter how and where I explicitly `set` the option OFF in top CMakeLists.txt of `llvm` or `clang`, if users build clang with the cmake flang`-DLLVM_ENABLE_PER_TARGET_RUNTIME_DIR=ON` in their scripts, it will override the explicit set-to-OFF in the cmake. This may be the same reason that all the places I have changed also have `NOT APPLE` to disable it.

https://github.com/llvm/llvm-project/pull/131200