[libcxx-commits] [PATCH] D70815: Enable `-funwind-tables` flag when building libunwind

Sergej Jaskiewicz via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Nov 29 11:19:45 PST 2019


broadwaylamb marked an inline comment as done.
broadwaylamb added inline comments.


================
Comment at: libunwind/CMakeLists.txt:236
+# libunwind quite useless in this setup.
+set(CMAKE_TRY_COMPILE_TARGET_TYPE STATIC_LIBRARY)
+
----------------
broadwaylamb wrote:
> smeenai wrote:
> > phosek wrote:
> > > This option is only available since CMake 3.6, but the minimum requirement for LLVM is 3.4. There's a separate discussion on llvm-dev about bumping that version, but until that happens, this should be behind a condition to avoid breaking users and bots that use CMake 3.4 and 3.5.
> > Won't it just end up being a no-op on older CMake version?
> > 
> > I believe compiler-rt has some custom machinery set up to run compile-only configuration checks. Idk if that's worth trying to replicate here though if we'll be upgrading CMake versions soon anyway.
> >Won't it just end up being a no-op on older CMake version?
> 
> Agreed, we're just setting a variable here, nothing should go wrong with older versions.
@compnerd I'm curious what do you think about this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70815





More information about the libcxx-commits mailing list