[PATCH][1/n] LTO option handling reorg
Joseph S. Myers
joseph@codesourcery.com
Tue Oct 25 16:44:00 GMT 2011
On Tue, 25 Oct 2011, Richard Guenther wrote:
> Joseph, does this look like a sensible use of the common
> machinery? Do we want the init from COLLECT_GCC_OPTIONS
> in opts-common.c instead?
Certainly there should be a single function to process COLLECT_GCC_OPTIONS
into an array of strings, even if only this one place needs then to
convert them to logical options. And that single function should actually
work properly with the quoting logic used in gcc.c to generate
COLLECT_GCC_OPTIONS - your code appears to ignore any possibility of '\''
being used for single-quotes in strings, and collect2.c:extract_string,
which is rather closer to what I'd expect, fails to reset backquote to 0
after processing the \'.
Note that decode_cmdline_options_to_array expects the array to start with
argv[0], the program name (though it's OK to have NULL there).
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Gcc-patches
mailing list