This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Add hasSideEffects flag to catch instructions
ClosedPublic

Authored by aheejin on Aug 17 2018, 1:26 PM.

Details

Summary

catch instruction certainly has rather huge side effects and the flag
was missing. At the moment this does not change any unit tests we
currently have.

Diff Detail

Event Timeline

aheejin created this revision.Aug 17 2018, 1:26 PM
dschuff accepted this revision.Aug 22 2018, 11:10 AM
This revision is now accepted and ready to land.Aug 22 2018, 11:10 AM
This revision was automatically updated to reflect the committed changes.