[PATCH] D14354: Add new compiler flag to enable the generation of dwarf accelerator tables

[Paul Robinson via cfe-commits]

probinson added a comment.

In http://reviews.llvm.org/D14354#282988, @tberghammer wrote:

> In http://reviews.llvm.org/D14354#282870, @probinson wrote:
>
> > So, currently you get accel tables by default for platforms that "tune" for LLDB, currently Darwin and FreeBSD.
> >  Are you wanting to use LLDB on another platform, or did you want accel tables for a different debugger?
> >
> > (Eventually I will have a chance to expose the "tuning" up through Clang, and if you're running LLDB on another platform, that will be the easier way to get what you want.)
>
>
> We are using LLDB for Android (as part of Android Studio) and some of us also use if for Linux (I hope we can get higher adaptation soon).


Okay.  I'll try to get the tuning-in-Clang task moved higher up my list, because if you had that you wouldn't need this patch.


================
Comment at: lib/Driver/Tools.cpp:3880
@@ +3879,3 @@
+  // -gdwarf-accel-tables should turn on -g and enable the genereation of the
+  // dwarf acceleration tables in the backend.
+  if (Args.hasArg(options::OPT_gdwarf_accel_tables)) {
----------------
This is not how other similar options work (-ggnu-pubnames, -gdwarf-aranges).  Do you have a particular reason why this one should imply -g?


http://reviews.llvm.org/D14354