New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix building XGBoost with libomp 15 #8384
Conversation
@trivialfis Can I get another round of review? |
# Homebrew not found | ||
HOMEBREW_LIBOMP_PREFIX='' | ||
fi | ||
OPENMP_CXXFLAGS="-Xpreprocessor -fopenmp -I${HOMEBREW_LIBOMP_PREFIX}/include" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's the difference between -Xclang and -Xpreprocessor ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No idea. I only know that Xpreprocessor works on my Macbook but Xclang doesn't.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it has to do with the fact that we have to specify extra flags to the preprocessor specifically, due to non-standard location of OpenMP.
This should now unblock the CI. |
The Homebrew package for libomp 15 is now keg-only, meaning that we now need to specify extra build flags.
This PR attempts to fix building XGBoost with libomp 15.
This should be also back-ported to the R package.