[all-commits] [llvm/llvm-project] 580d78: [WebAssembly] Fix resume-only case in Emscripten EH
Heejin Ahn via All-commits
all-commits at lists.llvm.org
Thu Jan 23 18:15:21 PST 2020
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: 580d7838dd08e13dac6caf4ab3142c9381bc7ad0
https://github.com/llvm/llvm-project/commit/580d7838dd08e13dac6caf4ab3142c9381bc7ad0
Author: Heejin Ahn <aheejin at gmail.com>
Date: 2020-01-23 (Thu, 23 Jan 2020)
Changed paths:
M llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp
R llvm/test/CodeGen/WebAssembly/lower-em-exceptions-lpad-only.ll
A llvm/test/CodeGen/WebAssembly/lower-em-exceptions-resume-only.ll
Log Message:
-----------
[WebAssembly] Fix resume-only case in Emscripten EH
Summary:
D72308 incorrectly assumed `resume` cannot exist without a `landingpad`,
which is not true. This sets `Changed` to true whenever we make changes
to a function, including creating a call to `__resumeException` within a
function without a landing pad.
Reviewers: tlively
Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73308
More information about the All-commits
mailing list