[PATCH] D87114: [CMake][ExecutionEngine] add HAVE_(DE)REGISTER_FRAME as a config.h macros

[David Tenty via Phabricator via llvm-commits]

daltenty added inline comments.


================
Comment at: llvm/cmake/config-ix.cmake:209
+# Determine whether we can register EH tables.
+check_symbol_exists(__register_frame "${CMAKE_CURRENT_LIST_DIR}/unwind.h" HAVE_REGISTER_FRAME)
+check_symbol_exists(__deregister_frame "${CMAKE_CURRENT_LIST_DIR}/unwind.h" HAVE_DEREGISTER_FRAME)
----------------
MaskRay wrote:
> Is CMAKE_CURRENT_LIST_DIR needed?
Yes, looks like CMake just inserts the arg as-is into the test program and we want to use the header stub, not the system unwind.h.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87114