[all-commits] [llvm/llvm-project] 21f7b3: [WebAssembly] Fix landingpad-only case in Emscript...
Heejin Ahn via All-commits
all-commits at lists.llvm.org
Mon Jan 6 17:06:55 PST 2020
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: 21f7b362095f7497aa87ccc334fe18bbc072854c
https://github.com/llvm/llvm-project/commit/21f7b362095f7497aa87ccc334fe18bbc072854c
Author: Heejin Ahn <aheejin at gmail.com>
Date: 2020-01-06 (Mon, 06 Jan 2020)
Changed paths:
M llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp
A llvm/test/CodeGen/WebAssembly/lower-em-exceptions-lpad-only.ll
Log Message:
-----------
[WebAssembly] Fix landingpad-only case in Emscripten EH
Summary:
Previously we didn't set `Changed` to true when there are only landing
pads but not invokes. This fixes it and we set `Changed` to true
whenever we have landing pads. (There can't be invokes without landing
pads, so that case is covered too)
The test case for this has to be a separate file because this pass is a
`ModulePass` and `Changed` is computed based on the whole module.
Reviewers: tlively
Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D72308
More information about the All-commits
mailing list