Re: [clang-tools-extra] r333994 - [clang-tidy] fix broken test (no compile command) from r331763

[Sam McCall via cfe-commits]

Ah, I've been there. No worries!

On Tue, Jun 5, 2018 at 1:08 PM Roman Lebedev  wrote:

> Ah, hm, yes, that was a further local problem in
> https://reviews.llvm.org/D46602
> And i didn't double-check on master initially, because that causes all
> the checks to be rebuilt :/
>
> So thanks for the fix!
>
> Roman.
>
>
> On Tue, Jun 5, 2018 at 1:40 PM, Sam McCall  wrote:
> > Hi Roman,
> >
> > Definitely something different in our configs, but it wasn't just me - a
> > couple of colleagues have mentioned those tests as being broken for a
> while!
> >
> > The problem is the lack of compilation database, so I can imagine it can
> > depend on filesystem layout, e.g. whether you have a separate
> build/source
> > directory. (Potentially the CDB from llvm itself may be picked up or not)
> >
> > I can't tell from your lit output whether clang-tidy or FileCheck failed,
> > what does this command return for you?
> >
> > bin/clang-tidy -enable-check-profile
> -checks='-*,readability-function-size'
> > ../src/llvm/tools/clang/tools/extra/test/clang-tid
> > y/clang-tidy-enable-check-profile-one-tu.cpp --
> >
> > (adjust path as needed)
> >
> > On Tue, Jun 5, 2018 at 12:30 PM Roman Lebedev 
> wrote:
> >>
> >> This is strange.
> >>
> >> First, i'm pretty sure the test worked for me before.
> >> Second, this commit actually *breaks* those two tests for me:
> >>
> >> $ ninja check-clang-tools
> >> [0/1] Running the Clang extra tools' regression tests
> >> FAIL: Clang Tools ::
> >> clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp (123 of 867)
> >>  TEST 'Clang Tools ::
> >> clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp' FAILED
> >> 
> >> Script:
> >> --
> >> : 'RUN: at line 1';   clang-tidy -enable-check-profile
> >> -checks='-*,readability-function-size'
> >>
> >>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> >> -- 2>&1 | FileCheck --match-full-lines
> >> -implicit-check-not='{{warning:|error:}}' /build/clang
> >> -tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> >> --
> >> Exit Code: 2
> >>
> >> Command Output (stderr):
> >> --
> >> FileCheck error: '-' is empty.
> >> FileCheck command line:  FileCheck --match-full-lines
> >> -implicit-check-not={{warning:|error:}}
> >>
> >>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> >>
> >> --
> >>
> >> 
> >> FAIL: Clang Tools ::
> >> clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp (125 of 867)
> >>  TEST 'Clang Tools ::
> >> clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp' FAILED
> >> 
> >> Script:
> >> --
> >> : 'RUN: at line 1';   clang-tidy -enable-check-profile
> >> -checks='-*,readability-function-size'
> >>
> >>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> >>
> >>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> >> -- 2>&1 |
> >> FileCheck --match-full-lines -implicit-check-not='{{warning:|error:}}'
> >>
> >>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> >> --
> >> Exit Code: 2
> >>
> >> Command Output (stderr):
> >> --
> >> FileCheck error: '-' is empty.
> >> FileCheck command line:  FileCheck --match-full-lines
> >> -implicit-check-not={{warning:|error:}}
> >>
> >>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> >>
> >> --
> >>
> >> Roman.
> >>
> >> On Tue, Jun 5, 2018 at 12:42 PM, Sam McCall via cfe-commits
> >>  wrote:
> >> > Author: sammccall
> >> > Date: Tue Jun  5 02:42:06 2018
> >> > New Revision: 333994
> >> >
> >> > URL: http://llvm.org/viewvc/llvm-project?rev=333994=rev
> >> > Log:
> >> > [clang-tidy] fix broken test (no compile command) from r331763
> >> >
> >> > Modified:
> >> >
> >> >
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> >> >
> >> >
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> >> >
> >> > Modified:
> >> >
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> >> > URL:
> >> >
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp?rev=333994=333993=333994=diff
> >> >
> >> >
> ==
> >> > ---
> >> >
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> >> > (original)
> >> > +++
> >> >
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> >> > Tue Jun  5 02:42:06 2018
> >> > @@ -1,4 +1,4 @@
> >> > -// RUN: clang-tidy -enable-check-profile
> >> > -checks='-*,readability-function-size' %s 2>&1 | FileCheck
> >> > --match-full-lines -implicit-check-not='{{warning:|error:}}' %s
> >> > +// RUN: clang-tidy -enable-check-profile
> >> > 

