Skip to content

Commit ebdad26

Browse files
committed
arm: Update fp16-aapcs-[24].c after insn_propagation patch
These tests used to generate: bl swap ldr r2, [sp, #4] mov r0, r2 @ __fp16 but g:9d20529d94b23275885f380d155fe8671ab5353a means that we can load directly into r0: bl swap ldrh r0, [sp, #4] @ __fp16 This patch updates the tests to "defend" this change. While there, the scans include: mov\tr1, r[03]} But if the spill of r2 occurs first, there's no real reason why r2 couldn't be used as the temporary, instead r3. The patch tries to update the scans while preserving the spirit of the originals. gcc/testsuite/ * gcc.target/arm/fp16-aapcs-2.c: Expect the return value to be loaded directly from the stack. Test that the swap generates two moves out of r0/r1 and two moves in. * gcc.target/arm/fp16-aapcs-4.c: Likewise.
1 parent 2ee70c9 commit ebdad26

File tree

2 files changed

+10
-6
lines changed

2 files changed

+10
-6
lines changed

gcc/testsuite/gcc.target/arm/fp16-aapcs-2.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@ F (__fp16 a, __fp16 b, __fp16 c)
1616
return c;
1717
}
1818

19-
/* { dg-final { scan-assembler-times {mov\tr[0-9]+, r[0-2]} 3 } } */
20-
/* { dg-final { scan-assembler-times {mov\tr1, r[03]} 1 } } */
21-
/* { dg-final { scan-assembler-times {mov\tr0, r[0-9]+} 2 } } */
19+
/* The swap must include two moves out of r0/r1 and two moves in. */
20+
/* { dg-final { scan-assembler-times {mov\tr[0-9]+, r[01]} 2 } } */
21+
/* { dg-final { scan-assembler-times {mov\tr[01], r[0-9]+} 2 } } */
22+
/* c should be spilled around the call. */
23+
/* { dg-final { scan-assembler {str\tr2, ([^\n]*).*ldrh\tr0, \1} { target arm_little_endian } } } */

gcc/testsuite/gcc.target/arm/fp16-aapcs-4.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@ F (__fp16 a, __fp16 b, __fp16 c)
1616
return c;
1717
}
1818

19-
/* { dg-final { scan-assembler-times {mov\tr[0-9]+, r[0-2]} 3 } } */
20-
/* { dg-final { scan-assembler-times {mov\tr1, r[03]} 1 } } */
21-
/* { dg-final { scan-assembler-times {mov\tr0, r[0-9]+} 2 } } */
19+
/* The swap must include two moves out of r0/r1 and two moves in. */
20+
/* { dg-final { scan-assembler-times {mov\tr[0-9]+, r[01]} 2 } } */
21+
/* { dg-final { scan-assembler-times {mov\tr[01], r[0-9]+} 2 } } */
22+
/* c should be spilled around the call. */
23+
/* { dg-final { scan-assembler {str\tr2, ([^\n]*).*ldrh\tr0, \1} { target arm_little_endian } } } */

0 commit comments

Comments
 (0)