Subzero: Enable the C++11 build.

src/PNaClTranslator.cpp

Index: src/PNaClTranslator.cpp
diff --git a/src/PNaClTranslator.cpp b/src/PNaClTranslator.cpp
index 35108ccc560410fcb01575c3b3b65144b50c883e..eeecdda2ed5d539416d1736ed92316a9247fb5c3 100644
--- a/src/PNaClTranslator.cpp
+++ b/src/PNaClTranslator.cpp
@@ -66,8 +66,7 @@ public:
     setErrStream(Translator.getContext()->getStrDump());
   }
 
-  virtual ~TopLevelParser() {}
-  LLVM_OVERRIDE;

[JF, 2014/09/25 20:57:07]

This should be:

~TopLevelParser() override {}


No virtual.

[Jim Stichnoth, 2014/09/26 00:11:17]

Done.

+  virtual ~TopLevelParser() LLVM_OVERRIDE {}
 
   Ice::Translator &getTranslator() { return Translator; }
 
@@ -2287,7 +2286,7 @@ void ModuleValuesymtabParser::setBbName(uint64_t Index, StringType &Name) {
   Error(StrBuf.str());
 }
 
-bool ModuleParser::ParseBlock(unsigned BlockID) LLVM_OVERRIDE {

[JF, 2014/09/25 20:57:07]

This should be override. Or does it not override (and is that intentional?).

[Jim Stichnoth, 2014/09/26 00:11:17]

The override is already provided in the class definition above.  Putting it on
the method definition outside the class definition seems to be an error.

+bool ModuleParser::ParseBlock(unsigned BlockID) {
   switch (BlockID) {
   case naclbitc::BLOCKINFO_BLOCK_ID:
     return NaClBitcodeParser::ParseBlock(BlockID);