Hi,

Could you fix the test for Darwin (or split it in two)? It seems some
functionalities are not supported:

/Users/filipe/work/
llvm.org/clang-only/tools/clang/test/Misc/ast-dump-attr.cpp:82:31: error:
only weak aliases are supported on darwin
int TestString __attribute__((alias("alias1")));
                              ^
1 error generated.
/Users/filipe/work/
llvm.org/clang-only/tools/clang/test/Misc/ast-dump-attr.cpp:84:18: error:
expected string not found in input
// CHECK-NEXT:   AliasAttr{{.*}} "alias1"
                 ^
<stdin>:84:129: note: scanning from here


Thanks,

  Filipe

  F


On Mon, Jan 7, 2013 at 11:10 AM, Alexander Kornienko <[email protected]>wrote:

> Should be fixed in r171768.
>
> Thank you for noticing this!
>
>
> On Mon, Jan 7, 2013 at 7:08 PM, Dmitri Gribenko <[email protected]>wrote:
>
>> On Mon, Jan 7, 2013 at 7:53 PM, Alexander Kornienko <[email protected]>
>> wrote:
>> > Author: alexfh
>> > Date: Mon Jan  7 11:53:08 2013
>> > New Revision: 171760
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=171760&view=rev
>> > Log:
>> > Implement Attr dumping for -ast-dump.
>> > http://llvm-reviews.chandlerc.com/D234
>> >
>> > Patch by Philip Craig!
>>
>> Hello Alexander,
>>
>> It looks like this commit introduced a few warnings like this:
>>
>> tools/clang/include/clang/AST/AttrDump.inc:80:5: warning: default
>> label in switch which covers all enumeration values
>> [-Wcovered-switch-default]
>>     default:
>>     ^
>>
>> Dmitri
>>
>> --
>> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
>> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/
>
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to