On 5 November 2014 12:08, Christophe Lyon <christophe.l...@linaro.org> wrote: > On 31 October 2014 18:15, Ramana Radhakrishnan > <ramana....@googlemail.com> wrote: >> On Wed, Oct 29, 2014 at 1:22 PM, Christophe Lyon >> <christophe.l...@linaro.org> wrote: >>> Hi, >>> >>> Following discussions after Thomas's patches improving bswap support >>> https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01279.html >>> >>> I noticed that: >>> * the associated tests weren't executed on aarch64_be >>> * ARM targets older than v6 do not support the needed instructions. >>> >>> The attached patch changes check_effective_target_bswap(): >>> - accept aarch64*-*-* instead of aarch64-*-* >>> - when target is arm*-*-*, check __ARM_ARCH >= 6 >>> >>> 2014-10-29 Christophe Lyon <christophe.l...@linaro.org> >>> >>> * lib/target-supports.exp (check_effective_target_bswap): Update >>> conditions for AArch64 and ARM targets. >>> >>> OK? >> >> The ARM (AArch32) changes are ok. >> >> Ramana >> > Thank you Ramana. > > Marcus, I believe the AArch64 part is obvious. OK?
Yep, it looks obvious to me. /Marcus