[all-commits] [llvm/llvm-project] fef73b: [OpenMP][libomp] Cleanup version script and export...

Jonathan Peyton via All-commits all-commits at lists.llvm.org
Tue Dec 6 06:34:20 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fef73b81e9a742c4db61ded747c801e3717dae78
      https://github.com/llvm/llvm-project/commit/fef73b81e9a742c4db61ded747c801e3717dae78
  Author: Jonathan Peyton <jonathan.l.peyton at intel.com>
  Date:   2022-12-06 (Tue, 06 Dec 2022)

  Changed paths:
    M openmp/runtime/src/exports_so.txt
    M openmp/runtime/src/kmp_runtime.cpp
    M openmp/runtime/src/z_Linux_util.cpp

  Log Message:
  -----------
  [OpenMP][libomp] Cleanup version script and exported symbols

This patch fixes issues seen once https://reviews.llvm.org/D135402 is applied.

The exports_so.txt file attempts to export functions which may not exist
depending on which features are enabled/disabled in the OpenMP
runtime library. There are not many of these so exporting dummy
symbols is feasible.

* Export dummy __kmp_reset_stats() function when stats is disabled.
* Export dummy debugging data when USE_DEBUGGER is disabled
* Export dummy __kmp_itt_[fini|init]_ittlib() functions
   when ITT Notify is disabled
* Export dummy __kmp_reap_monitor() function when KMP_USE_MONITOR
  is disabled
* Remove __kmp_launch_monitor and __kmp_launch_worker from being exported.
  They have been static symbols since library inception.

Fixes: https://github.com/llvm/llvm-project/issues/58858
Differential Revision: https://reviews.llvm.org/D138049




More information about the All-commits mailing list