[PATCH] D35282: [Dominators] Split SemiNCA into smaller functions

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 08:19:50 PDT 2017


davide added inline comments.


================
Comment at: include/llvm/Support/GenericDomTreeConstruction.h:34
+#define DTB_DEBUG_TYPE "dom-tree-builder"
+#define DTB_DEBUG(Arg) DEBUG_WITH_TYPE(DTB_DEBUG_TYPE, Arg)
+
----------------
kuhar wrote:
> dberlin wrote:
> > Why doesn't #define DEBUG_TYPE work?
> The macro can clash with DEBUG_TYPE in a file including it. Is it better change it to DEBUG_TYPE and undef it at the bottom of this header?
This seems to be symptom of a more general problem, there shouldn't be clashes. Which file is actually defining another `dom-tree-builder` ?


https://reviews.llvm.org/D35282





More information about the llvm-commits mailing list