This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, ARM, testsuite]
On 21 September 2012 10:47, Matthew Gretton-Dann
<matthew.gretton-dann@linaro.org> wrote:
> On 20 September 2012 23:06, Christophe Lyon <christophe.lyon@linaro.org> wrote:
>> Hi,
>>
>> GCC for ARM does not support compiling in Thumb1 mode and
>> float-abi=hard. But it does not fail unless the program being
>> compiled actually contains a function with parameters and/or a return
>> value.
>>
>> This is a (minor) problem in the testsuite in some configurations.
>>
>> For instance, if I run the testsuite forcing -mthumb (via site.exp)
>> for a GCC configured for float-abi=hard, and a test uses
>> /* { dg-require-effective-target arm_arch_v6_ok } */
>> /* { dg-add-options arm_arch_v6 } */
>>
>> it won't be unresolved since effective-target arm_arch_v6_ok is successful.
>>
>> The attached patch adds a dummy function body in the test such that it fails.
>>
>> Another way of achieving the same result is by making sure that the
>> relevant tests use
>> arm_arch_v6_multilib
>> instead of
>> arm_arch_v6_ok
>>
>> even if the test is not intended to be executed.
>>
>> OK?
>
> [I'm not a maintainer]
>
> You could argue that as the test is checking for just ARMv6, but then
> uses ARMv6+VFPv2 features - and so it going wrong is to be expected
> :-).
>
No: what I have in mind is a test that doesn't need VFP by itself, but
which happens to become not-compilable if one forces float-abi=hard
and thumb when running the testsuite.
I'd like that the test is UNSUPPORTED rather than FAIL.
Christophe.