ayermolo added inline comments.
================ Comment at: bolt/include/bolt/Core/DebugData.h:375-377 + if (Optional<uint64_t> DWOId = Unit.getDWOId()) + return *DWOId; + return Unit.getOffset(); ---------------- ayermolo wrote: > dblaikie wrote: > > That seems like a somewhat problematic API - returning two very different > > kinds of data (the DWO_ID or the unit offset) seems quite easy to misuse > > this? > The idea I had behind this APIS is for it to return unique ID representing > the CU. As it applies to .debug_addr. For monolithic case is its offset with > .debug_info for Split Dwarf case is its DWO ID. At least in my head viewed in > that context the return data is the same. It's just something that uniquely > identifies this CU, and logic is encapsulated in it. > @dblaikie What do you think? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122988/new/ https://reviews.llvm.org/D122988 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
