On 3 December 2013 19:42, Cary Coutant <ccout...@google.com> wrote: >> This patches gen_compile_unit_die to use the DW_LANG_D DWARF language >> code for D. Is in relation to some other D language fixes that are >> going to be submitted to gdb. > > Is this for a private front end? I'm not aware of any front ends that > set the language name to "GNU D". > > Since it's so trivial, though, I have no problem with this patch for > Stage 3 -- if you do have a separate front end that sets that language > string, then it's arguably a bug fix. If this patch is preparation for > more substantial changes to the GCC tree, however, I suspect you're > going to need to wait for Stage 1 to reopen anyway. > > So, if this is a standalone patch, it's OK, but you also need a ChangeLog > entry. > > -cary
The frontend isn't private, but is currently external to GCC. I've had plans to get the frontend merged for some time now. And was adviced last time I submitted the code for review to send patches that can be merged into GCC prior to re-submitting the frontend - which as you have already said will have to wait for Stage 1 to reopen. Will make a changelog entry for the patch. Regards Iain.
