JDevlieghere marked an inline comment as done. JDevlieghere added inline comments.
================ Comment at: lldb/source/Expression/IRExecutionUnit.cpp:785 + auto get_external_load_address = + [target, &symbol_was_missing_weak]( + lldb::addr_t &load_address, lldb::addr_t &best_internal_load_address, ---------------- shafik wrote: > It feels like the lambda should really be a small `class` that tracks the > state of `load_address` and `best_internal_load_address` over multiple calls > and then the user can access the state. > > The lambda is doing a lot of logic and there is state being carried across > calls and that feels like we have crossed the boundary and a class would be > better. Thanks, I really like that idea. Updated the diff accordingly. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107206/new/ https://reviews.llvm.org/D107206 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
