DavidSpickett wrote:
`lldb/include/lldb/Symbol/CompileUnit.h`
```
/// \param[in] lambda
/// The lambda that should be applied to every function. The lambda can
/// return true if the iteration should be aborted earlier.
///
/// \return
/// If the lambda early-exited, this function returns true to
/// propagate the early exit.
virtual bool ForEachExternalModule(
llvm::DenseSet<lldb_private::SymbolFile *> &visited_symbol_files,
llvm::function_ref<bool(Module &)> lambda);
```
I would have guessed true means continue.
Also, changing this didn't break any tests. So it would be nice to cover this
but I don't know how. Would the whole test suite have to be built with modules
enabled?
https://github.com/llvm/llvm-project/pull/75927
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
