Skip to content

Commit f8409af

Browse files
committed
[libunwind] Install the headers by default
Just like we install libc++ and libc++abi headers by default when we install the library, it makes sense to install the libunwind headers by default when we build libunwind. In the current state of things, there is an increased risk that folks are using older (previously installed) libunwind headers along with a recent libunwind dylib, which is not ideal. Differential Revision: https://reviews.llvm.org/D135663
1 parent c249463 commit f8409af

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

libunwind/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ option(LIBUNWIND_INCLUDE_TESTS "Build the libunwind tests." ${LLVM_INCLUDE_TESTS
4949
option(LIBUNWIND_IS_BAREMETAL "Build libunwind for baremetal targets." OFF)
5050
option(LIBUNWIND_USE_FRAME_HEADER_CACHE "Cache frame headers for unwinding. Requires locking dl_iterate_phdr." OFF)
5151
option(LIBUNWIND_REMEMBER_HEAP_ALLOC "Use heap instead of the stack for .cfi_remember_state." OFF)
52-
option(LIBUNWIND_INSTALL_HEADERS "Install the libunwind headers." OFF)
52+
option(LIBUNWIND_INSTALL_HEADERS "Install the libunwind headers." ON)
5353

5454
set(LIBUNWIND_LIBDIR_SUFFIX "${LLVM_LIBDIR_SUFFIX}" CACHE STRING
5555
"Define suffix of library directory name (32/64)")

0 commit comments

Comments
 (0)