http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3/builds/1739 http://bb.pgr.jp/builders/cmake-clang-x86_64-linux/builds/29232
I am still confused. FYI, g++-4.8.1 is not affected. 2014-10-31 9:50 GMT+09:00 Richard Smith <[email protected]>: > On Thu, Oct 30, 2014 at 5:30 PM, NAKAMURA Takumi <[email protected]> > wrote: >> >> Author: chapuni >> Date: Thu Oct 30 19:30:37 2014 >> New Revision: 220940 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=220940&view=rev >> Log: >> ASTDumper.cpp: Appease g++, for now. > > > Thanks! > >> >> Modified: >> cfe/trunk/lib/AST/ASTDumper.cpp >> >> Modified: cfe/trunk/lib/AST/ASTDumper.cpp >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ASTDumper.cpp?rev=220940&r1=220939&r2=220940&view=diff >> >> ============================================================================== >> --- cfe/trunk/lib/AST/ASTDumper.cpp (original) >> +++ cfe/trunk/lib/AST/ASTDumper.cpp Thu Oct 30 19:30:37 2014 >> @@ -152,8 +152,8 @@ namespace { >> OS << '\n'; >> ColorScope Color(*this, IndentColor); >> OS << Prefix << (isLastChild ? '`' : '|') << '-'; >> - Prefix.push_back(isLastChild ? ' ' : '|'); >> - Prefix.push_back(' '); >> + this->Prefix.push_back(isLastChild ? ' ' : '|'); >> + this->Prefix.push_back(' '); >> } >> >> FirstChild = true; >> @@ -166,11 +166,11 @@ namespace { >> // Dump those ones out now. >> while (Depth < Pending.size()) { >> Pending.back()(true); > > > Do you need to also change these references to Pending... > >> >> - Pending.pop_back(); >> + this->Pending.pop_back(); >> } >> >> // Restore the old prefix. >> - Prefix.resize(Prefix.size() - 2); >> + this->Prefix.resize(Prefix.size() - 2); > > > ... and the second reference to Prefix here? > > What error does g++ give here? _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
