From: Benjamin Kramer <benny.kra@googlemail.com>
Date: Sat, 18 Jul 2015 14:35:41 +0000 (+0000)
Subject: [clang-modernize] Don't rely on iterator auto-dereferencing.
X-Git-Tag: studio-1.4~2164
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3204a105e31987f7dc6b8cf61ac7a26be5cf6213;p=platform%2Fupstream%2Fllvm.git

[clang-modernize] Don't rely on iterator auto-dereferencing.

llvm-svn: 242607
---

diff --git a/clang-tools-extra/clang-modernize/UseAuto/UseAutoActions.cpp b/clang-tools-extra/clang-modernize/UseAuto/UseAutoActions.cpp
index 2a8d5c5..c4c3ce6 100644
--- a/clang-tools-extra/clang-modernize/UseAuto/UseAutoActions.cpp
+++ b/clang-tools-extra/clang-modernize/UseAuto/UseAutoActions.cpp
@@ -46,7 +46,7 @@ void IteratorReplacer::run(const MatchFinder::MatchResult &Result) {
            "Expected constructor with single argument");
 
     // Drill down to the as-written initializer.
-    const Expr *E = Construct->arg_begin()->IgnoreParenImpCasts();
+    const Expr *E = (*Construct->arg_begin())->IgnoreParenImpCasts();
     if (E != E->IgnoreConversionOperator())
       // We hit a conversion operator. Early-out now as they imply an implicit
       // conversion from a different type. Could also mean an explicit