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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 31 09:43:34 PDT 2021


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



More information about the llvm-commits mailing list