[PATCH] D36186: [clang-diff] Improve and test getNodeValue

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 11 08:35:26 PDT 2017


arphaman added inline comments.


================
Comment at: lib/Tooling/ASTDiff/ASTDiff.cpp:428
+    Value += getRelativeName(V) + "(" + V->getType().getAsString(TypePP) + ")";
+    if (auto *C = dyn_cast<CXXConstructorDecl>(D)) {
+      for (auto *Init : C->inits()) {
----------------
It looks like you're doing a lot of work for constructors here, but I don't seem them tested in this patch.


https://reviews.llvm.org/D36186





More information about the cfe-commits mailing list