Re: [clang-tools-extra] r333994 - [clang-tidy] fix broken test (no compile command) from r331763

[Roman Lebedev via cfe-commits]

Ah, hm, yes, that was a further local problem in https://reviews.llvm.org/D46602
And i didn't double-check on master initially, because that causes all
the checks to be rebuilt :/

So thanks for the fix!

Roman.


On Tue, Jun 5, 2018 at 1:40 PM, Sam McCall  wrote:
> Hi Roman,
>
> Definitely something different in our configs, but it wasn't just me - a
> couple of colleagues have mentioned those tests as being broken for a while!
>
> The problem is the lack of compilation database, so I can imagine it can
> depend on filesystem layout, e.g. whether you have a separate build/source
> directory. (Potentially the CDB from llvm itself may be picked up or not)
>
> I can't tell from your lit output whether clang-tidy or FileCheck failed,
> what does this command return for you?
>
> bin/clang-tidy -enable-check-profile -checks='-*,readability-function-size'
> ../src/llvm/tools/clang/tools/extra/test/clang-tid
> y/clang-tidy-enable-check-profile-one-tu.cpp --
>
> (adjust path as needed)
>
> On Tue, Jun 5, 2018 at 12:30 PM Roman Lebedev  wrote:
>>
>> This is strange.
>>
>> First, i'm pretty sure the test worked for me before.
>> Second, this commit actually *breaks* those two tests for me:
>>
>> $ ninja check-clang-tools
>> [0/1] Running the Clang extra tools' regression tests
>> FAIL: Clang Tools ::
>> clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp (123 of 867)
>>  TEST 'Clang Tools ::
>> clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp' FAILED
>> 
>> Script:
>> --
>> : 'RUN: at line 1';   clang-tidy -enable-check-profile
>> -checks='-*,readability-function-size'
>>
>> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>> -- 2>&1 | FileCheck --match-full-lines
>> -implicit-check-not='{{warning:|error:}}' /build/clang
>> -tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>> --
>> Exit Code: 2
>>
>> Command Output (stderr):
>> --
>> FileCheck error: '-' is empty.
>> FileCheck command line:  FileCheck --match-full-lines
>> -implicit-check-not={{warning:|error:}}
>>
>> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>>
>> --
>>
>> 
>> FAIL: Clang Tools ::
>> clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp (125 of 867)
>>  TEST 'Clang Tools ::
>> clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp' FAILED
>> 
>> Script:
>> --
>> : 'RUN: at line 1';   clang-tidy -enable-check-profile
>> -checks='-*,readability-function-size'
>>
>> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>>
>> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>> -- 2>&1 |
>> FileCheck --match-full-lines -implicit-check-not='{{warning:|error:}}'
>>
>> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>> --
>> Exit Code: 2
>>
>> Command Output (stderr):
>> --
>> FileCheck error: '-' is empty.
>> FileCheck command line:  FileCheck --match-full-lines
>> -implicit-check-not={{warning:|error:}}
>>
>> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>>
>> --
>>
>> Roman.
>>
>> On Tue, Jun 5, 2018 at 12:42 PM, Sam McCall via cfe-commits
>>  wrote:
>> > Author: sammccall
>> > Date: Tue Jun  5 02:42:06 2018
>> > New Revision: 333994
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=333994=rev
>> > Log:
>> > [clang-tidy] fix broken test (no compile command) from r331763
>> >
>> > Modified:
>> >
>> > clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>> >
>> > clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>> >
>> > Modified:
>> > clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>> > URL:
>> > http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp?rev=333994=333993=333994=diff
>> >
>> > ==
>> > ---
>> > clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>> > (original)
>> > +++
>> > clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>> > Tue Jun  5 02:42:06 2018
>> > @@ -1,4 +1,4 @@
>> > -// RUN: clang-tidy -enable-check-profile
>> > -checks='-*,readability-function-size' %s 2>&1 | FileCheck
>> > --match-full-lines -implicit-check-not='{{warning:|error:}}' %s
>> > +// RUN: clang-tidy -enable-check-profile
>> > -checks='-*,readability-function-size' %s -- 2>&1 | FileCheck
>> > --match-full-lines -implicit-check-not='{{warning:|error:}}' %s
>> >
>> >  // CHECK:
>> > ===-===
>> >  // CHECK-NEXT: {{.*}}  --- Name ---
>> >
>> > Modified:
>> > clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>> > URL:
>> > 

Re: [clang-tools-extra] r333994 - [clang-tidy] fix broken test (no compile command) from r331763

[Sam McCall via cfe-commits]

Hi Roman,

