[PATCH] D8506: Set a debugger "target" to guide DWARF choices

Eric Christopher echristo at gmail.com
Tue Jul 14 23:11:42 PDT 2015


echristo accepted this revision.
echristo added a comment.

I'm a bit unhappy with the TLS op tuning, but I don't have any better ideas. It's less tuning and more correctness, but...

Anyhow, an inline comment about one of your FIXMEs, otherwise LGTM.

Thanks for all of the iterations here.

-eric


================
Comment at: test/DebugInfo/X86/debugger-tune.ll:4
@@ +3,3 @@
+; We use existence of the .debug_pubnames section to distinguish the GDB case.
+; FIXME: Find something to distinguish LLDB from SCE.
+
----------------
Are you using the same accelerator tables?


http://reviews.llvm.org/D8506







More information about the llvm-commits mailing list