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

Shilei Tian via All-commits all-commits at lists.llvm.org
Thu Jan 28 04:24:52 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c571b168349fdf22d1dc8b920bcffa3d5161f0a2
      https://github.com/llvm/llvm-project/commit/c571b168349fdf22d1dc8b920bcffa3d5161f0a2
  Author: Shilei Tian <tianshilei1992 at gmail.com>
  Date:   2021-01-28 (Thu, 28 Jan 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




More information about the All-commits mailing list