[all-commits] [llvm/llvm-project] ae032e: [CMake][ExecutionEngine] add HAVE_(DE)REGISTER_FRA...

[David Tenty via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ae032e2714675354457c474f4c8f5581714e33d1
      https://github.com/llvm/llvm-project/commit/ae032e2714675354457c474f4c8f5581714e33d1
  Author: David Tenty <daltenty at ibm.com>
  Date:   2020-11-10 (Tue, 10 Nov 2020)

  Changed paths:
    M llvm/cmake/config-ix.cmake
    A llvm/cmake/unwind.h
    M llvm/include/llvm/Config/config.h.cmake
    M llvm/lib/ExecutionEngine/JITLink/EHFrameSupport.cpp
    M llvm/lib/ExecutionEngine/RuntimeDyld/RTDyldMemoryManager.cpp

  Log Message:
  -----------
  [CMake][ExecutionEngine] add HAVE_(DE)REGISTER_FRAME  as a config.h macros

The macro HAVE_EHTABLE_SUPPORT is used by parts of ExecutionEngine to tell __register_frame/__deregister_frame is available to register the
FDE for a generated (JIT) code. It's currently set by a slowly growing set of macro tests in the respective headers, which is updated now and then when it fails to link on some platform or another due to the symbols being missing (see for example https://bugs.llvm.org/show_bug.cgi?id=5715).

This change converts the macro in two HAVE_(DE)REGISTER_FRAME config.h macros (like most of the other HAVE_* macros) and set's them based on whether CMake can actually find a definition for these symbols to link to at configuration time.

Reviewed By: hubert.reinterpretcast

Differential Revision: https://reviews.llvm.org/D87114