From: Daniel Dunbar Date: Fri, 2 Apr 2010 22:29:35 +0000 (+0000) Subject: Merge several tests into switch.c. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=fae7bf269034202adf43840379340a60a9992292;p=clang Merge several tests into switch.c. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@100235 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/test/CodeGen/rdr-6098585-default-after-caserange.c b/test/CodeGen/rdr-6098585-default-after-caserange.c deleted file mode 100644 index 3a89aa39a0..0000000000 --- a/test/CodeGen/rdr-6098585-default-after-caserange.c +++ /dev/null @@ -1,18 +0,0 @@ -// RUN: %clang_cc1 -triple i386-unknown-unknown -emit-llvm-bc -o - %s | opt -std-compile-opts | llvm-dis > %t -// RUN: grep "ret i32" %t | count 1 -// RUN: grep "ret i32 10" %t | count 1 - -// Ensure that default after a case range is not ignored. - -static int f1(unsigned x) { - switch(x) { - case 10 ... 0xFFFFFFFF: - return 0; - default: - return 10; - } -} - -int g() { - return f1(2); -} diff --git a/test/CodeGen/rdr-6098585-default-fallthrough-to-caserange.c b/test/CodeGen/rdr-6098585-default-fallthrough-to-caserange.c deleted file mode 100644 index ba41b519fc..0000000000 --- a/test/CodeGen/rdr-6098585-default-fallthrough-to-caserange.c +++ /dev/null @@ -1,20 +0,0 @@ -// RUN: %clang_cc1 -triple i386-unknown-unknown -emit-llvm-bc -o - %s | opt -std-compile-opts | llvm-dis > %t -// RUN: grep "ret i32 10" %t - -// Ensure that this doesn't compile to infinite loop in g() due to -// miscompilation of fallthrough from default to a (tested) case -// range. - -static int f0(unsigned x) { - switch(x) { - default: - x += 1; - case 10 ... 0xFFFFFFFF: - return 0; - } -} - -int g() { - f0(1); - return 10; -} diff --git a/test/CodeGen/rdr-6098585-empty-case-range.c b/test/CodeGen/rdr-6098585-empty-case-range.c deleted file mode 100644 index 1cf77ac6aa..0000000000 --- a/test/CodeGen/rdr-6098585-empty-case-range.c +++ /dev/null @@ -1,23 +0,0 @@ -// RUN: %clang_cc1 -triple i386-unknown-unknown -emit-llvm-bc -o - %s | opt -std-compile-opts | llvm-dis > %t -// RUN: grep "ret i32" %t | count 2 -// RUN: grep "ret i32 3" %t | count 2 - -// This generated incorrect code because of poor switch chaining. -int f1(int x) { - switch(x) { - default: - return 3; - case 10 ... 0xFFFFFFFF: - return 0; - } -} - -// This just asserted because of the way case ranges were calculated. -int f2(int x) { - switch (x) { - default: - return 3; - case 10 ... -1: - return 0; - } -} diff --git a/test/CodeGen/rdr-6098585-fallthrough-to-empty-range.c b/test/CodeGen/rdr-6098585-fallthrough-to-empty-range.c deleted file mode 100644 index 48a6cc2285..0000000000 --- a/test/CodeGen/rdr-6098585-fallthrough-to-empty-range.c +++ /dev/null @@ -1,15 +0,0 @@ -// RUN: %clang_cc1 -triple i386-unknown-unknown -emit-llvm-bc -o - %s | opt -std-compile-opts | llvm-dis > %t -// RUN: grep "ret i32 %" %t - -// Make sure return is not constant (if empty range is skipped or miscompiled) - -int f0(unsigned x) { - switch(x) { - case 2: - // fallthrough empty range - case 10 ... 9: - return 10; - default: - return 0; - } -} diff --git a/test/CodeGen/rdr-6098585-unsigned-caserange.c b/test/CodeGen/rdr-6098585-unsigned-caserange.c deleted file mode 100644 index 6f577df188..0000000000 --- a/test/CodeGen/rdr-6098585-unsigned-caserange.c +++ /dev/null @@ -1,12 +0,0 @@ -// RUN: %clang_cc1 -triple i386-unknown-unknown -emit-llvm-bc -o - %s | opt -std-compile-opts | llvm-dis > %t -// RUN: grep "ret i32" %t | count 1 -// RUN: grep "ret i32 3" %t | count 1 - -int f2(unsigned x) { - switch(x) { - default: - return 3; - case 0xFFFFFFFF ... 1: // This range should be empty because x is unsigned. - return 0; - } -} diff --git a/test/CodeGen/switch.c b/test/CodeGen/switch.c index 519ccbac01..dc2d27bc16 100644 --- a/test/CodeGen/switch.c +++ b/test/CodeGen/switch.c @@ -100,3 +100,97 @@ void foo7(){ } } + +// CHECK: define i32 @f8( +// CHECK: ret i32 3 +// CHECK: } +int f8(unsigned x) { + switch(x) { + default: + return 3; + case 0xFFFFFFFF ... 1: // This range should be empty because x is unsigned. + return 0; + } +} + +// Ensure that default after a case range is not ignored. +// +// CHECK: define i32 @f9() +// CHECK: ret i32 10 +// CHECK: } +static int f9_0(unsigned x) { + switch(x) { + case 10 ... 0xFFFFFFFF: + return 0; + default: + return 10; + } +} +int f9() { + return f9_0(2); +} + +// Ensure that this doesn't compile to infinite loop in g() due to +// miscompilation of fallthrough from default to a (tested) case +// range. +// +// CHECK: define i32 @f10() +// CHECK: ret i32 10 +// CHECK: } +static int f10_0(unsigned x) { + switch(x) { + default: + x += 1; + case 10 ... 0xFFFFFFFF: + return 0; + } +} + +int f10() { + f10_0(1); + return 10; +} + +// This generated incorrect code because of poor switch chaining. +// +// CHECK: define i32 @f11( +// CHECK: ret i32 3 +// CHECK: } +int f11(int x) { + switch(x) { + default: + return 3; + case 10 ... 0xFFFFFFFF: + return 0; + } +} + +// This just asserted because of the way case ranges were calculated. +// +// CHECK: define i32 @f12( +// CHECK: ret i32 3 +// CHECK: } +int f12(int x) { + switch (x) { + default: + return 3; + case 10 ... -1: + return 0; + } +} + +// Make sure return is not constant (if empty range is skipped or miscompiled) +// +// CHECK: define i32 @f13( +// CHECK: ret i32 % +// CHECK: } +int f13(unsigned x) { + switch(x) { + case 2: + // fallthrough empty range + case 10 ... 9: + return 10; + default: + return 0; + } +}