[cfe-commits] r83325 - /cfe/trunk/test/CodeGen/mmintrin-test.c

[Daniel Dunbar]

On Sat, Oct 17, 2009 at 10:06 AM, Chris Lattner <sabre at nondot.org> wrote:
>
> On Oct 17, 2009, at 2:12 AM, Daniel Dunbar wrote:
>
>> On Mon, Oct 5, 2009 at 2:16 PM, Chris Lattner <sabre at nondot.org> wrote:
>>>
>>> Author: lattner
>>> Date: Mon Oct  5 16:16:22 2009
>>> New Revision: 83325
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=83325&view=rev
>>> Log:
>>> remove this test, grepping for define isn't really testing anything.
>>
>> I agree grepping for define isn't testing anything, but the syntax
>> checking of the test was?
>
> These headers are included by cocoa.h and other things, I'm pretty sure we
> get coverage over them.  I killed them off because I was trying to reduce
> uses of 'count'.

That covers the sema of the intrinsics, but not that they can actually
be used for anything. This test came from actual code which didn't
parse at one time... it's not perfect but I think its worth putting
back as just a Sema and IRgen test, without count or the grep.

 - Daniel

>
> -Chris
>
>>
>> - Daniel
>>
>>> Removed:
>>>   cfe/trunk/test/CodeGen/mmintrin-test.c
>>>
>>> Removed: cfe/trunk/test/CodeGen/mmintrin-test.c
>>> URL:
>>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/mmintrin-test.c?rev=83324&view=auto
>>>
>>>
>>> ==============================================================================
>>> --- cfe/trunk/test/CodeGen/mmintrin-test.c (original)
>>> +++ cfe/trunk/test/CodeGen/mmintrin-test.c (removed)
>>> @@ -1,26 +0,0 @@
>>> -// RUN: clang-cc -triple i386-apple-darwin9 -mcpu=pentium4 -emit-llvm -o
>>> %t %s &&
>>> -// RUN: grep define %t | count 1 &&
>>> -// RUN: clang-cc -triple i386-apple-darwin9 -mcpu=pentium4 -g -emit-llvm
>>> -o %t %s &&
>>> -// RUN: grep define %t | count 1
>>> -
>>> -#include <mmintrin.h>
>>> -#include <stdio.h>
>>> -
>>> -int main(int argc, char *argv[]) {
>>> -  int array[16] = { 0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15 };
>>> -  __m64 *p = (__m64 *)array;
>>> -
>>> -  __m64 accum = _mm_setzero_si64();
>>> -
>>> -  for (int i=0; i<8; ++i)
>>> -    accum = _mm_add_pi32(p[i], accum);
>>> -
>>> -  __m64 accum2 = _mm_unpackhi_pi32(accum, accum);
>>> -  accum = _mm_add_pi32(accum, accum2);
>>> -
>>> -  int result = _mm_cvtsi64_si32(accum);
>>> -  _mm_empty();
>>> -  printf("%d\n", result );
>>> -
>>> -  return 0;
>>> -}
>>>
>>>
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>>
>
>