Re: [Patch, microblaze]: Add TARGET_ASM_OUTPUT_MI_THUNK to support varargs thunk

[Michael Eager]

On 03/18/13 05:49, David Holsgrove wrote:

Changelog

2013-03-18  David Holsgrove <david.holsgr...@xilinx.com>

  * gcc/config/microblaze/microblaze.c: Add microblaze_asm_output_mi_thunk
    and define TARGET_ASM_OUTPUT_MI_THUNK and TARGET_ASM_CAN_OUTPUT_MI_THUNK

Sorry it has taken so long to review this patch.

The gcc microblaze-xilinx-elf build with this patch fails here:

+microblaze_asm_output_mi_thunk (FILE *file, tree thunk_fndecl ATTRIBUTE_UNUSED,
+                               HOST_WIDE_INT delta, HOST_WIDE_INT vcall_offset,
+                               tree function)
...
+  emit_insn (gen_jump (funexp));

(actually, in output_operand() downstream from this statement) while compiling
c++98/strstream.cc, with an error that the "%l" operand was not a label.

This is the first occasion when this routine is called.

--
Michael Eager    ea...@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077