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

[Jonas Devlieghere via lldb-commits]

================
@@ -315,6 +315,22 @@ 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.
+  ///
+  /// \param[in] lookup_infos
+  ///     The vector of lookup infos of the function we are looking for.
+  ///
+  /// \param[out] sc_list
+  ///     A symbol context list that gets filled in with all of the
+  ///     matches.
----------------
JDevlieghere wrote:

The Doxygen comment is missing arguments. I personally think it adds little value compared to the cost of keeping it in sync as the function will eventually change, so I would just omit it.

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