2017-05-31 01:32:31 +00:00
|
|
|
From 0343a584d6b5128908eabf1db43c70bed7114989 Mon Sep 17 00:00:00 2001
|
|
|
|
|
From: Max Filippov <jcmvbkbc@gmail.com>
|
|
|
|
|
Date: Sun, 28 May 2017 19:56:56 -0700
|
|
|
|
|
Subject: [PATCH] gcc: xtensa: fix fprintf format specifiers
|
|
|
|
|
|
|
|
|
|
HOST_WIDE_INT may not be long as assumed in print_operand and
|
|
|
|
|
xtensa_emit_call. Use HOST_WIDE_INT_PRINT_DEC/HOST_WIDE_INT_PRINT_HEX
|
|
|
|
|
format strings instead of %ld/0x%lx. This fixes incorrect assembly code
|
|
|
|
|
generation by the compiler running on armhf host.
|
|
|
|
|
|
|
|
|
|
2017-05-28 Max Filippov <jcmvbkbc@gmail.com>
|
|
|
|
|
gcc/
|
|
|
|
|
* config/xtensa/xtensa.c (xtensa_emit_call): Use
|
|
|
|
|
HOST_WIDE_INT_PRINT_HEX instead of 0x%lx format string.
|
|
|
|
|
(print_operand): Use HOST_WIDE_INT_PRINT_DEC instead of %ld
|
|
|
|
|
format string.
|
|
|
|
|
|
|
|
|
|
Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
|
|
|
|
|
---
|
2017-12-02 20:44:39 +00:00
|
|
|
gcc/config/xtensa/xtensa.c | 10 +++++-----
|
2017-05-31 01:32:31 +00:00
|
|
|
1 file changed, 5 insertions(+), 5 deletions(-)
|
|
|
|
|
|
|
|
|
|
--- a/gcc/config/xtensa/xtensa.c
|
|
|
|
|
+++ b/gcc/config/xtensa/xtensa.c
|
2017-12-02 20:44:39 +00:00
|
|
|
@@ -1773,7 +1773,7 @@
|
2017-05-31 01:32:31 +00:00
|
|
|
rtx tgt = operands[callop];
|
|
|
|
|
|
|
|
|
|
if (GET_CODE (tgt) == CONST_INT)
|
|
|
|
|
- sprintf (result, "call8\t0x%lx", INTVAL (tgt));
|
|
|
|
|
+ sprintf (result, "call8\t" HOST_WIDE_INT_PRINT_HEX, INTVAL (tgt));
|
|
|
|
|
else if (register_operand (tgt, VOIDmode))
|
|
|
|
|
sprintf (result, "callx8\t%%%d", callop);
|
|
|
|
|
else
|
2017-12-02 20:44:39 +00:00
|
|
|
@@ -2348,14 +2348,14 @@
|
2017-05-31 01:32:31 +00:00
|
|
|
|
|
|
|
|
case 'L':
|
|
|
|
|
if (GET_CODE (x) == CONST_INT)
|
|
|
|
|
- fprintf (file, "%ld", (32 - INTVAL (x)) & 0x1f);
|
|
|
|
|
+ fprintf (file, HOST_WIDE_INT_PRINT_DEC, (32 - INTVAL (x)) & 0x1f);
|
|
|
|
|
else
|
|
|
|
|
output_operand_lossage ("invalid %%L value");
|
|
|
|
|
break;
|
|
|
|
|
|
|
|
|
|
case 'R':
|
|
|
|
|
if (GET_CODE (x) == CONST_INT)
|
|
|
|
|
- fprintf (file, "%ld", INTVAL (x) & 0x1f);
|
|
|
|
|
+ fprintf (file, HOST_WIDE_INT_PRINT_DEC, INTVAL (x) & 0x1f);
|
|
|
|
|
else
|
|
|
|
|
output_operand_lossage ("invalid %%R value");
|
|
|
|
|
break;
|
2017-12-02 20:44:39 +00:00
|
|
|
@@ -2369,7 +2369,7 @@
|
2017-05-31 01:32:31 +00:00
|
|
|
|
|
|
|
|
case 'd':
|
|
|
|
|
if (GET_CODE (x) == CONST_INT)
|
|
|
|
|
- fprintf (file, "%ld", INTVAL (x));
|
|
|
|
|
+ fprintf (file, HOST_WIDE_INT_PRINT_DEC, INTVAL (x));
|
|
|
|
|
else
|
|
|
|
|
output_operand_lossage ("invalid %%d value");
|
|
|
|
|
break;
|
2017-12-02 20:44:39 +00:00
|
|
|
@@ -2438,7 +2438,7 @@
|
2017-05-31 01:32:31 +00:00
|
|
|
else if (GET_CODE (x) == MEM)
|
|
|
|
|
output_address (XEXP (x, 0));
|
|
|
|
|
else if (GET_CODE (x) == CONST_INT)
|
|
|
|
|
- fprintf (file, "%ld", INTVAL (x));
|
|
|
|
|
+ fprintf (file, HOST_WIDE_INT_PRINT_DEC, INTVAL (x));
|
|
|
|
|
else
|
|
|
|
|
output_addr_const (file, x);
|
|
|
|
|
}
|
