On Apr 23, 2013, at 9:01 AM, Rafael Espíndola <[email protected]>
wrote:
> This test looks really odd. It seems designed to be an execution test
> (has a main function, uses assert), but we only produce IL from it. It
> has been odd since it was added in r72437.
>
> Douglas, what was the intent of this test? Should it be moved to the
> llvm-testsuite where it will actually be executed?
This test can simply be removed. It's not adding any value.
- Doug
> On 23 April 2013 11:44, Jyotsna Verma <[email protected]> wrote:
>> The attached patch modifies SemaTemplate/example-dynarray.cpp test to have
>> explicit declaration for malloc, free and assert functions and removes its
>> dependency on <stdlib.h> and <assert.h>.
>>
>>
>>
>> Please let me know if the patch is okay to commit.
>>
>>
>>
>> Thanks,
>>
>> Jyotsna
>>
>>
>>
>> --
>>
>> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
>> The Linux Foundation
>>
>>
>>
>>
>> _______________________________________________
>> 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
