[PATCH][ARM] Fix gcc.target/arm/builtin-bswap16-1.c

Kyrill Tkachov kyrylo.tkachov@foss.arm.com
Fri Jun 3 08:30:00 GMT 2016


Hi all,

The test gcc.target/arm/builtin-bswap16-1.c refuses to compile when testing a toolchain configured with
--with-mode=thumb --with-float=hard and an architecture that supports Thumb2.
This is because the test explicitly sets the -march option to armv6 and we get an error complaining
about Thumb1 used with the hard-float ABI.

The proposed solution in this patch is to bump the architecture to armv6t2 so that it uses Thumb2 when
-mthumb is used.

But we don't want to lose Thumb1 test coverage. So this patch moves the actual C code into a separate
.x file and includes it in two different tests, each testing Thumb1 or Thumb2.

The new test passes and builtin-bswap16-1.c also now passes rather than complaining about the float ABI.

Ok for trunk?

Thanks,
Kyrill

2016-06-03  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

     * gcc.target/arm/builtin-bswap16-1.c: Add -mfloat-abi=soft
     and -mthumb to dg-options.  Include builtin-bswap16.x.
     * gcc.target/arm/builtin-bswap16: New file.
     * gcc.target/arm/builtin-bswap16-2.c: New test.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: arm-bswap-test.patch
Type: text/x-patch
Size: 1983 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20160603/7206e7e7/attachment.bin>


More information about the Gcc-patches mailing list