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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits