[all-commits] [llvm/llvm-project] 561234: [lldb][Expression] Remove m_found_function_with_ty...

Michael Buch via All-commits all-commits at lists.llvm.org
Wed May 28 08:37:09 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 561234d21cfca899ca898dd8b9c96f825a3cc1f6
      https://github.com/llvm/llvm-project/commit/561234d21cfca899ca898dd8b9c96f825a3cc1f6
  Author: Michael Buch <michaelbuch12 at gmail.com>
  Date:   2025-05-28 (Wed, 28 May 2025)

  Changed paths:
    M lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.h
    M lldb/source/Plugins/ExpressionParser/Clang/NameSearchContext.h

  Log Message:
  -----------
  [lldb][Expression] Remove m_found_function_with_type_info in favour of boolean return (#141774)

I've been skimming this code while investigating a bug around module
lookup and this looked like something we could clean up. We don't need
to be carrying around state inside of `NameSearchContext` to tell us to
look into modules. We can signal this via a boolean return from
`LookupFunction`.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list