[PATCH] Don't replace macro usage if macro body has NULL

Manuel Klimek klimek at google.com
Tue Mar 19 15:27:05 PDT 2013



================
Comment at: cpp11-migrate/UseNullptr/NullptrActions.cpp:125
@@ +124,3 @@
+    // coming from a macro called NULL.
+    if (SM.isMacroArgExpansion(StartLoc) && SM.isMacroArgExpansion(EndLoc)) {
+      StartLoc = SM.getFileLoc(StartLoc);
----------------
I assume we don't care that those can both be true, but start and end can still be produced from different macro expansions? Or do we check that somewhere else?


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



More information about the cfe-commits mailing list