[all-commits] [llvm/llvm-project] adbc7a: [llvm-jitlink] Don't add process symbols to every ...

lhames via All-commits all-commits at lists.llvm.org
Mon Feb 7 21:29:28 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: adbc7a2665768d54a7b35e8d9e5d4a02dee6de6d
      https://github.com/llvm/llvm-project/commit/adbc7a2665768d54a7b35e8d9e5d4a02dee6de6d
  Author: Lang Hames <lhames at gmail.com>
  Date:   2022-02-08 (Tue, 08 Feb 2022)

  Changed paths:
    M llvm/tools/llvm-jitlink/llvm-jitlink.cpp

  Log Message:
  -----------
  [llvm-jitlink] Don't add process symbols to every JITDylib.

The addProcessSymbols function added a generator for process symbols to every
JITDylib in the session, but this is unhelpful default behavior (e.g. it will
cause the ORC runtime's definition of __cxa_atexit to be shadowed by the
process's definition for all JITDylibs except main).

Since the loadProcessSymbols function already added a generator to main we only
need to drop this function. Other JITDylibs wishing to resolve process symbols
can link against the main JITDylib by passing `-lmain`.




More information about the All-commits mailing list