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
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
