aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
diff options
context:
space:
mode:
authorMichael Buch <michaelbuch12@gmail.com>2025-07-03 10:28:49 +0100
committerGitHub <noreply@github.com>2025-07-03 10:28:49 +0100
commit4017dc06e3b5c4b97d8b1089070f88363e0db6f0 (patch)
tree62ec164909d816700edc8e25dfcf863ceccb2a30 /llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
parent58d84a615e9180eeff583a9d30033ba21343550d (diff)
downloadllvm-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