arphaman accepted this revision.
arphaman added a comment.
This revision is now accepted and ready to land.
LGTM with one request below:
================
Comment at: tools/clang-diff/ClangDiff.cpp:319
+ "A Binary operator is supposed to have two arguments.");
+ for (int I : {1, 0, 2})
+ Offset = printHtmlForNode(OS, Diff, Tree, IsLeft, Children[I], Offset);
----------------
Please add a short comment that describes why this out-of-order traversal is
required
https://reviews.llvm.org/D37004
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
