[all-commits] [llvm/llvm-project] f5602e: [OpenMP] Disabled profiling in `libomp` by default...

Shilei Tian via All-commits all-commits at lists.llvm.org
Wed Feb 3 16:18:47 PST 2021


  Branch: refs/heads/release/12.x
  Home:   https://github.com/llvm/llvm-project
  Commit: f5602e0bf31ab590da19fa357980a753dbfd666e
      https://github.com/llvm/llvm-project/commit/f5602e0bf31ab590da19fa357980a753dbfd666e
  Author: Shilei Tian <tianshilei1992 at gmail.com>
  Date:   2021-02-03 (Wed, 03 Feb 2021)

  Changed paths:
    M openmp/CMakeLists.txt
    M openmp/docs/design/Runtimes.rst
    M openmp/runtime/CMakeLists.txt
    M openmp/runtime/src/CMakeLists.txt
    M openmp/runtime/src/kmp_config.h.cmake
    M openmp/runtime/src/kmp_runtime.cpp

  Log Message:
  -----------
  [OpenMP] Disabled profiling in `libomp` by default to unblock link errors

Link error occurred when time profiling in libomp is enabled by default
because `libomp` is assumed to be a C library but the dependence on
`libLLVMSupport` for profiling is a C++ library. Currently the issue blocks all
OpenMP tests in Phabricator.

This patch set a new CMake option `OPENMP_ENABLE_LIBOMP_PROFILING` to
enable/disable the feature. By default it is disabled. Note that once time
profiling is enabled for `libomp`, it becomes a C++ library.

Reviewed By: jdoerfert

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

(cherry picked from commit c571b168349fdf22d1dc8b920bcffa3d5161f0a2)




More information about the All-commits mailing list