arphaman added a comment.

LGTM with one inline comment below:



================
Comment at: utils/TableGen/TableGen.cpp:151
         clEnumValN(GenOptDocs, "gen-opt-docs", "Generate option 
documentation"),
+        clEnumValN(GenDataCollectors, "gen-clang-data-collectors", "Generate 
data collectors for AST nodes"),
         clEnumValN(GenTestPragmaAttributeSupportedAttributes,
----------------
Looks like an 80 column violation, please reformat this line.


https://reviews.llvm.org/D37383



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to