[all-commits] [llvm/llvm-project] be6e2e: [offload] - Fix issue with standalone debug offloa...

[Michał Górny via All-commits]

  Branch: refs/heads/release/19.x
  Home:   https://github.com/llvm/llvm-project
  Commit: be6e2e72cb51f95c852e6d6567c8b2d066dfd19e
      https://github.com/llvm/llvm-project/commit/be6e2e72cb51f95c852e6d6567c8b2d066dfd19e
  Author: estewart08 <ethan.stewart at amd.com>
  Date:   2024-12-17 (Tue, 17 Dec 2024)

  Changed paths:
    M offload/plugins-nextgen/common/CMakeLists.txt

  Log Message:
  -----------
  [offload] - Fix issue with standalone debug offload build (#104647)

Error: CommandLine Error: Option 'attributor-manifest-internal'
registered more than once

During the standalone debug build of offload the above error is seen at
app runtime when using a prebuilt llvm with LLVM_LINK_LLVM_DYLIB=ON.
This is caused by linking both libLLVM.so and various archives that are
found via llvm_map_components_to_libnames for jit support.


  Commit: e3b2d0149ffadef53fe8165f55e648466c025416
      https://github.com/llvm/llvm-project/commit/e3b2d0149ffadef53fe8165f55e648466c025416
  Author: Michał Górny <mgorny at gentoo.org>
  Date:   2024-12-17 (Tue, 17 Dec 2024)

  Changed paths:
    M offload/CMakeLists.txt

  Log Message:
  -----------
  [offload] Include CheckCXXCompilerFlag in CMake

Include CheckCXXCompilerFlag before it is used in the top-level CMake
file.  This fixes standalone builds, but it is also cleaner than
assuming that some previous CMake file will include it for us.


  Commit: 5ce79b5045ae2e362fbd7b28351b10634d05b0f5
      https://github.com/llvm/llvm-project/commit/5ce79b5045ae2e362fbd7b28351b10634d05b0f5
  Author: Michał Górny <mgorny at gentoo.org>
  Date:   2024-12-17 (Tue, 17 Dec 2024)

  Changed paths:
    M offload/test/CMakeLists.txt

  Log Message:
  -----------
  [offload] [test] Add "omp" test dependency only when present

Add the `omp` target dependency to tests only when the respective target
is present, i.e. when not building standalone against system libomp.


  Commit: 909104bb5a0540a3bffaa4a816efc1c54be81969
      https://github.com/llvm/llvm-project/commit/909104bb5a0540a3bffaa4a816efc1c54be81969
  Author: Michał Górny <mgorny at gentoo.org>
  Date:   2024-12-17 (Tue, 17 Dec 2024)

  Changed paths:
    M offload/CMakeLists.txt

  Log Message:
  -----------
  [offload] Define OPENMP_TEST*_FLAGS in standalone builds

Copy the `OPENMP_TEST_FLAGS` and `OPENMP_TEST_OPENMP_FLAGS` from openmp
for standalone builds, as required by the lit configs.


  Commit: b70bf59262cbae12728bd866506debff5c89eb4e
      https://github.com/llvm/llvm-project/commit/b70bf59262cbae12728bd866506debff5c89eb4e
  Author: Michał Górny <mgorny at gentoo.org>
  Date:   2024-12-17 (Tue, 17 Dec 2024)

  Changed paths:
    M offload/CMakeLists.txt

  Log Message:
  -----------
  [offload] Define LIBOMPTARGET_OPENMP_*_FOLDER in standalone builds

Define `LIBOMPTARGET_OPENMP_HEADER_FOLDER`
and `LIBOMPTARGET_OPENMP_HOST_RTL_FOLDER` needed for testing
in standalone builds.


  Commit: 2384a062fc1cc9d115c537f0cfe36ae6028e7bfd
      https://github.com/llvm/llvm-project/commit/2384a062fc1cc9d115c537f0cfe36ae6028e7bfd
  Author: Michał Górny <mgorny at gentoo.org>
  Date:   2024-12-17 (Tue, 17 Dec 2024)

  Changed paths:
    M offload/cmake/OpenMPTesting.cmake

  Log Message:
  -----------
  [cmake] Use DetectTestCompiler from openmp directory

Fix the DetectTestCompiler project use to reference the openmp source
tree, since the respective files were not copied to offload, and there
is no point in duplicating them.

Fixes #90333


Compare: https://github.com/llvm/llvm-project/compare/d2953ab0a79f...2384a062fc1c

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications