Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(23)

Unified Diff: src/IceGlobalInits.h

Issue 1221643012: Subzero: Add -Wshadow to the build. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Change the previous underscore naming style Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/IceGlobalInits.h
diff --git a/src/IceGlobalInits.h b/src/IceGlobalInits.h
index 08f4fe3afcc24e8c4ba00b226d094fe8a1fdb544..71efd0ba6dfd4f22b260a36bef06a1bb4ecdd10b 100644
--- a/src/IceGlobalInits.h
+++ b/src/IceGlobalInits.h
@@ -23,6 +23,7 @@
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wunused-parameter"
+#pragma clang diagnostic ignored "-Wshadow"
#include "llvm/Bitcode/NaCl/NaClBitcodeParser.h" // for NaClBitcodeRecord.
#include "llvm/IR/CallingConv.h"
#include "llvm/IR/GlobalValue.h" // for GlobalValue::LinkageTypes.
@@ -86,9 +87,9 @@ public:
}
protected:
- GlobalDeclaration(GlobalDeclarationKind Kind,
- llvm::GlobalValue::LinkageTypes Linkage)
- : Kind(Kind), Linkage(Linkage) {}
+ GlobalDeclaration(GlobalDeclarationKind MyKind,
+ llvm::GlobalValue::LinkageTypes MyLinkage)
+ : Kind(MyKind), Linkage(MyLinkage) {}
const GlobalDeclarationKind Kind;
IceString Name;
@@ -127,11 +128,12 @@ private:
llvm::CallingConv::ID CallingConv;
bool IsProto;
- FunctionDeclaration(const FuncSigType &Signature,
- llvm::CallingConv::ID CallingConv,
- llvm::GlobalValue::LinkageTypes Linkage, bool IsProto)
+ FunctionDeclaration(const FuncSigType &MySignature,
+ llvm::CallingConv::ID MyCallingConv,
+ llvm::GlobalValue::LinkageTypes Linkage, bool MyIsProto)
: GlobalDeclaration(FunctionDeclarationKind, Linkage),
- Signature(Signature), CallingConv(CallingConv), IsProto(IsProto) {}
+ Signature(MySignature), CallingConv(MyCallingConv), IsProto(MyIsProto) {
+ }
};
/// Models a global variable declaration, and its initializers.
@@ -163,7 +165,7 @@ public:
virtual void dumpType(Ostream &Stream) const;
protected:
- explicit Initializer(InitializerKind Kind) : Kind(Kind) {}
+ explicit Initializer(InitializerKind MyKind) : Kind(MyKind) {}
private:
const InitializerKind Kind;
@@ -227,8 +229,8 @@ public:
private:
ENABLE_MAKE_UNIQUE;
- explicit ZeroInitializer(SizeT Size)
- : Initializer(ZeroInitializerKind), Size(Size) {}
+ explicit ZeroInitializer(SizeT MySize)
+ : Initializer(ZeroInitializerKind), Size(MySize) {}
// The number of bytes to be zero initialized.
SizeT Size;
@@ -257,9 +259,10 @@ public:
private:
ENABLE_MAKE_UNIQUE;
- RelocInitializer(const GlobalDeclaration *Declaration, RelocOffsetT Offset)
- : Initializer(RelocInitializerKind), Declaration(Declaration),
- Offset(Offset) {} // The global declaration used in the relocation.
+ RelocInitializer(const GlobalDeclaration *MyDeclaration,
+ RelocOffsetT MyOffset)
+ : Initializer(RelocInitializerKind), Declaration(MyDeclaration),
+ Offset(MyOffset) {} // The global declaration used in the relocation.
const GlobalDeclaration *Declaration;
// The offset to add to the relocation.

Powered by Google App Engine
This is Rietveld 408576698