Skip to content

release/22.x: [OpenMP][omptest] Improve CMake and address review comments (#159416)#183775

Closed
llvmbot wants to merge 2 commits intollvm:release/22.xfrom
llvmbot:issue159416
Closed

release/22.x: [OpenMP][omptest] Improve CMake and address review comments (#159416)#183775
llvmbot wants to merge 2 commits intollvm:release/22.xfrom
llvmbot:issue159416

Conversation

@llvmbot
Copy link
Member

@llvmbot llvmbot commented Feb 27, 2026

Backport d5fea7e 696b91a

Requested by: @mstorsjo

)

Avoid explicit ABI breaking check deactivation
Replace whole-archive linking with dedicated build of GoogleTest lib

Addresses remaining post-merge review comments of
llvm#154786

(cherry picked from commit d5fea7e)
Fix visibility of target properties:
 * INCLUDE_DIRECTORIES
 * COMPILE_DEFINITIONS

implicitly pulled in from `default_gtest`.

Fixes: llvm#159416
(cherry picked from commit 696b91a)
@llvmbot
Copy link
Member Author

llvmbot commented Feb 27, 2026

@mhalk @Kewen12 What do you think about merging this PR to the release branch?

@llvmbot llvmbot added openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime labels Feb 27, 2026
@llvmbot llvmbot requested review from Kewen12 and mhalk February 27, 2026 17:22
@mstorsjo mstorsjo requested a review from Meinersbur February 27, 2026 17:26
@mstorsjo mstorsjo changed the title release/22.x: [OpenMP][omptest] Fix CMake target properties (#176802) release/22.x: [OpenMP][omptest] Improve CMake and address review comments (#159416) Feb 27, 2026
Copy link
Collaborator

@jprotze jprotze left a comment

Choose a reason for hiding this comment

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

Follow-up fixes for #154786, which is already on release/22.x.

@github-project-automation github-project-automation bot moved this from Needs Triage to Needs Merge in LLVM Release Status Feb 27, 2026
Copy link
Member

@Meinersbur Meinersbur left a comment

Choose a reason for hiding this comment

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

LGTM

@dyung
Copy link
Collaborator

dyung commented Mar 5, 2026

The two commits were merged into release/22.x with commits 10b0c4c and 8e5ee76.

@dyung dyung closed this Mar 5, 2026
@dyung dyung moved this from Needs Merge to Done in LLVM Release Status Mar 5, 2026
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

Development

Successfully merging this pull request may close these issues.

5 participants