On Fri, Jun 04, 2010 at 08:32:26AM -0700, Ian Lance Taylor wrote:
> Nathan Froyd <[email protected]> writes:
> > Looking at things a little more closely, output_address is exported in
> > output.h. I suppose output_operand should be exported there as well?
>
> Yes, put the declaration there, by output_operand_lossage.
This is what I committed.
-Nathan
* config/i386/i386-protos.h (ix86_print_operand): Declare.
* config/i386/i386.c (ix86_print_operand): Make non-static.
* config/i386/sol2.h (ASM_OUTPUT_CALL): Call ix86_print_operand.
* output.h (output_operand): Declare.
* final.c (output_operand): Make non-static.
Index: final.c
===================================================================
--- final.c (revision 160285)
+++ final.c (working copy)
@@ -220,7 +220,6 @@ static void output_asm_name (void);
static void output_alternate_entry_point (FILE *, rtx);
static tree get_mem_expr_from_op (rtx, int *);
static void output_asm_operand_names (rtx *, int *, int);
-static void output_operand (rtx, int);
#ifdef LEAF_REGISTERS
static void leaf_renumber_regs (rtx);
#endif
@@ -3478,7 +3477,7 @@ mark_symbol_refs_as_used (rtx x)
The meanings of the letters are machine-dependent and controlled
by TARGET_PRINT_OPERAND. */
-static void
+void
output_operand (rtx x, int code ATTRIBUTE_UNUSED)
{
if (x && GET_CODE (x) == SUBREG)
Index: ChangeLog
===================================================================
Index: output.h
===================================================================
--- output.h (revision 160285)
+++ output.h (working copy)
@@ -77,6 +77,9 @@ extern rtx final_scan_insn (rtx, FILE *,
subreg of. */
extern rtx alter_subreg (rtx *);
+/* Print an operand using machine-dependent assembler syntax. */
+extern void output_operand (rtx, int);
+
/* Report inconsistency between the assembler template and the operands.
In an `asm', it's the user's fault; otherwise, the compiler's fault. */
extern void output_operand_lossage (const char *, ...) ATTRIBUTE_PRINTF_1;
Index: config/i386/sol2.h
===================================================================
--- config/i386/sol2.h (revision 160285)
+++ config/i386/sol2.h (working copy)
@@ -145,7 +145,7 @@ along with GCC; see the file COPYING3.
do \
{ \
fprintf (FILE, "\tcall\t"); \
- print_operand (FILE, XEXP (DECL_RTL (FN), 0), 'P'); \
+ ix86_print_operand (FILE, XEXP (DECL_RTL (FN), 0), 'P'); \
fprintf (FILE, "\n"); \
} \
while (0)
Index: config/i386/i386-protos.h
===================================================================
--- config/i386/i386-protos.h (revision 160285)
+++ config/i386/i386-protos.h (working copy)
@@ -60,6 +60,7 @@ extern bool legitimate_pic_operand_p (rt
extern int legitimate_pic_address_disp_p (rtx);
extern void print_reg (rtx, int, FILE*);
+extern void ix86_print_operand (FILE *, rtx, int);
extern bool output_addr_const_extra (FILE*, rtx);
extern void split_di (rtx[], int, rtx[], rtx[]);
Index: config/i386/i386.c
===================================================================
--- config/i386/i386.c (revision 160285)
+++ config/i386/i386.c (working copy)
@@ -11579,7 +11579,7 @@ get_some_local_dynamic_name (void)
; -- print a semicolon (after prefixes due to bug in older gas).
*/
-static void
+void
ix86_print_operand (FILE *file, rtx x, int code)
{
if (code)
