r205506 - [OPENMP] Small update for C++11

[Alexey Bataev]

Hi Richard,
Thanks for review.
But I made this changes just to be fully compatible with the previous 
one made by Aaron Ballman (see changes in revision 203937).

Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team
Intel Corp.

3 Апрель 2014 г. 22:04:27, Richard Smith писал:
> On Wed, Apr 2, 2014 at 7:55 PM, Alexey Bataev <a.bataev at hotmail.com
> <mailto:a.bataev at hotmail.com>> wrote:
>
>     Author: abataev
>     Date: Wed Apr  2 21:55:46 2014
>     New Revision: 205506
>
>     URL: http://llvm.org/viewvc/llvm-project?rev=205506&view=rev
>     Log:
>     [OPENMP] Small update for C++11
>
>     Modified:
>         cfe/trunk/lib/Sema/TreeTransform.h
>         cfe/trunk/lib/Serialization/ASTWriterStmt.cpp
>
>     Modified: cfe/trunk/lib/Sema/TreeTransform.h
>     URL:
>     http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/TreeTransform.h?rev=205506&r1=205505&r2=205506&view=diff
>     ==============================================================================
>     --- cfe/trunk/lib/Sema/TreeTransform.h (original)
>     +++ cfe/trunk/lib/Sema/TreeTransform.h Wed Apr  2 21:55:46 2014
>     @@ -6439,10 +6439,8 @@ OMPClause *
>      TreeTransform<Derived>::TransformOMPCopyinClause(OMPCopyinClause
>     *C) {
>        llvm::SmallVector<Expr *, 16> Vars;
>        Vars.reserve(C->varlist_size());
>     -  for (OMPCopyinClause::varlist_iterator I = C->varlist_begin(),
>     -                                         E = C->varlist_end();
>     -       I != E; ++I) {
>     -    ExprResult EVar = getDerived().TransformExpr(cast<Expr>(*I));
>     +  for (auto *I : C->varlists()) {
>
>
> This is not an iterator, so should not be named I.
>
>     +    ExprResult EVar = getDerived().TransformExpr(cast<Expr>(I));
>          if (EVar.isInvalid())
>            return 0;
>          Vars.push_back(EVar.take());
>
>     Modified: cfe/trunk/lib/Serialization/ASTWriterStmt.cpp
>     URL:
>     http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Serialization/ASTWriterStmt.cpp?rev=205506&r1=205505&r2=205506&view=diff
>     ==============================================================================
>     --- cfe/trunk/lib/Serialization/ASTWriterStmt.cpp (original)
>     +++ cfe/trunk/lib/Serialization/ASTWriterStmt.cpp Wed Apr  2
>     21:55:46 2014
>     @@ -1719,10 +1719,8 @@ void OMPClauseWriter::VisitOMPSharedClau
>      void OMPClauseWriter::VisitOMPCopyinClause(OMPCopyinClause *C) {
>        Record.push_back(C->varlist_size());
>        Writer->Writer.AddSourceLocation(C->getLParenLoc(), Record);
>     -  for (OMPCopyinClause::varlist_iterator I = C->varlist_begin(),
>     -                                         E = C->varlist_end();
>     -       I != E; ++I)
>     -    Writer->Writer.AddStmt(*I);
>     +  for (auto *I : C->varlists())
>
>
> Likewise.
>
>     +    Writer->Writer.AddStmt(I);
>      }
>
>      //===----------------------------------------------------------------------===//
>
>
>     _______________________________________________
>     cfe-commits mailing list
>     cfe-commits at cs.uiuc.edu <mailto:cfe-commits at cs.uiuc.edu>
>     http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>