diff options
| author | Heejin Ahn <aheejin@gmail.com> | 2024-01-13 14:55:54 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-01-13 14:55:54 -0800 |
| commit | d871f40deb7a7e6286b6d6b4476f6d30e444075e (patch) | |
| tree | 79606a22d33337d8e1ef1d1853975c3e9f82f4a6 /llvm/lib/Support/CommandLine.cpp | |
| parent | 91b53a2c791c65e17aa80dce7c357ff705d3fd4e (diff) | |
| download | llvm-d871f40deb7a7e6286b6d6b4476f6d30e444075e.zip llvm-d871f40deb7a7e6286b6d6b4476f6d30e444075e.tar.gz llvm-d871f40deb7a7e6286b6d6b4476f6d30e444075e.tar.bz2 | |
[WebAssembly] Use DebugValueManager only when subprogram exists (#77978)
We previously scanned the whole BB for `DBG_VALUE` instruction even when
the program doesn't have debug info, i.e., the function doesn't have a
subprogram associated with it, which can make compilation unnecessarily
slow. This disables `DebugValueManager` when a `DISubprogram` doesn't
exist for a function.
This only reduces unnecessary work in non-debug mode and does not change
output, so it's hard to add a test to test this behavior.
Test changes were necessary because their `DISubprogram`s were not
correctly linked with the functions, so with this PR the compiler
incorrectly assumed the functions didn't have a subprogram and the tests
started to fail.
Fixes https://github.com/emscripten-core/emscripten/issues/21048.
Diffstat (limited to 'llvm/lib/Support/CommandLine.cpp')
0 files changed, 0 insertions, 0 deletions