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

Side by Side Diff: unittest/IceELFSectionTest.cpp

Issue 1221643012: Subzero: Add -Wshadow to the build. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Code review changes Created 5 years, 5 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 unified diff | Download patch
OLDNEW
1 //===- unittest/IceELFSectionTest.cpp - ELF Section unit tests ------------===// 1 //===- unittest/IceELFSectionTest.cpp - ELF Section unit tests ------------===//
2 // 2 //
3 // The Subzero Code Generator 3 // The Subzero Code Generator
4 // 4 //
5 // This file is distributed under the University of Illinois Open Source 5 // This file is distributed under the University of Illinois Open Source
6 // License. See LICENSE.TXT for details. 6 // License. See LICENSE.TXT for details.
7 // 7 //
8 //===----------------------------------------------------------------------===// 8 //===----------------------------------------------------------------------===//
9 9
10 #include <algorithm> 10 #include <algorithm>
11 11
12 #pragma clang diagnostic push
13 #pragma clang diagnostic ignored "-Wunused-parameter"
14 #pragma clang diagnostic ignored "-Wshadow"
12 #include "gtest/gtest.h" 15 #include "gtest/gtest.h"
16 #pragma clang diagnostic pop
13 17
14 #include "IceDefs.h" 18 #include "IceDefs.h"
15 #include "IceELFSection.h" 19 #include "IceELFSection.h"
16 20
17 namespace Ice { 21 namespace Ice {
18 namespace { 22 namespace {
19 23
20 // Test string table layout for various permutations. Test that pop, 24 // Test string table layout for various permutations. Test that pop,
21 // lollipop, and lipop are able to share data, while the other strings do not. 25 // lollipop, and lipop are able to share data, while the other strings do not.
22 void CheckStringTablePermLayout(const ELFStringTableSection &Strtab) { 26 void CheckStringTablePermLayout(const ELFStringTableSection &Strtab) {
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 Strtab.add("lollipop"); 132 Strtab.add("lollipop");
129 Strtab.add("pop"); 133 Strtab.add("pop");
130 Strtab.add("unpop"); 134 Strtab.add("unpop");
131 135
132 Strtab.doLayout(); 136 Strtab.doLayout();
133 CheckStringTablePermLayout(Strtab); 137 CheckStringTablePermLayout(Strtab);
134 } 138 }
135 139
136 } // end of anonymous namespace 140 } // end of anonymous namespace
137 } // end of namespace Ice 141 } // end of namespace Ice
OLDNEW
« src/IceInst.h ('K') | « unittest/BitcodeMunge.h ('k') | unittest/IceParseInstsTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698