[Lldb-commits] [lldb] [lldb] Refactor LookupInfo object to be per-language (PR #168797)

[Felipe de Azevedo Piovezan via lldb-commits]

================
@@ -315,6 +315,15 @@ class Module : public std::enable_shared_from_this<Module>,
                      const ModuleFunctionSearchOptions &options,
                      SymbolContextList &sc_list);
 
+  /// Find functions by a vector of lookup infos.
+  ///
+  /// If the function is an inlined function, it will have a block,
+  /// representing the inlined function, and the function will be the
+  /// containing function.  If it is not inlined, then the block will be NULL.
+  void FindFunctions(const std::vector<LookupInfo> &lookup_infos,
----------------
felipepiovezan wrote:

we should never use `const vector` in LLVM, instead we should always use `llvm::ArrayRef`, as it works with more kinds of containers and, more importantly, it works with a single element of the value type.

https://github.com/llvm/llvm-project/pull/168797