-
Notifications
You must be signed in to change notification settings - Fork 14.3k
This commit fixes the build failure due to OMP_TRAIT_PROPERTY macro r… #126222
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
Conversation
@llvm/pr-subscribers-flang-openmp Author: None (Ritanya-B-Bharadwaj) Changes…edefinition - #126043 Full diff: https://github.com/llvm/llvm-project/pull/126222.diff 1 Files Affected:
diff --git a/llvm/lib/Frontend/OpenMP/OMPContext.cpp b/llvm/lib/Frontend/OpenMP/OMPContext.cpp
index cca5f8806f9e5cb..5e13da172d67777 100644
--- a/llvm/lib/Frontend/OpenMP/OMPContext.cpp
+++ b/llvm/lib/Frontend/OpenMP/OMPContext.cpp
@@ -67,7 +67,7 @@ OMPContext::OMPContext(bool IsDeviceCompilation, Triple TargetTriple,
TargetOffloadTriple.getArch() == Triple::x86_64) \
ActiveTraits.set(unsigned(TraitProperty::Enum)); \
}
-#undef OMP_TRAIT_PROPERTY
+#include "llvm/Frontend/OpenMP/OMPKinds.def"
} else {
// Add the appropriate device kind trait based on the triple and the
// IsDeviceCompilation flag.
|
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'm going to assume that someone forgot to include the Kinds file for this.
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/190/builds/14349 Here is the relevant piece of the build log for the reference
|
…edefinition - #126043