[libcxx-commits] [PATCH] D107835: [libunwind] Compile with -Wunused-but-set-variable

[Fangrui Song via Phabricator via libcxx-commits]

MaskRay added a comment.

Some users do standalone builds. They use `libunwind/CMakeLists.txt` as the main config, skipping `llvm/CMakeLists.txt`.

`cmake -GNinja -Hlibunwind -B/tmp/out/play` In this setting, some `llvm/CMakeLists.txt` options (e.g. `-fvisibility-inlines-hidden`) are not inherited to libunwind.

Note that libcxx sets its own options as well as inherits options from `llvm/CMakeLists.txt`. I am on the fence whether libunwind should inherit llvm/'s options.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107835/new/

https://reviews.llvm.org/D107835