r212016 - Fixed CodeGen/captured-statements-nested.c test

Alexey Bataev a.bataev at hotmail.com
Mon Jun 30 19:10:37 PDT 2014


Hi David,
I removed some extra checks which make this test incompatible with 32 
bit mode (in 64 bit mode there are some additional conversions are 
generated, in 32 bit mode - not)

Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team

30.06.2014 19:38, David Blaikie пишет:
> On Sun, Jun 29, 2014 at 10:02 PM, Alexey Bataev <a.bataev at hotmail.com> wrote:
>> Author: abataev
>> Date: Mon Jun 30 00:02:50 2014
>> New Revision: 212016
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=212016&view=rev
>> Log:
>> Fixed CodeGen/captured-statements-nested.c test
> An explanation of what you changed/fixed might be helpful - I couldn't
> spot it easily just looknig at the diff
>
>> Modified:
>>      cfe/trunk/test/CodeGen/captured-statements-nested.c
>>
>> Modified: cfe/trunk/test/CodeGen/captured-statements-nested.c
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/captured-statements-nested.c?rev=212016&r1=212015&r2=212016&view=diff
>> ==============================================================================
>> --- cfe/trunk/test/CodeGen/captured-statements-nested.c (original)
>> +++ cfe/trunk/test/CodeGen/captured-statements-nested.c Mon Jun 30 00:02:50 2014
>> @@ -1,7 +1,6 @@
>>   // RUN: %clang_cc1 -fblocks -emit-llvm %s -o %t
>>   // RUN: FileCheck %s -input-file=%t -check-prefix=CHECK1
>>   // RUN: FileCheck %s -input-file=%t -check-prefix=CHECK2
>> -// REQUIRES:disabled
>>
>>   struct A {
>>     int a;
>> @@ -65,27 +64,27 @@ void test_nest_captured_stmt(int param,
>>           // CHECK1-NEXT: store i8 99
>>           //
>>           // CHECK1: [[SIZE_ADDR_REF:%.*]] = getelementptr inbounds [[T]]* {{.*}}, i32 0, i32 5
>> -        // CHECK1-NEXT: [[SIZE_ADDR:%.*]] = load i32** [[SIZE_ADDR_REF]]
>> -        // CHECK1-NEXT: [[SIZE:%.*]] = load i32* [[SIZE_ADDR]]
>> -        // CHECK1-NEXT: [[SIZE_MINUS_1:%.*]] = sub nsw i32 [[SIZE]], 1
>> -        // CHECK1-NEXT: [[PARAM_ARR_IDX:%.*]] = {{.*}} [[SIZE_MINUS_1]]
>> -        // CHECK1-NEXT: [[PARAM_ARR_ADDR_REF:%.*]] = getelementptr inbounds [[T]]* {{.*}}, i32 0, i32 6
>> -        // CHECK1-NEXT: [[PARAM_ARR_ADDR:%.*]] = load i32*** [[PARAM_ARR_ADDR_REF]]
>> -        // CHECK1-NEXT: [[PARAM_ARR:%.*]] = load i32** [[PARAM_ARR_ADDR]]
>> -        // CHECK1-NEXT: [[PARAM_ARR_SIZE_MINUS_1_ADDR:%.*]] = getelementptr inbounds i32* [[PARAM_ARR]], i{{.*}} [[PARAM_ARR_IDX]]
>> -        // CHECK1-NEXT: store i32 2, i32* [[PARAM_ARR_SIZE_MINUS_1_ADDR]]
>> +        // CHECK1-DAG: [[SIZE_ADDR:%.*]] = load i32** [[SIZE_ADDR_REF]]
>> +        // CHECK1-DAG: [[SIZE:%.*]] = load i32* [[SIZE_ADDR]]
>> +        // CHECK1-DAG: [[SIZE_MINUS_1:%.*]] = sub nsw i32 [[SIZE]], 1
>> +        // CHECK1-DAG: [[PARAM_ARR_IDX:%.*]] = {{.*}} [[SIZE_MINUS_1]]
>> +        // CHECK1-DAG: [[PARAM_ARR_ADDR_REF:%.*]] = getelementptr inbounds [[T]]* {{.*}}, i32 0, i32 6
>> +        // CHECK1-DAG: [[PARAM_ARR_ADDR:%.*]] = load i32*** [[PARAM_ARR_ADDR_REF]]
>> +        // CHECK1-DAG: [[PARAM_ARR:%.*]] = load i32** [[PARAM_ARR_ADDR]]
>> +        // CHECK1-DAG: [[PARAM_ARR_SIZE_MINUS_1_ADDR:%.*]] = getelementptr inbounds i32* [[PARAM_ARR]], i{{.*}} [[PARAM_ARR_IDX]]
>> +        // CHECK1: store i32 2, i32* [[PARAM_ARR_SIZE_MINUS_1_ADDR]]
>>           //
>>           // CHECK1: [[Z_ADDR_REF:%.*]] = getelementptr inbounds [[T]]* {{.*}}, i32 0, i32 2
>> -        // CHECK1-NEXT: [[Z_ADDR:%.*]] = load %struct.A** [[Z_ADDR_REF]]
>> -        // CHECK1-NEXT: [[Z_A_ADDR:%.*]] = getelementptr inbounds %struct.A* [[Z_ADDR]], i32 0, i32 0
>> -        // CHECK1-NEXT: [[Z_A:%.*]] = load i32* [[Z_A_ADDR]]
>> -        // CHECK1-NEXT: [[ARR_IDX_2:%.*]] = {{.*}} [[Z_A]]
>> -        // CHECK1-NEXT: [[ARR_ADDR_REF:%.*]] = getelementptr inbounds [[T]]* {{.*}}, i32 0, i32 7
>> -        // CHECK1-NEXT: [[ARR_ADDR:%.*]] = load i32** [[ARR_ADDR_REF]]
>> -        // CHECK1-NEXT: [[ARR_IDX_1:%.*]] = mul {{.*}} 10
>> -        // CHECK1-NEXT: [[ARR_10_ADDR:%.*]] = getelementptr inbounds i32* [[ARR_ADDR]], i{{.*}} [[ARR_IDX_1]]
>> -        // CHECK1-NEXT: [[ARR_10_Z_A_ADDR:%.*]] = getelementptr inbounds i32* [[ARR_10_ADDR]], i{{.*}} [[ARR_IDX_2]]
>> -        // CHECK1-NEXT: store i32 12, i32* [[ARR_10_Z_A_ADDR]]
>> +        // CHECK1-DAG: [[Z_ADDR:%.*]] = load %struct.A** [[Z_ADDR_REF]]
>> +        // CHECK1-DAG: [[Z_A_ADDR:%.*]] = getelementptr inbounds %struct.A* [[Z_ADDR]], i32 0, i32 0
>> +        // CHECK1-DAG: [[Z_A:%.*]] = load i32* [[Z_A_ADDR]]
>> +        // CHECK1-DAG: [[ARR_IDX_2:%.*]] = {{.*}} [[Z_A]]
>> +        // CHECK1-DAG: [[ARR_ADDR_REF:%.*]] = getelementptr inbounds [[T]]* {{.*}}, i32 0, i32 7
>> +        // CHECK1-DAG: [[ARR_ADDR:%.*]] = load i32** [[ARR_ADDR_REF]]
>> +        // CHECK1-DAG: [[ARR_IDX_1:%.*]] = mul {{.*}} 10
>> +        // CHECK1-DAG: [[ARR_10_ADDR:%.*]] = getelementptr inbounds i32* [[ARR_ADDR]], i{{.*}} [[ARR_IDX_1]]
>> +        // CHECK1-DAG: [[ARR_10_Z_A_ADDR:%.*]] = getelementptr inbounds i32* [[ARR_10_ADDR]], i{{.*}} [[ARR_IDX_2]]
>> +        // CHECK1: store i32 12, i32* [[ARR_10_Z_A_ADDR]]
>>         }
>>       }
>>     }
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits





More information about the cfe-commits mailing list