From 1b8942ce72a25e1e3d63128660d76b5ecde8acfb Mon Sep 17 00:00:00 2001
From: Heejin Ahn <aheejin@gmail.com>
Date: Tue, 2 Jun 2020 00:01:58 -0700
Subject: Fix SideEffect::Branches to only represent branches (#2886)

After #2783 `SideEffects::Branches` includes possibly throwing
expressions, which can be calls (when EH is enabled). This changes
`SideEffects::Branches` back to only include branches, returns, and
infinite loops as it was before #2783.
---
 src/ir/effects.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/ir/effects.h b/src/ir/effects.h
index ff641e4d9..d2afc8133 100644
--- a/src/ir/effects.h
+++ b/src/ir/effects.h
@@ -504,7 +504,7 @@ struct EffectAnalyzer
   };
   uint32_t getSideEffects() const {
     uint32_t effects = 0;
-    if (transfersControlFlow()) {
+    if (branchesOut || hasExternalBreakTargets()) {
       effects |= SideEffects::Branches;
     }
     if (calls) {
-- 
cgit v1.2.3