[gold][aarch64]Patch for Relaxation

Ryan Mansfield rmansfield@qnx.com
Thu Oct 16 15:24:00 GMT 2014
On 14-10-15 05:05 PM, Cary Coutant wrote:
>> Hi Cary, thanks. Attached the updated patch.
>> The only diff from patch2 is the "hash_value" function, which I
>> directly pasted below for ease of review.
>
> Thanks! This is OK to commit.

I noticed that this change doesn't build with gcc 4.2.

g++ -DHAVE_CONFIG_H -I. -I../../gold  -I../../gold 
-I../../gold/../include -I../../gold/../elfcpp 
-DLOCALEDIR="\"/usr/local/share/locale\"" -DBINDIR="\"/usr/local/bin\"" 
-DTOOLBINDIR="\"/usr/local/arm-linux-gnueabi/bin\"" 
-DTOOLLIBDIR="\"/usr/local/arm-linux-gnueabi/lib\""   -W -Wall 
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -frandom-seed=aarch64.o -g 
-O2 -MT aarch64.o -MD -MP -MF .deps/aarch64.Tpo -c -o aarch64.o 
../../gold/aarch64.cc
../../gold/aarch64.cc: In static member function 'static 
bool<unnamed>::AArch64_relocate_functions<size, 
big_endian>::maybe_apply_stub(unsigned int, const 
gold::Relocate_info<size, big_endian>*, const elfcpp::Rela<size, 
big_endian>&, unsigned char*, typename 
elfcpp::Elf_types<size>::Elf_Addr, const gold::Sized_symbol<size>*, 
const gold::Symbol_value<size>*, const gold::Sized_relobj_file<size, 
big_endian>*)':
../../gold/aarch64.cc:4096: error: expected `;' before 'status'
../../gold/aarch64.cc:4098: error: 'status' was not declared in this scope

<snip>

I'm not aware of a documented required g++ version for binutils/gold. Is 
there? I looked around but didn't find anything.

Regards,

Ryan Mansfield



More information about the Binutils mailing list