tstellar created this revision.
tstellar added reviewers: Pierre, Anastasia.
Herald added a subscriber: yaxunl.
Herald added a project: clang.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D64608
Files:
clang/lib/Sema/SemaLookup.cpp
clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp
Index: clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp
===================================================================
--- clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp
+++ clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp
@@ -207,7 +207,7 @@
}
void BuiltinNameEmitter::EmitSignatureTable() {
- OS << "OpenCLType OpenCLSignature[] = {\n";
+ OS << "static const OpenCLType OpenCLSignature[] = {\n";
for (auto &P : SignatureSet) {
OS << "// " << P.second << "\n";
for (Record *R : P.first) {
@@ -222,7 +222,7 @@
}
void BuiltinNameEmitter::EmitBuiltinTable() {
- OS << "OpenCLBuiltinDecl OpenCLBuiltins[] = {\n";
+ OS << "static const OpenCLBuiltinDecl OpenCLBuiltins[] = {\n";
for (auto &i : OverloadInfo) {
StringRef Name = i.first;
OS << "// " << Name << "\n";
@@ -255,7 +255,7 @@
OS << R"(
// Return 0 if name is not a recognized OpenCL builtin, or an index
// into a table of declarations if it is an OpenCL builtin.
-std::pair<unsigned, unsigned> isOpenCLBuiltin(llvm::StringRef name) {
+static std::pair<unsigned, unsigned> isOpenCLBuiltin(llvm::StringRef name) {
)";
Index: clang/lib/Sema/SemaLookup.cpp
===================================================================
--- clang/lib/Sema/SemaLookup.cpp
+++ clang/lib/Sema/SemaLookup.cpp
@@ -688,7 +688,7 @@
unsigned Len) {
for (unsigned i = 0; i < Len; ++i) {
- OpenCLBuiltinDecl &Decl = OpenCLBuiltins[Index - 1 + i];
+ const OpenCLBuiltinDecl &Decl = OpenCLBuiltins[Index - 1 + i];
ASTContext &Context = S.Context;
// Ignore this BIF if the version is incorrect.
Index: clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp
===================================================================
--- clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp
+++ clang/utils/TableGen/ClangOpenCLBuiltinEmitter.cpp
@@ -207,7 +207,7 @@
}
void BuiltinNameEmitter::EmitSignatureTable() {
- OS << "OpenCLType OpenCLSignature[] = {\n";
+ OS << "static const OpenCLType OpenCLSignature[] = {\n";
for (auto &P : SignatureSet) {
OS << "// " << P.second << "\n";
for (Record *R : P.first) {
@@ -222,7 +222,7 @@
}
void BuiltinNameEmitter::EmitBuiltinTable() {
- OS << "OpenCLBuiltinDecl OpenCLBuiltins[] = {\n";
+ OS << "static const OpenCLBuiltinDecl OpenCLBuiltins[] = {\n";
for (auto &i : OverloadInfo) {
StringRef Name = i.first;
OS << "// " << Name << "\n";
@@ -255,7 +255,7 @@
OS << R"(
// Return 0 if name is not a recognized OpenCL builtin, or an index
// into a table of declarations if it is an OpenCL builtin.
-std::pair<unsigned, unsigned> isOpenCLBuiltin(llvm::StringRef name) {
+static std::pair<unsigned, unsigned> isOpenCLBuiltin(llvm::StringRef name) {
)";
Index: clang/lib/Sema/SemaLookup.cpp
===================================================================
--- clang/lib/Sema/SemaLookup.cpp
+++ clang/lib/Sema/SemaLookup.cpp
@@ -688,7 +688,7 @@
unsigned Len) {
for (unsigned i = 0; i < Len; ++i) {
- OpenCLBuiltinDecl &Decl = OpenCLBuiltins[Index - 1 + i];
+ const OpenCLBuiltinDecl &Decl = OpenCLBuiltins[Index - 1 + i];
ASTContext &Context = S.Context;
// Ignore this BIF if the version is incorrect.
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