Definitely something different in our configs, but it wasn't just me - a
couple of colleagues have mentioned those tests as being broken for a while!

The problem is the lack of compilation database, so I can imagine it can
depend on filesystem layout, e.g. whether you have a separate build/source
directory. (Potentially the CDB from llvm itself may be picked up or not)

I can't tell from your lit output whether clang-tidy or FileCheck failed,
what does this command return for you?

bin/clang-tidy -enable-check-profile -checks='-*,readability-function-size'
../src/llvm/tools/clang/tools/extra/test/clang-tid
y/clang-tidy-enable-check-profile-one-tu.cpp --

(adjust path as needed)

On Tue, Jun 5, 2018 at 12:30 PM Roman Lebedev  wrote:

> This is strange.
>
> First, i'm pretty sure the test worked for me before.
> Second, this commit actually *breaks* those two tests for me:
>
> $ ninja check-clang-tools
> [0/1] Running the Clang extra tools' regression tests
> FAIL: Clang Tools ::
> clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp (123 of 867)
>  TEST 'Clang Tools ::
> clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp' FAILED
> 
> Script:
> --
> : 'RUN: at line 1';   clang-tidy -enable-check-profile
> -checks='-*,readability-function-size'
>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> -- 2>&1 | FileCheck --match-full-lines
> -implicit-check-not='{{warning:|error:}}' /build/clang
> -tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> --
> Exit Code: 2
>
> Command Output (stderr):
> --
> FileCheck error: '-' is empty.
> FileCheck command line:  FileCheck --match-full-lines
> -implicit-check-not={{warning:|error:}}
>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>
> --
>
> 
> FAIL: Clang Tools ::
> clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp (125 of 867)
>  TEST 'Clang Tools ::
> clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp' FAILED
> 
> Script:
> --
> : 'RUN: at line 1';   clang-tidy -enable-check-profile
> -checks='-*,readability-function-size'
>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> -- 2>&1 |
> FileCheck --match-full-lines -implicit-check-not='{{warning:|error:}}'
>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> --
> Exit Code: 2
>
> Command Output (stderr):
> --
> FileCheck error: '-' is empty.
> FileCheck command line:  FileCheck --match-full-lines
> -implicit-check-not={{warning:|error:}}
>
> /build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>
> --
>
> Roman.
>
> On Tue, Jun 5, 2018 at 12:42 PM, Sam McCall via cfe-commits
>  wrote:
> > Author: sammccall
> > Date: Tue Jun  5 02:42:06 2018
> > New Revision: 333994
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=333994=rev
> > Log:
> > [clang-tidy] fix broken test (no compile command) from r331763
> >
> > Modified:
> >
>  
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> >
>  
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> >
> > Modified:
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp?rev=333994=333993=333994=diff
> >
> ==
> > ---
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> (original)
> > +++
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> Tue Jun  5 02:42:06 2018
> > @@ -1,4 +1,4 @@
> > -// RUN: clang-tidy -enable-check-profile
> -checks='-*,readability-function-size' %s 2>&1 | FileCheck
> --match-full-lines -implicit-check-not='{{warning:|error:}}' %s
> > +// RUN: clang-tidy -enable-check-profile
> -checks='-*,readability-function-size' %s -- 2>&1 | FileCheck
> --match-full-lines -implicit-check-not='{{warning:|error:}}' %s
> >
> >  // CHECK:
> ===-===
> >  // CHECK-NEXT: {{.*}}  --- Name ---
> >
> > Modified:
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp?rev=333994=333993=333994=diff
> >
> ==
> > ---
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> (original)
> > +++
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> Tue 

Re: [clang-tools-extra] r333994 - [clang-tidy] fix broken test (no compile command) from r331763

[Roman Lebedev via cfe-commits]

This is strange.

First, i'm pretty sure the test worked for me before.
Second, this commit actually *breaks* those two tests for me:

$ ninja check-clang-tools
[0/1] Running the Clang extra tools' regression tests
FAIL: Clang Tools ::
clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp (123 of 867)
 TEST 'Clang Tools ::
clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp' FAILED

Script:
--
: 'RUN: at line 1';   clang-tidy -enable-check-profile
-checks='-*,readability-function-size'
/build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
-- 2>&1 | FileCheck --match-full-lines
-implicit-check-not='{{warning:|error:}}' /build/clang
-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
--
Exit Code: 2

Command Output (stderr):
--
FileCheck error: '-' is empty.
FileCheck command line:  FileCheck --match-full-lines
-implicit-check-not={{warning:|error:}}
/build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp

--


FAIL: Clang Tools ::
clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp (125 of 867)
 TEST 'Clang Tools ::
clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp' FAILED

