This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix CRISv32 compilation



On Mon, 2 Sep 2013, Pedro Alves wrote:

On 08/30/2013 02:12 PM, Ricard Wanderlof wrote:

2013-08-30  Ricard Wanderlof  <ricardw@axis.com>

gdbserver

   	* linux-crisv32-low.c: Fix compilation errors.

Write:

   	* linux-crisv32-low.c (elf_gregset_t): Delete typedef.
	(initialize_low_arch): Call init_registers_crisv32 rather than
	init_register_crisv32.

OK with that change.

Ok with me. Thanks for the update. Should I resubmit the patch ?


(I can't see how the typedef was breaking the build though)

With the typedef left in, I get

crisv32-axis-linux-gnu-gcc -isystem /home/ricardw/P3301-head/target/crisv32-axis-linux-gnu/include -isystem /home/ricardw/P3301-head/target/crisv32-axis-linux-gnu/usr/include -g -O2 -I. -I. -I./../common -I./../regformats -I./../ -I./../../include -I./../gnulib/import -Ibuild-gnulib-gdbserver/import -Wall -Wdeclaration-after-statement -Wpointer-arith -Wformat-nonliteral -Wno-char-subscripts -Wempty-body -Werror -DGDBSERVER -c -o linux-crisv32-low.o -MT linux-crisv32-low.o -MMD -MP -MF .deps/linux-crisv32-low.Tpo linux-crisv32-low.c
linux-crisv32-low.c:372: error: conflicting types for 'elf_gregset_t'
/home/ricardw/P3301-head/target/crisv32-axis-linux-gnu/include/asm/elf.h:36: error: previous declaration of 'elf_gregset_t' was here

when building gdbserver using the target compiler.


/Ricard
--
Ricard Wolf WanderlÃf                           ricardw(at)axis.com
Axis Communications AB, Lund, Sweden            www.axis.com
Phone +46 46 272 2016                           Fax +46 46 13 61 30


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]