diff options
author | Heejin Ahn <aheejin@gmail.com> | 2021-08-01 18:30:18 -0700 |
---|---|---|
committer | Heejin Ahn <aheejin@gmail.com> | 2021-08-03 21:03:46 -0700 |
commit | 9bd02c433b5ee0235f263dab4e2f307a5ceb5b0d (patch) | |
tree | f98274596af9105c464a7edffc04540be4f5c466 /llvm/lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp | |
parent | ad253446208a06902d3bc4939ac7a64f8026cc84 (diff) | |
download | llvm-9bd02c433b5ee0235f263dab4e2f307a5ceb5b0d.zip llvm-9bd02c433b5ee0235f263dab4e2f307a5ceb5b0d.tar.gz llvm-9bd02c433b5ee0235f263dab4e2f307a5ceb5b0d.tar.bz2 |
[WebAssembly] Misc. cosmetic changes in EH (NFC)
- Rename `wasm.catch` intrinsic to `wasm.catch.exn`, because we are
planning to add a separate `wasm.catch.longjmp` intrinsic which
returns two values.
- Rename several variables
- Remove an unnecessary parameter from `canLongjmp` and `isEmAsmCall`
from LowerEmscriptenEHSjLj pass
- Add `-verify-machineinstrs` in a test for a safety measure
- Add more comments + fix some errors in comments
- Replace `std::vector` with `SmallVector` for cases likely with small
number of elements
- Renamed `EnableEH`/`EnableSjLj` to `EnableEmEH`/`EnableEmSjLj`: We are
soon going to add `EnableWasmSjLj`, so this makes the distincion
clearer
Reviewed By: tlively
Differential Revision: https://reviews.llvm.org/D107405
Diffstat (limited to 'llvm/lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp')
-rw-r--r-- | llvm/lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/llvm/lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp b/llvm/lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp index ec2380a..2f874bb 100644 --- a/llvm/lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp +++ b/llvm/lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp @@ -38,8 +38,8 @@ cl::opt<bool> " instruction output for test purposes only."), cl::init(false)); -extern cl::opt<bool> EnableEmException; -extern cl::opt<bool> EnableEmSjLj; +extern cl::opt<bool> WasmEnableEmException; +extern cl::opt<bool> WasmEnableEmSjLj; static void removeRegisterOperands(const MachineInstr *MI, MCInst &OutMI); @@ -82,7 +82,8 @@ WebAssemblyMCInstLower::GetGlobalAddressSymbol(const MachineOperand &MO) const { bool InvokeDetected = false; auto *WasmSym = Printer.getMCSymbolForFunction( - F, EnableEmException || EnableEmSjLj, Signature.get(), InvokeDetected); + F, WasmEnableEmException || WasmEnableEmSjLj, Signature.get(), + InvokeDetected); WasmSym->setSignature(Signature.get()); Printer.addSignature(std::move(Signature)); WasmSym->setType(wasm::WASM_SYMBOL_TYPE_FUNCTION); |