diff options
| author | Wouter van Oortmerssen <aardappel@gmail.com> | 2022-02-22 15:45:49 -0800 |
|---|---|---|
| committer | Wouter van Oortmerssen <aardappel@gmail.com> | 2022-02-22 16:14:26 -0800 |
| commit | 2368f18eb305ae9d5a4f2110c048e5daf5007992 (patch) | |
| tree | bae3e4966a20c67c66399c88045fdf8fa329ac93 /llvm/lib/Object/MachOObjectFile.cpp | |
| parent | 8b9f42b61b33ec1493e6d71d0240da6dfc847be2 (diff) | |
| download | llvm-2368f18eb305ae9d5a4f2110c048e5daf5007992.zip llvm-2368f18eb305ae9d5a4f2110c048e5daf5007992.tar.gz llvm-2368f18eb305ae9d5a4f2110c048e5daf5007992.tar.bz2 | |
[WebAssembly] Fixed AsmPrinter not emitting .functype for intrinsics
Intrinsics like `memset` were not emitted as `.functype` because
WebAssemblyAsmPrinter::emitExternalDecls explicitly skips symbols
that are isIntrinsic. Removing that check doesn't work, since the symbol
from the module refers to a 4-argument `llvm.memset.p0i8.i32` rather
than the 3-argument `memset` symbol referenced in the call.
Our `WebAssemblyMCLowerPrePass` however does collect the
`memset` symbol, so the current solution is as simple as emitting
`.functype` for those.
Fixes: https://github.com/llvm/llvm-project/issues/53712
Differential Revision: https://reviews.llvm.org/D120365
Diffstat (limited to 'llvm/lib/Object/MachOObjectFile.cpp')
0 files changed, 0 insertions, 0 deletions