diff options
author | Michael Buch <michaelbuch12@gmail.com> | 2025-07-03 10:28:49 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-07-03 10:28:49 +0100 |
commit | 4017dc06e3b5c4b97d8b1089070f88363e0db6f0 (patch) | |
tree | 62ec164909d816700edc8e25dfcf863ceccb2a30 /llvm/lib/Bitcode/Writer/BitcodeWriter.cpp | |
parent | 58d84a615e9180eeff583a9d30033ba21343550d (diff) | |
download | llvm-4017dc06e3b5c4b97d8b1089070f88363e0db6f0.zip llvm-4017dc06e3b5c4b97d8b1089070f88363e0db6f0.tar.gz llvm-4017dc06e3b5c4b97d8b1089070f88363e0db6f0.tar.bz2 |
[lldb][Commands][NFC] image lookup: remove unused local variable (#146554)
The `current_module` pointer here was never set, but we check it when
looping over the `target_modules` list. Presumably the intention was to
avoid calling `LookupInModule` if we already found the type in the
current module. This patch removes this `current_module`. If we decide
the output below is not what the user should see, we can revisit the
implementation.
Current output:
```
(lldb) im loo -vt Foo --all
Best match found in /Users/jonas/Git/llvm-worktrees/llvm-project/a.out:
id = {0x00000037}, name = "Foo", byte-size = 1, decl = foo.cpp:1, compiler_type = "struct Foo {
}"
1 match found in /Users/jonas/Git/llvm-worktrees/llvm-project/a.out:
id = {0x00000037}, name = "Foo", byte-size = 1, decl = foo.cpp:1, compiler_type = "struct Foo {
}"
```
which seems fine.
Note, there can be multiple matches *within* the current module, so if
we did the naive thing of skipping the `current_module` when printing
with `--all`, then we would miss some matches.
Diffstat (limited to 'llvm/lib/Bitcode/Writer/BitcodeWriter.cpp')
0 files changed, 0 insertions, 0 deletions