This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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 opcodes]: Fix boostrap for --enable-targets=all with gcc 4.8

From: Alan Modra <amodra at gmail dot com>
To: Kai Tietz <ktietz70 at googlemail dot com>
Cc: Binutils <binutils at sourceware dot org>, Doug Evans <dje at google dot com>
Date: Tue, 16 Oct 2012 11:15:38 +1030
Subject: Re: [patch opcodes]: Fix boostrap for --enable-targets=all with gcc 4.8
References: <CAEwic4YVJtnA8JaEmE+VSEydjMOra-Xh6QGuWLGofkmEDqYxhA@mail.gmail.com> <507C1A00.7050800@arm.com>

On Mon, Oct 15, 2012 at 03:13:20PM +0100, Richard Earnshaw wrote:
> On 10/10/12 16:38, Kai Tietz wrote:
> >Hi,
> >
> >this patch fixes new warnings occuring by using gcc 4.8.
> >
> >ChangeLog
> >
> >2012-10-10  Kai Tietz
> >
> >	* aarch64-asm.c (aarch64_ins_ldst_reglist): Initialize
> >	value with a default.
> >	(do_special_encoding): Likewise.
> >	(aarch64_ins_ldst_elemlist): Pre-initialize QSsize, and opcodeh2
> >	variables with default.
> >	* arc-dis.c (write_comments_): Don't use strncat due
> >	size of state->commentBuffer pointer isn't predictable.
> >
> >Tested for x86_64-w64-mingw32, and x86_64-unknown-linux-gnu.  Ok for apply?
> >
> 
> The AArch64 bits are OK.

The arc-dis.c change is OK too.  The code there look half-baked and in
fact is dead.

-- 
Alan Modra
Australia Development Lab, IBM

References:
- [patch opcodes]: Fix boostrap for --enable-targets=all with gcc 4.8
  - From: Kai Tietz
- Re: [patch opcodes]: Fix boostrap for --enable-targets=all with gcc 4.8
  - From: Richard Earnshaw

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