On Mon, 28 Jul 2014, Herman, Andrei wrote: > Please find attached the fixed patch files for this change: > 1. Add command line option -fforce-dwarf-lexical-blocks. > 2. Support flag_force_dwarf_blocks in C.
The front-end parts of these patches are OK with the changes indicated below if the non-front-end parts are approved. > + /* The -fforce-dwarf-lexical-blocks option is only relevant when debug > + info is in DWARF4 format. */ > + if (flag_force_dwarf_blocks) > + { > + if (write_symbols != DWARF2_DEBUG) > + flag_force_dwarf_blocks = 0; > + if (write_symbols == DWARF2_DEBUG && dwarf_version < 4) > + { > + inform (input_location, > + "-fforce-dwarf-lexical-blocks is only supported with " > + "DWARF4 debug format"); > + flag_force_dwarf_blocks = 0; > + } > + } If it's deliberate not to give the diagnostic for write_symbols != DWARF2_DEBUG, please add a comment explaining why the diagnostic is not given in that case. > +fforce-dwarf-lexical-blocks > +C C++ Var(flag_force_dwarf_blocks) > +Force generation of lexical blocks in DWARF output As previously noted, should also be enabled for ObjC and ObjC++ unless there is a clear reason not to do so. > +/* Information about a statement list created for a label (is_label=true) > + or for a forced c99 scope. The -fforce-dwarf-lexical-blocks will "The -fforce-dwarf-lexical-blocks option". -- Joseph S. Myers jos...@codesourcery.com