Skip to content

Conversation

@llvmbot
Copy link
Member

@llvmbot llvmbot commented Aug 3, 2022

omp50_taskdep_depobj.c relies on the test compiler's omp.h file.
If the test compiler does not have an omp.h file, then use the one
within the build tree.

Fixes: llvm/llvm-project#56820
Differential Revision: https://reviews.llvm.org/D131000

(cherry picked from commit 9cf6511)
@llvmbot
Copy link
Member Author

llvmbot commented Aug 3, 2022

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

@tru
Copy link
Contributor

tru commented Aug 7, 2022

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

@mgorny
Copy link
Member

mgorny commented Aug 7, 2022

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

I've requested it and it was approved in llvm/llvm-project#56820.

@tru
Copy link
Contributor

tru commented Aug 7, 2022

I've requested it and it was approved in llvm/llvm-project#56820.

Yeah we always just double check when the PR is ready to make sure the bot didn't mess anything up. I will mark it for inclusion.

@mgorny
Copy link
Member

mgorny commented Aug 7, 2022

Ah. I've just applied this on top of the current 15.x and confirmed it solves the issue for me.

@tru tru merged commit 63be0bc into release/15.x Aug 9, 2022
@tru tru deleted the llvm-issue56820 branch August 9, 2022 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[openmp] Test tasking/omp50_taskdep_depobj.c fails w/ standalone build unless openmp is installed first

5 participants