[PATCH] D36182: [clang-diff] Add HTML side-by-side diff output

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 9 04:36:48 PDT 2017


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-matches -- | FileCheck %t.filecheck
----------------
Why did you decide to use a python script for this? Can you just file check the HTML output directly, or do you think this is better?


https://reviews.llvm.org/D36182





More information about the cfe-commits mailing list