diff options
| author | Heejin Ahn <aheejin@gmail.com> | 2021-08-24 20:40:21 -0700 |
|---|---|---|
| committer | Heejin Ahn <aheejin@gmail.com> | 2021-08-25 15:45:38 -0700 |
| commit | 2f88a30ca660ca60b460a08d33eef3ab9517ee28 (patch) | |
| tree | 24bbca276a8a5814528578dec44c0d2abaf23f6f /llvm/lib/CodeGen/MachineScheduler.cpp | |
| parent | 18da6db838c5ba32c326e9d3cd8e8bd35408e809 (diff) | |
| download | llvm-2f88a30ca660ca60b460a08d33eef3ab9517ee28.zip llvm-2f88a30ca660ca60b460a08d33eef3ab9517ee28.tar.gz llvm-2f88a30ca660ca60b460a08d33eef3ab9517ee28.tar.bz2 | |
[WebAssembly] Extract longjmp handling in EmSjLj to a function (NFC)
Emscripten SjLj and (soon-to-be-added) Wasm SjLj transformation share
many steps:
1. Initialize `setjmpTable` and `setjmpTableSize` in the entry BB
2. Handle `setjmp` callsites
3. Handle `longjmp` callsites
4. Cleanup and update SSA
1, 3, and 4 are identical for Emscripten SjLj and Wasm SjLj. Only the
step 2 is different. This CL extracts the current Emscripten SjLj's
longjmp callsites handling into a function. The reason to make this a
separate CL is, without this, the diff tool cannot compare things well
in the presence of moved code and added code in the followup Wasm SjLj
CL, and it ends up mixing them together, making the diff unreadable.
Also fixes some typos and variable names. So far we've been calling the
buffer argument to `setjmp` and `longjmp` `jmpbuf`, but the name used in
the man page for those functions is `env`, so updated them to be
consistent.
Reviewed By: tlively
Differential Revision: https://reviews.llvm.org/D108728
Diffstat (limited to 'llvm/lib/CodeGen/MachineScheduler.cpp')
0 files changed, 0 insertions, 0 deletions