[PATCH] D67373: Don't emit .gnu_pubnames when tuning for LLDB

This revision was automatically updated to reflect the committed changes. Closed by commit rL371530: Don't emit .gnu_pubnames when tuning for LLDB. (authored by adrian, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://review

[PATCH] D67373: Don't emit .gnu_pubnames when tuning for LLDB

dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Seems good to me - thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67373/new/ https://reviews.llvm.org/D67373 _

[PATCH] D67373: Don't emit .gnu_pubnames when tuning for LLDB

JDevlieghere added inline comments. Comment at: clang/lib/Driver/ToolChains/Clang.cpp:3400 if (DwarfFission != DwarfFissionKind::None || - DebuggerTuning == llvm::DebuggerKind::LLDB || (PubnamesArg && checkDebugInfoOption(PubnamesArg, Args, D, TC))) ---

[PATCH] D67373: Don't emit .gnu_pubnames when tuning for LLDB

aprantl added a comment. In D67373#1663924 , @dblaikie wrote: > Have you got a link to the original thread where this was discussed/I > mentioned it? Want to page in some context to double-check if I had any ideas > that might've let this simplify things

[PATCH] D67373: Don't emit .gnu_pubnames when tuning for LLDB

dblaikie added a comment. Have you got a link to the original thread where this was discussed/I mentioned it? Want to page in some context to double-check if I had any ideas that might've let this simplify things. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revie

[PATCH] D67373: Don't emit .gnu_pubnames when tuning for LLDB

aprantl created this revision. aprantl added reviewers: jasonmolenda, dblaikie, friss, emaste, JDevlieghere. Herald added a project: clang. LLDB reads the various `.apple*` accelerator tables which should make `.gnu_pubnames` redundant. This changes the Clang driver to no longer pass `-ggnu-pubn