mstorsjo added a comment. In D134585#3814463 <https://reviews.llvm.org/D134585#3814463>, @DavidSpickett wrote:
>> MSVC/link.exe doesn't write symbols into linked PE images at all. > > So by the time we get to a debugger, it's not an issue anyway. Yep, most of these patches about symbol table handling doesn't make any difference for the MSVC ecosystem usage at all (although some of these patches fix other generic windows-specific bugs noticed too). > Then this LGTM. Thanks! Can you mark it formally approved too? :-) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134585/new/ https://reviews.llvm.org/D134585 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
