This revision was automatically updated to reflect the committed changes.
Closed by commit rL311241: [clang-diff] Add HTML side-by-side diff output
(authored by krobelus).
Changed prior to commit:
https://reviews.llvm.org/D36182?vs=110553&id=111829#toc
Repository:
rL LLVM
https://reviews.ll
arphaman added a comment.
Awesome, thanks! LGTM.
https://reviews.llvm.org/D36182
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
johannes updated this revision to Diff 110553.
johannes edited the summary of this revision.
johannes added a comment.
change tests
https://reviews.llvm.org/D36182
Files:
test/Tooling/Inputs/clang-diff-basic-src.cpp
test/Tooling/clang-diff-basic.cpp
test/Tooling/clang-diff-html.test
too
arphaman added inline comments.
Comment at: test/Tooling/clang-diff-html.py:1
+# RUN: clang-diff %S/Inputs/clang-diff-basic-src.cpp %S/clang-diff-basic.cpp
-html -- | %python %s > %t.filecheck
+# RUN: clang-diff %S/Inputs/clang-diff-basic-src.cpp %S/clang-diff-basic.cpp
-dump-m
johannes updated this revision to Diff 109539.
johannes added a comment.
make test work with python 2
https://reviews.llvm.org/D36182
Files:
test/Tooling/Inputs/clang-diff-basic-src.cpp
test/Tooling/clang-diff-basic.cpp
test/Tooling/clang-diff-html.py
tools/clang-diff/CMakeLists.txt
t
johannes updated this revision to Diff 109421.
johannes added a comment.
-
https://reviews.llvm.org/D36182
Files:
test/Tooling/Inputs/clang-diff-basic-src.cpp
test/Tooling/clang-diff-basic.cpp
test/Tooling/clang-diff-html.py
tools/clang-diff/CMakeLists.txt
tools/clang-diff/ClangDiff.c
