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 <[email protected]> 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
[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