[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