Hi Richard, > Did you test this on a thumb1 target? It seems to me that the target parts > that you've > removed were likely related to that. In fact, I don't see why this test > would need to be changed at all.
The testcase explicitly forces a Thumb-2 target (arm_arch_v6t2). The patterns were wrong for Thumb-2 indeed, and the testcase was explicitly testing for this. There is a separate builtin-bswap-2.c for Thumb-1 target (arm_arch_v6m). Cheers, Wilco