From 1b8942ce72a25e1e3d63128660d76b5ecde8acfb Mon Sep 17 00:00:00 2001 From: Heejin Ahn 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. --- test/binaryen.js/sideffects.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'test/binaryen.js') diff --git a/test/binaryen.js/sideffects.js b/test/binaryen.js/sideffects.js index 7db9ab329..76bc1db68 100644 --- a/test/binaryen.js/sideffects.js +++ b/test/binaryen.js/sideffects.js @@ -95,11 +95,11 @@ assert( ); // If exception handling feature is enabled, calls can throw -var module_all_features = new binaryen.Module(); -module_all_features.setFeatures(binaryen.Features.All); +module.setFeatures(binaryen.Features.All); assert( binaryen.getSideEffects( - module.call("test", [], binaryen.i32) + module.call("test", [], binaryen.i32), + module.getFeatures() ) == binaryen.SideEffects.Calls | binaryen.SideEffects.Throws -- cgit v1.2.3