[all-commits] [llvm/llvm-project] 296861: [WebAssembly] Fix delegate's argument computation
Heejin Ahn via All-commits
all-commits at lists.llvm.org
Thu Feb 11 21:58:00 PST 2021
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 2968611fdaff215ff1e0064a3c147229211edf9c
https://github.com/llvm/llvm-project/commit/2968611fdaff215ff1e0064a3c147229211edf9c
Author: Heejin Ahn <aheejin at gmail.com>
Date: 2021-02-11 (Thu, 11 Feb 2021)
Changed paths:
M llvm/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyInstPrinter.cpp
M llvm/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyInstPrinter.h
M llvm/lib/Target/WebAssembly/WebAssemblyCFGStackify.cpp
M llvm/test/CodeGen/WebAssembly/cfg-stackify-eh.ll
Log Message:
-----------
[WebAssembly] Fix delegate's argument computation
I previously assumed `delegate`'s immediate argument computation
followed a different rule than that of branches, but we agreed to make
it the same
(https://github.com/WebAssembly/exception-handling/issues/146). This
removes the need for a separate `DelegateStack` in both CFGStackify and
InstPrinter.
When computing the immediate argument, we use a different function for
`delegate` computation because in MIR `DELEGATE`'s instruction's
destination is the destination catch BB or delegate BB, and when it is a
catch BB, we need an additional step of getting its corresponding `end`
marker.
Reviewed By: tlively, dschuff
Differential Revision: https://reviews.llvm.org/D96525
More information about the All-commits
mailing list