Skip to content

[OpenMP][omptest] Fix CMake target properties#176802

Merged
mhalk merged 1 commit intollvm:mainfrom
mhalk:mhalk/fix/omptest-cmake-target
Jan 19, 2026
Merged

[OpenMP][omptest] Fix CMake target properties#176802
mhalk merged 1 commit intollvm:mainfrom
mhalk:mhalk/fix/omptest-cmake-target

Conversation

@mhalk
Copy link
Contributor

@mhalk mhalk commented Jan 19, 2026

Fix visibility of target properties:

  • INCLUDE_DIRECTORIES
  • COMPILE_DEFINITIONS

implicitly pulled in from default_gtest.

Fixes: #159416

Fix visibility of target properties:
 * INCLUDE_DIRECTORIES
 * COMPILE_DEFINITIONS

implicitly pulled in from default_gtest.

Fixes: llvm#159416
@llvmbot llvmbot added openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime labels Jan 19, 2026
@mhalk mhalk enabled auto-merge (squash) January 19, 2026 18:57
@mhalk mhalk merged commit 696b91a into llvm:main Jan 19, 2026
14 checks passed
Copy link
Contributor

@Kewen12 Kewen12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@mhalk mhalk deleted the mhalk/fix/omptest-cmake-target branch January 19, 2026 21:57
BStott6 pushed a commit to BStott6/llvm-project that referenced this pull request Jan 22, 2026
Fix visibility of target properties:
 * INCLUDE_DIRECTORIES
 * COMPILE_DEFINITIONS

implicitly pulled in from `default_gtest`.

Fixes: llvm#159416
@DavidSpickett
Copy link
Collaborator

This got the flang-libcxx back to green after a very long time being red. Thanks for all your effort in this area!

dyung pushed a commit that referenced this pull request Mar 5, 2026
Fix visibility of target properties:
 * INCLUDE_DIRECTORIES
 * COMPILE_DEFINITIONS

implicitly pulled in from `default_gtest`.

Fixes: #159416
(cherry picked from commit 696b91a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants