On 26 September 2014 04:25, Thomas Preud'homme <thomas.preudho...@arm.com> wrote: >> From: Christophe Lyon [mailto:christophe.l...@linaro.org] >> Sent: Thursday, September 25, 2014 10:08 PM >> > >> While attempting to try this, I noticed that more precisely the test >> is currently UNSUPPORTED on aarch64_be, >> which is because check_effective_target_bswap only accepts istarget >> aarch64-*-*. > > Ah yes, of course. > >> >> I didn't try yet to change it into istarget aarch64*-*-*. > > It should probably be added no matter the result anyway, since this target > has bswap instructions. >
Fixing check_effective_target_bswap to accept aarch64*-*-* makes the test pass, so we should submit that patch. I tried the other change you suggested, but it seem that scan-tree-dump-times only matched 3 times. I did this in a bit of a hurry though, so I may have done something wrong. I'm not sure when I have time to look at that again, so I prefer to give this little feedback now :-) Christophe. > Best regards, > > Thomas > > >
