jankratochvil added inline comments.
================ Comment at: lldb/source/Host/common/DebugInfoD.cpp:39 + +llvm::Expected<std::string> findSource(UUID buildID, const std::string &path) { + if (!buildID.IsValid()) ---------------- `const UUID &buildID` as it is even bigger (40 bytes) than `std::string` (32 bytes). ================ Comment at: lldb/source/Host/common/DebugInfoD.cpp:56 + // return 0; // continue + // }); + ---------------- Excessive leftover comment. ================ Comment at: lldb/source/Host/common/DebugInfoD.cpp:59 + char *cache_path = nullptr; + int rc = debuginfod_find_source(client, buildID.GetBytes().data(), + buildID.GetBytes().size(), path.c_str(), ---------------- Here it will contact the server even if the binary does not contain any build-id - LLDB then generates UUID as 4 bytes long one: ``` // Use 4 bytes of crc from the .gnu_debuglink section. u32le data(gnu_debuglink_crc); uuid = UUID::fromData(&data, sizeof(data)); ``` That is a needless performance regression. I sure do not like making such decision on the LLDB side. Maybe libdebuginfod could rather make such optimization - IMO as Frank Eigler. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75750/new/ https://reviews.llvm.org/D75750 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits