[PATCH,RFC 5/9] gas: add new command line option --scfi[=all,none]

[Nick Clifton]

Hi Indu,

> When the command line option --scfi (default is --scfi=all) is passed to
> the GNU assembler, it will synthesize DWARF call frame information (CFI)
> for the assembly.

> Also document the option.

Please could you also add an entry in the gas/NEWS file about this new feature.

> +# ifdef TARGET_USE_SCFI
> +  fprintf (stream, _("\
> +  --scfi=[all,none]	  synthesize DWARF CFI for hand-written asm (not inline)\n\
> +			  (default --scfi=all)\n"));
> +# endif

I was wondering how the code would detect inline assembler.
Presumably it looks for the #NOAPP preprocessor directive ?
Or is the point of the comment in the above delta that the
user should not use this new option in conjunction with actual
compiled code ?  Either way I think that you need to extend
the documentation to include describing what happens when
compiled code and this option are used together.

Also - I have not checked this yet - if this option is enabled
and no CFI statements are generated (or maybe if an incomplete
sequence is generated) - presumably because of badly written
code - will there be an error message generated ?

Cheers
   Nick