[PATCH] D29923: Send UndefMacroDirective to MacroDefined callback

Manman Ren via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 9 10:44:29 PST 2017


manmanren added a comment.

Please update the patch with context: http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface

Thanks,
Manman



================
Comment at: unittests/Basic/SourceManagerTest.cpp:251
   std::string Name;
-  bool isDefinition; // if false, it is expansion.
-  
-  MacroAction(SourceLocation Loc, StringRef Name, bool isDefinition)
-    : Loc(Loc), Name(Name), isDefinition(isDefinition) { }
+  int Kind; // 0 expansion, 1 definition, 2 undefinition
+
----------------
Can we make this an enum?


https://reviews.llvm.org/D29923





More information about the cfe-commits mailing list