| commit | 421048698b6b6bf86754190bcfe98a0ed82ee5b5 | |
| author | Chris Lattner <sabre@nondot.org> | |
| Sun, 17 Apr 2011 23:21:26 +0000 (17 23:21 +0000) | ||
| committer | Chris Lattner <sabre@nondot.org> | |
| Sun, 17 Apr 2011 23:21:26 +0000 (17 23:21 +0000) | ||
| tree | ed832ae025ced205bcd95935183e876906d14ac6 | treesnapshot (tar.gz zip) |
| parent | f54d81f516ee01467ab0ed54b5031e552c857ff7 | commitdiff |
Fix a miscompilation I introduced in r129652, thanks for Eli for tracking
it down. we effectively were compile the testcase into:
void test14(int x) {
switch (x) {
case 11: break;
case 42: test14(97); // fallthrough
default: test14(42); break;
which is not the same thing at all. This fixes a miscompilation of
MallocBench/gs seen on the clang-x86_64-linux-fnt buildbot.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@129679 91177308-0d34-0410-b5e6-96231b3b80d8
it down. we effectively were compile the testcase into:
void test14(int x) {
switch (x) {
case 11: break;
case 42: test14(97); // fallthrough
default: test14(42); break;
which is not the same thing at all. This fixes a miscompilation of
MallocBench/gs seen on the clang-x86_64-linux-fnt buildbot.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@129679 91177308-0d34-0410-b5e6-96231b3b80d8
| lib/CodeGen/CGStmt.cpp | diffblobblamehistory | |
| test/CodeGen/switch-dce.c | diffblobblamehistory |