[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 (file)
--- 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