Prohibit inlining of functions containing throw. Inliner does not handle abnormal control flow.

runtime/vm/intermediate_language.cc

Index: runtime/vm/intermediate_language.cc
diff --git a/runtime/vm/intermediate_language.cc b/runtime/vm/intermediate_language.cc
index c5b9ef62044b292083ecff0d5804b992ec6753d1..cb5a666e19ece8a4c866c119eb129a2197f23206 100644
--- a/runtime/vm/intermediate_language.cc
+++ b/runtime/vm/intermediate_language.cc
@@ -1140,9 +1140,6 @@ Definition* StrictCompareInstr::Canonicalize() {
   if ((kind() == Token::kEQ_STRICT) &&
       (right_constant.raw() == Bool::True()) &&
       (left()->ResultCid() == kBoolCid)) {
-    // Remove the constant from the graph.
-    Definition* right_defn = right()->definition();
-    right_defn->RemoveFromGraph();

[srdjan, 2012/09/13 12:51:09]

Why don't you remove the right hand side?

[Vyacheslav Egorov (Google), 2012/09/13 13:15:57]

It can still have uses. It's not for canonicalization pass to remove it. In any
case register allocator will ignore constants that are not used.

     // Return left subexpression as the replacement for this instruction.
     return left_defn;
   }