Index: llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp =================================================================== --- llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp +++ llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp @@ -2486,10 +2486,24 @@ SawCatchAll = true; break; } - if (AlreadyCaught.count(TypeInfo)) - // Already caught by an earlier clause, so having it in the filter - // is pointless. - continue; + + // Even if we've seen a type in a catch clause, we don't want to + // remove it from the filter. An unexpected type handler may be + // set up for a call site which throws an exception of the same + // type caught. In order for the exception thrown by the unexpected + // handler to propogate correctly, the filter must be correctly + // described for the call site. + // + // Example: + // + // void unexpected() { throw 1;} + // void foo() throw (int) { + // std::set_unexpected(unexpected); + // try { + // throw 2.0; + // } catch (int i) {} + // } + // There is no point in having multiple copies of the same typeinfo in // a filter, so only add it if we didn't already. if (SeenInFilter.insert(TypeInfo).second) Index: llvm/trunk/test/Transforms/InstCombine/LandingPadClauses.ll =================================================================== --- llvm/trunk/test/Transforms/InstCombine/LandingPadClauses.ll +++ llvm/trunk/test/Transforms/InstCombine/LandingPadClauses.ll @@ -69,9 +69,11 @@ filter [1 x i32*] [i32* @T1] catch i32* @T2 unreachable +; Caught types should not be removed from filters ; CHECK: %c = landingpad -; CHECK-NEXT: @T1 -; CHECK-NEXT: filter [0 x i32*] +; CHECK-NEXT: catch i32* @T1 +; CHECK-NEXT: filter [1 x i32*] [i32* @T1] +; CHECK-NEXT: catch i32* @T2 ; CHECK-NEXT: unreachable lpad.d: @@ -87,9 +89,10 @@ catch i32* @T1 filter [3 x i32*] [i32* @T1, i32* @T2, i32* @T2] unreachable +; Caught types should not be removed from filters ; CHECK: %e = landingpad -; CHECK-NEXT: @T1 -; CHECK-NEXT: filter [1 x i32*] [i32* @T2] +; CHECK-NEXT: catch i32* @T1 +; CHECK-NEXT: filter [2 x i32*] [i32* @T1, i32* @T2] ; CHECK-NEXT: unreachable lpad.f: