[PATCH] testsuite: Enable fenv effective-targets for C++

Jeff Law law@redhat.com
Tue Aug 25 19:27:00 GMT 2020


On Thu, 2020-07-30 at 11:23 +0100, Jonathan Wakely via Gcc-patches wrote:
> I recently wanted to use { dg-require-effective-target fenv } in a
> libstdc++ test, but it uses -std=gnu99 which is only valid for C.
> 
> This allows C++ tests to use the fenv and fenv_exceptions
> effective-target keywords.
> 
> gcc/testsuite/ChangeLog:
> 
> 	* lib/target-supports.exp (check_effective_target_fenv): Use
> 	-std=gnu++11 instead of -std=gnu99 for C++ tests.
> 	(check_effective_target_fenv_exceptions): Likewise.
> 
> 
> I don't actually need to use the 'fenv' target now, but does this seem
> like a reasonable change anyway?
It seems reasonable to me.  Selecting the right option automatically seems better
than any of the alternatives.

jeff



More information about the Gcc-patches mailing list