Re: [PATCH mips] Remove fp64 multilibs from mips-mti-* targets.

2014-08-10 Thread Eric Christopher
OK. -eric On Fri, Aug 8, 2014 at 1:07 PM, Steve Ellcey wrote: > Here is another MIPS patch. This removes the fp64 multilib from the > mips-mti-* targets. With the new fpxx we no longer want special fp64 > multilibs in the mti targets. Since it doesn't affect any other targets > hopefully ther

Re: [PATCH mips] Pass -msoft-float/-mhard-float flags to GAS

2014-08-12 Thread Eric Christopher
On Sat, Aug 9, 2014 at 12:00 PM, Matthew Fortune wrote: > Moore, Catherine writes: >> > -Original Message- >> > From: Steve Ellcey [mailto:sell...@mips.com] >> > Sent: Friday, August 08, 2014 3:42 PM >> > To: Moore, Catherine; matthew.fort...@imgtec.com; echri...@gmail.com; >> > >> > 2014

Re: [PATCH mips] Do not compile mips16.S in soft-float mode

2014-08-12 Thread Eric Christopher
>> >> -#ifdef __mips_micromips >> +#if defined(__mips_micromips) || defined(__mips_soft_float) >>/* DO NOTHING */ >> #else >> Mind adding a comment here explaining why we don't want to do anything for soft float (and micromips)? OK otherwise. Thanks! -eric

Re: [PATCH mips] Do not compile mips16.S in soft-float mode

2014-08-12 Thread Eric Christopher
Thanks! -eric On Tue, Aug 12, 2014 at 8:33 AM, Steve Ellcey wrote: > On Tue, 2014-08-12 at 00:07 -0700, Eric Christopher wrote: >> >> >> >> -#ifdef __mips_micromips >> >> +#if defined(__mips_micromips) || defined(__mips_soft_float) >> >>/*

Re: [PATCH mips] Pass -msoft-float/-mhard-float flags to GAS

