Re: RFA: Add "dg-add-options c99_runtime"

[Janis Johnson <janis187 at us dot ibm dot com>]

On Wed, 2007-09-19 at 13:44 +0100, Richard Sandiford wrote:
>  The patch therefore
> adds a new directive:
> 
>   /* { dg-add-options c99_runtime } */
> 
> to add the necessary target-specific flags to the current dg-options,
> if any.  (Note that the dg-extra-tool-flags variable is always defined,
> regardless of whether the test has dg-options.)
> 
> More generally, dg-add-options is a new directive that takes a list of
> features as arguments, with c99_runtime being the only such feature
> at the moment.  It relies on a function called add_options_for_<feature>
> to provide the necessary flags, much like the current
> check_effective_target_<feature> arrangement for target selectors.
> 
> I think this part qualifies as a clean-up in its own right, which
> is why I've split it out.  Bootstrapped & regression-tested on
> x86_64-linux-gnu.  Also regression-tested on mipsisa64-elfoabi.
> As a sanity check, I ran the modified tests on x86_64-linux-gnu with
> "--target sparc-sun-solaris2.10" and "--target powerpc-apple-darwin".
> The tests were compiled with the expected flags both times, although
> the compilation expectedly failed for Darwin because x86_64-linux-gnu-gcc
> doesn't recognise -mmacosx-version-min.  OK to install?

Yes; this is a good idea.

Janis