[PATCH] D36178: [clang-diff] Move the JSON export function to clang-diff

[Johannes Altmanninger via Phabricator via cfe-commits]

johannes added inline comments.


================
Comment at: test/Tooling/clang-diff-json.cpp:1
+// RUN: clang-diff -ast-dump %s -- | python -m json.tool | FileCheck %s
+
----------------
johannes wrote:
> arphaman wrote:
> > I think you have to use `%python` instead of `python`, like LLVM tests do.
> ok
> So I have to use python 2.7? (for the test in https://reviews.llvm.org/D36182)
It looks like %python is expanded in the wrong way.
I get the same as when using %Sython, the directory of the test case is substituted


https://reviews.llvm.org/D36178