[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