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

[Alex Lorenz via Phabricator via cfe-commits]

arphaman added inline comments.


================
Comment at: test/Tooling/clang-diff-json.cpp:1
+// RUN: clang-diff -ast-dump %s -- | python -m json.tool | FileCheck %s
+
----------------
arphaman wrote:
> johannes wrote:
> > 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
> Yeah, 2.7 support is still a requirement.
It might not be in the lit.cfg for Clang, let me check.


https://reviews.llvm.org/D36178