2014-08-13 Thread Eric Christopher
On Tue, Aug 12, 2014 at 2:07 AM, Matthew Fortune wrote: > Eric Christopher writes: >> On Sat, Aug 9, 2014 at 12:00 PM, Matthew Fortune >> wrote: >> > Moore, Catherine writes: >> >> > -Original Message- >> >> > From: Steve Ellcey [ma

Re: [build, driver] RFC: Support compressed debug sections

2014-06-04 Thread Eric Christopher
>> If it is just to reach compatibility with the debugger, then I’d rather >> either just mandate a certain debugger or autoconf for what the current >> debugger supports. As of late people seem to just break the debugging >> experience with non-updated gdbs and assume that a newer gdb is used. >

Re: [build, driver] RFC: Support compressed debug sections

2014-06-27 Thread Eric Christopher
On Thu, Jun 26, 2014 at 6:32 AM, Rainer Orth wrote: > Eric Christopher writes: > >>>> If it is just to reach compatibility with the debugger, then I’d rather >>>> either just mandate a certain debugger or autoconf for what the current >>>> debugger suppo

Re: [PATCH][MIPS] Fix ICE in bitmap routines with LRA and inline assembly language

2014-09-18 Thread Eric Christopher
On Thu, Sep 18, 2014 at 1:11 PM, Eric Christopher wrote: > > > > On Wed, Sep 10, 2014 at 3:39 AM, Matthew Fortune > wrote: >> >> > The patch is ok to commit. I think I made a typo as other analogous >> > code contains '|='. >> > >&

Re: [Patch AArch64] Fix extended register width

2014-09-30 Thread Eric Christopher
On Tue, Sep 30, 2014 at 5:57 AM, Marcus Shawcroft wrote: > On 22 September 2014 19:41, Carrot Wei wrote: >> Hi >> >> The extended register width in add/adds/sub/subs/cmp instructions is >> not always the same as target register, it depends on both target >> register width and extension type. But

Re: [Patch AArch64] Fix extended register width

2014-10-01 Thread Eric Christopher
On Wed, Oct 1, 2014 at 1:42 AM, Richard Earnshaw wrote: > On 30/09/14 21:30, Eric Christopher wrote: >> On Tue, Sep 30, 2014 at 5:57 AM, Marcus Shawcroft >> wrote: >>> On 22 September 2014 19:41, Carrot Wei wrote: >>>> Hi >>>> >>&

[PATCH] Use CHECKSUM_ macros and ULEB128 checksum for DIE tag

2013-07-22 Thread Eric Christopher
Hi Cary, This patch changes the ODR checker to use the CHECKSUM_ macros and instead of depending on size of int to use the ULEB128 of the tag (similar to the deep hash) to compute the values. Thoughts? -eric 2013-07-22 Eric Christopher * dwarf2out.c (die_odr_checksum): New function

Re: [PATCH] Use CHECKSUM_ macros and ULEB128 checksum for DIE tag

2013-07-22 Thread Eric Christopher
eave the >> rest to Cary. > > Agree with the function comment and the spaces. Oh yeah, thanks. Sorry, it's been a while. > >> 2013-07-22 Eric Christopher >> >> * dwarf2out.c (die_odr_checksum): New function to use >> CHECKSUM_ macros

Re: Darwin bootstrap failure following wide int merge (was: we are starting the wide int merge)

2014-05-26 Thread Eric Christopher
On Mon, May 26, 2014 at 1:14 AM, FX wrote: >> > .././../gcc-4.10-20140518/gcc/wide-int.cc:1274:23: error: invalid use of a >> > cast in a inline asm context requiring an l-value: remove the cast or >> > build with -fheinous-gnu-extensions >> > umul_ppmm (val[1], val[0], op1.ulow (), op2.

Re: [PATCH, rs6000] Fix aggregate alignment ABI issue

2014-07-09 Thread Eric Christopher
On Wed, Jul 9, 2014 at 11:25 AM, David Edelsohn wrote: > On Wed, Jul 9, 2014 at 12:04 PM, Ulrich Weigand wrote: >> Hello, >> >> last year, Bill added a patch to address PR 57949 by aligning aggregates >> requiring at least 128-bit alignment at a quadword boundary in the >> parameter save area: >>

Re: [PATCH, rs6000] Fix aggregate alignment ABI issue

2014-07-09 Thread Eric Christopher
On Wed, Jul 9, 2014 at 12:01 PM, Mike Stump wrote: > On Jul 9, 2014, at 11:29 AM, Eric Christopher wrote: >>>> - The change only affects rare cases: passing a struct by value that is >>>> * not a float/vector special case, and >>>> * has a size of 1, 2,

Re: Some DWARFv5 proposal prototypes (atomic_type, aligned_type)

2014-07-15 Thread Eric Christopher
On Tue, Jul 15, 2014 at 4:36 AM, Mark Wielaard wrote: > On Mon, 2014-07-14 at 13:58 -0600, Tom Tromey wrote: >> > "Mark" == Mark Wielaard writes: >> Mark> The following two patches are based on the earlier "dwarf2out.c: Pass >> Mark> one cv_quals argument" and "Emit DW_tag_restrict_type" patc

[PATCH, docs] Document -z option

2014-07-15 Thread Eric Christopher
Just to document that it's passed directly on to the linker. OK? Wording changes? -eric 2014-07-15 Eric Christopher * doc/invoke.texi (Link Options): Document -z option. Index: gcc/doc/invoke.texi === --- gc

Re: [PATCH, docs] Document -z option

2014-07-15 Thread Eric Christopher
wrote: > On Tue, Jul 15, 2014 at 5:23 PM, Eric Christopher wrote: >> Just to document that it's passed directly on to the linker. >> >> OK? Wording changes? >> >> -eric >> >> 2014-07-15 Eric Christopher >> >> * doc/invok

Re: [PATCH][AArch64] Vectorise bswap[16,32,64]

2014-04-15 Thread Eric Christopher
Testcase weirdness? for (i < 0; i < N; ++i) { arr[i] = i; expect[i] = __builtin_bswap64 (i); if (y) /* Avoid vectorisation. */ abort (); } i < 0 :) duplicated in all 3 testcases btw. -eric On Tue, Apr 15, 2014 at 4:25 AM, Kyrill Tkachov wrote: > Hi all, > > This