[PATCH] D25571: Add and use isDiscardableGVALinkage function.
Justin Lebar via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 13 11:42:34 PDT 2016
jlebar created this revision.
jlebar added a reviewer: rnk.
jlebar added a subscriber: cfe-commits.
https://reviews.llvm.org/D25571
Files:
clang/include/clang/Basic/Linkage.h
clang/include/clang/Sema/Sema.h
clang/lib/AST/ASTContext.cpp
clang/lib/Sema/SemaCUDA.cpp
Index: clang/lib/Sema/SemaCUDA.cpp
===================================================================
--- clang/lib/Sema/SemaCUDA.cpp
+++ clang/lib/Sema/SemaCUDA.cpp
@@ -543,7 +543,7 @@
return false;
// Externally-visible and similar functions are always emitted.
- if (S.getASTContext().GetGVALinkageForFunction(FD) > GVA_DiscardableODR)
+ if (!isDiscardableGVALinkage(S.getASTContext().GetGVALinkageForFunction(FD)))
return true;
// Otherwise, the function is known-emitted if it's in our set of
Index: clang/lib/AST/ASTContext.cpp
===================================================================
--- clang/lib/AST/ASTContext.cpp
+++ clang/lib/AST/ASTContext.cpp
@@ -8802,15 +8802,10 @@
}
}
- GVALinkage Linkage = GetGVALinkageForFunction(FD);
-
// static, static inline, always_inline, and extern inline functions can
// always be deferred. Normal inline functions can be deferred in C99/C++.
// Implicit template instantiations can also be deferred in C++.
- if (Linkage == GVA_Internal || Linkage == GVA_AvailableExternally ||
- Linkage == GVA_DiscardableODR)
- return false;
- return true;
+ return !isDiscardableGVALinkage(GetGVALinkageForFunction(FD));
}
const VarDecl *VD = cast<VarDecl>(D);
@@ -8821,9 +8816,7 @@
return false;
// Variables that can be needed in other TUs are required.
- GVALinkage L = GetGVALinkageForVariable(VD);
- if (L != GVA_Internal && L != GVA_AvailableExternally &&
- L != GVA_DiscardableODR)
+ if (!isDiscardableGVALinkage(GetGVALinkageForVariable(VD)))
return true;
// Variables that have destruction with side-effects are required.
Index: clang/include/clang/Sema/Sema.h
===================================================================
--- clang/include/clang/Sema/Sema.h
+++ clang/include/clang/Sema/Sema.h
@@ -9241,7 +9241,7 @@
/// discover that a function is known-emitted, we remove it and everything it
/// transitively calls from this set and add those functions to
/// CUDAKnownEmittedFns.
- llvm::DenseMap<FunctionDecl *, llvm::DenseSet<FunctionDecl *>> CUDACallGraph;
+ llvm::DenseMap<FunctionDecl *, llvm::SetVector<FunctionDecl *>> CUDACallGraph;
/// Diagnostic builder for CUDA errors which may or may not be deferred.
///
Index: clang/include/clang/Basic/Linkage.h
===================================================================
--- clang/include/clang/Basic/Linkage.h
+++ clang/include/clang/Basic/Linkage.h
@@ -69,6 +69,10 @@
GVA_StrongODR
};
+inline bool isDiscardableGVALinkage(GVALinkage L) {
+ return L <= GVA_DiscardableODR;
+}
+
inline bool isExternallyVisible(Linkage L) {
return L == ExternalLinkage || L == VisibleNoLinkage;
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25571.74562.patch
Type: text/x-patch
Size: 2746 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161013/b3246ec5/attachment.bin>
More information about the cfe-commits
mailing list