[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