[PATCH] Introducing initial UseAuto transform

Dmitri Gribenko gribozavr at gmail.com
Tue Feb 12 07:49:12 PST 2013



================
Comment at: cpp11-migrate/UseAuto/UseAutoActions.cpp:44-47
@@ +43,6 @@
+
+    E2 = E->IgnoreConversionOperator();
+    if (E2 != E) {
+      E = E2;
+      continue;
+    }
----------------
I think this might lead to unexpected results.  Conversion operators can do nontrivial work and skipping them could create a correctness problem.


http://llvm-reviews.chandlerc.com/D392



More information about the cfe-commits mailing list