Re: [clang-tools-extra] r295482 - [clang-tidy] Add -path option to clang-tidy-diff.py

2017-02-17 Thread Ehsan Akhgari via cfe-commits
What's strange is that I copied this pattern from clang-tidy-run-with-database.cpp, and I'm not sure why it would work there and not here.. On Fri, Feb 17, 2017 at 8:16 PM, Ehsan Akhgari wrote: > Hi Douglas, > > Sorry about this. It seems to me that the reason for this test failure is > the sla

Re: [clang-tools-extra] r295482 - [clang-tidy] Add -path option to clang-tidy-diff.py

2017-02-17 Thread Ehsan Akhgari via cfe-commits
Hi Douglas, Sorry about this. It seems to me that the reason for this test failure is the slashes that appear in the -path argument in the log: "-path" "C:\Buildbot\Slave\llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast\llvm.obj\tools\clang\tools\extra\test\clang-tidy\Output/compilation-databas

RE: [clang-tools-extra] r295482 - [clang-tidy] Add -path option to clang-tidy-diff.py

2017-02-17 Thread Yung, Douglas via cfe-commits
Hi Ehsan, Your commit has caused the PS4 Windows bot to go red. Can you take a look? http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/5661 $ "FileCheck" "-check-prefix=CHECK" "C:\Buildbot\Slave\llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast\llvm.src\to