Script:
--
: 'RUN: at line 1';   clang-tidy -enable-check-profile
-checks='-*,readability-function-size'
/build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
/build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
-- 2>&1 |
FileCheck --match-full-lines -implicit-check-not='{{warning:|error:}}'
/build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
--
Exit Code: 2

Command Output (stderr):
--
FileCheck error: '-' is empty.
FileCheck command line:  FileCheck --match-full-lines
-implicit-check-not={{warning:|error:}}
/build/clang-tools-extra/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp

--

Roman.

On Tue, Jun 5, 2018 at 12:42 PM, Sam McCall via cfe-commits
 wrote:
> Author: sammccall
> Date: Tue Jun  5 02:42:06 2018
> New Revision: 333994
>
> URL: http://llvm.org/viewvc/llvm-project?rev=333994=rev
> Log:
> [clang-tidy] fix broken test (no compile command) from r331763
>
> Modified:
> 
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> 
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>
> Modified: 
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
> URL: 
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp?rev=333994=333993=333994=diff
> ==
> --- 
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>  (original)
> +++ 
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
>  Tue Jun  5 02:42:06 2018
> @@ -1,4 +1,4 @@
> -// RUN: clang-tidy -enable-check-profile 
> -checks='-*,readability-function-size' %s 2>&1 | FileCheck --match-full-lines 
> -implicit-check-not='{{warning:|error:}}' %s
> +// RUN: clang-tidy -enable-check-profile 
> -checks='-*,readability-function-size' %s -- 2>&1 | FileCheck 
> --match-full-lines -implicit-check-not='{{warning:|error:}}' %s
>
>  // CHECK: 
> ===-===
>  // CHECK-NEXT: {{.*}}  --- Name ---
>
> Modified: 
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
> URL: 
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp?rev=333994=333993=333994=diff
> ==
> --- 
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>  (original)
> +++ 
> clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
>  Tue Jun  5 02:42:06 2018
> @@ -1,4 +1,4 @@
> -// RUN: clang-tidy -enable-check-profile 
> -checks='-*,readability-function-size' %s %s 2>&1 | FileCheck 
> --match-full-lines -implicit-check-not='{{warning:|error:}}' %s
> +// RUN: clang-tidy -enable-check-profile 
> -checks='-*,readability-function-size' %s %s -- 2>&1 | FileCheck 
> --match-full-lines -implicit-check-not='{{warning:|error:}}' %s
>
>  // CHECK: 
> ===-===
>  // CHECK-NEXT: {{.*}}  --- Name ---
>
>
> ___
> cfe-commits mailing list
> cfe-commits@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] r333994 - [clang-tidy] fix broken test (no compile command) from r331763

[Sam McCall via cfe-commits]

Author: sammccall
Date: Tue Jun  5 02:42:06 2018
New Revision: 333994

URL: http://llvm.org/viewvc/llvm-project?rev=333994=rev
Log:
[clang-tidy] fix broken test (no compile command) from r331763

Modified:

clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp

clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp

Modified: 
clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
URL: 
http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp?rev=333994=333993=333994=diff
==
--- 
clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
 (original)
+++ 
clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-one-tu.cpp
 Tue Jun  5 02:42:06 2018
@@ -1,4 +1,4 @@
-// RUN: clang-tidy -enable-check-profile 
-checks='-*,readability-function-size' %s 2>&1 | FileCheck --match-full-lines 
-implicit-check-not='{{warning:|error:}}' %s
+// RUN: clang-tidy -enable-check-profile 
-checks='-*,readability-function-size' %s -- 2>&1 | FileCheck 
--match-full-lines -implicit-check-not='{{warning:|error:}}' %s
 
 // CHECK: 
===-===
 // CHECK-NEXT: {{.*}}  --- Name ---

Modified: 
clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
URL: 
http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp?rev=333994=333993=333994=diff
==
--- 
clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
 (original)
+++ 
clang-tools-extra/trunk/test/clang-tidy/clang-tidy-enable-check-profile-two-tu.cpp
 Tue Jun  5 02:42:06 2018
@@ -1,4 +1,4 @@
-// RUN: clang-tidy -enable-check-profile 
-checks='-*,readability-function-size' %s %s 2>&1 | FileCheck 
--match-full-lines -implicit-check-not='{{warning:|error:}}' %s
+// RUN: clang-tidy -enable-check-profile 
-checks='-*,readability-function-size' %s %s -- 2>&1 | FileCheck 
--match-full-lines -implicit-check-not='{{warning:|error:}}' %s
 
 // CHECK: 
===-===
 // CHECK-NEXT: {{.*}}  --- Name ---


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits