================
Comment at: cpp11-migrate/UseNullptr/NullptrActions.cpp:113
@@ -112,3 +112,3 @@
   if (Cast) {
-    SourceLocation StartLoc = Cast->getLocStart();
-    SourceLocation EndLoc = Cast->getLocEnd();
+    const Expr *E = Cast->IgnoreParenImpCasts();
+
----------------
Does this need to happen for explicit casts as well?

================
Comment at: test/cpp11-migrate/UseNullptr/basic.cpp:222
@@ +221,2 @@
+  // CHECK: return((((nullptr))));
+}
----------------
As per above, add a test case like

return ((static_cast<void*>(0)));

or similar


http://llvm-reviews.chandlerc.com/D496
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to