r208703 - Updated the test case to show that no diagnostics are expected, and not require emitting the AST until after the AST printing for statement attributes is updated.

[Aaron Ballman]

On Tue, May 13, 2014 at 11:44 AM, Nico Weber <thakis at chromium.org> wrote:
> I think expected-no-diagnostics goes with -verify on the run line.

Good catch! However, this is temporary. I'm fixing -ast-print so that
it properly prints out attributed statements, at which point I'll
bring back FileCheck for this test. Currently, we pretty print the
attributes improperly by putting extra square brackets around
everything.

~Aaron

>
> On May 13, 2014 5:22 PM, "Aaron Ballman" <aaron at aaronballman.com> wrote:
>>
>> Author: aaronballman
>> Date: Tue May 13 10:14:59 2014
>> New Revision: 208703
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=208703&view=rev
>> Log:
>> Updated the test case to show that no diagnostics are expected, and not
>> require emitting the AST until after the AST printing for statement
>> attributes is updated.
>>
>> Modified:
>>     cfe/trunk/test/PCH/stmt-attrs.cpp
>>
>> Modified: cfe/trunk/test/PCH/stmt-attrs.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/PCH/stmt-attrs.cpp?rev=208703&r1=208702&r2=208703&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/test/PCH/stmt-attrs.cpp (original)
>> +++ cfe/trunk/test/PCH/stmt-attrs.cpp Tue May 13 10:14:59 2014
>> @@ -1,5 +1,6 @@
>>  // RUN: %clang_cc1 -std=c++11 -emit-pch -o %t.a %s
>> -// RUN: %clang_cc1 -std=c++11 -include-pch %t.a %s -ast-print -o - |
>> FileCheck %s
>> +// RUN: %clang_cc1 -std=c++11 -fsyntax-only -include-pch %t.a %s
>> +// expected-no-diagnostics
>>
>>  #ifndef HEADER
>>  #define HEADER
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits