Re: [PATCH] PR36312

2014-11-12 Thread Markus Trippelsdorf
On 2014.11.05 at 18:32 +0100, Manuel López-Ibáñez wrote: > > I committed this as r217149. This patch causes kernel build failures when using GCC_COMPARE_DEBUG=1. "GCC_COMPARE_DEBUG=1 make CC=/var/tmp/gcc_trunk/usr/local/bin/gcc" is enough to reproduce. See: https://gcc.gnu.org/bugzilla/show_bug

Re: [PATCH] PR36312

2014-11-06 Thread Anthony Brandon
Sorry, I didn't realize the default didn't build all languages. I will configure with --enable-languages=all from now on. On Thu, Nov 6, 2014 at 1:26 AM, Manuel López-Ibáñez wrote: > On 5 November 2014 21:57, Eric Botcazou wrote: >>> 2014-10-25 Anthony Brandon >>> >>> PR driver/36312 >>>

Re: [PATCH] PR36312

2014-11-06 Thread Anthony Brandon
It looks like -soname, and libgnat-5.0.so both end up in the infiles array. I'm not sure if that's supposed to happen or not. On Wed, Nov 5, 2014 at 9:57 PM, Eric Botcazou wrote: >> 2014-10-25 Anthony Brandon >> >> PR driver/36312 >> * diagnostic-core.h: Add prototype for fatal_error.

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 5 November 2014 21:57, Eric Botcazou wrote: >> 2014-10-25 Anthony Brandon >> >> PR driver/36312 >> * diagnostic-core.h: Add prototype for fatal_error. >> * diagnostic.c (fatal_error): New function fatal_error. >> * gcc.c (store_arg): Remove have_o_argbuf_index. >> (proces

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 6 November 2014 00:33, Joseph Myers wrote: > On Wed, 5 Nov 2014, Manuel López-Ibáñez wrote: > >> Sorry for the breakage. I guess Anthony didn't configure with >> --enable-languages=all,ada. It is a typical mistake I have myself done >> in the past. Since the fix is not obvious to me and I'm not

Re: [PATCH] PR36312

2014-11-05 Thread Joseph Myers
On Wed, 5 Nov 2014, Manuel López-Ibáñez wrote: > Sorry for the breakage. I guess Anthony didn't configure with > --enable-languages=all,ada. It is a typical mistake I have myself done > in the past. Since the fix is not obvious to me and I'm not sure how > much time Anthony would require to find a

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 5 November 2014 23:39, Eric Botcazou wrote: >> Sorry for the breakage. I guess Anthony didn't configure with >> --enable-languages=all,ada. It is a typical mistake I have myself done >> in the past. Since the fix is not obvious to me and I'm not sure how >> much time Anthony would require to fi

Re: [PATCH] PR36312

2014-11-05 Thread Eric Botcazou
> Sorry for the breakage. I guess Anthony didn't configure with > --enable-languages=all,ada. It is a typical mistake I have myself done > in the past. Since the fix is not obvious to me and I'm not sure how > much time Anthony would require to find a fix, I reverted the patch > for now. Thanks.

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 5 November 2014 21:57, Eric Botcazou wrote: >> 2014-10-25 Anthony Brandon >> >> PR driver/36312 >> * diagnostic-core.h: Add prototype for fatal_error. >> * diagnostic.c (fatal_error): New function fatal_error. >> * gcc.c (store_arg): Remove have_o_argbuf_index. >> (proces

Re: [PATCH] PR36312

2014-11-05 Thread Eric Botcazou
> 2014-10-25 Anthony Brandon > > PR driver/36312 > * diagnostic-core.h: Add prototype for fatal_error. > * diagnostic.c (fatal_error): New function fatal_error. > * gcc.c (store_arg): Remove have_o_argbuf_index. > (process_command): Check if input and output files are the sa

Re: [PATCH] PR36312

2014-11-05 Thread Manuel López-Ibáñez
On 4 November 2014 23:40, Jeff Law wrote: > On 10/25/14 04:20, Anthony Brandon wrote: >> >> Hi, >> >> Sorry for the delay. Here are the updated diff and changelog. >> >> gcc/testsuite/ChangeLog: >> >> 2014-10-25 Anthony Brandon >> >> PR driver/36312 >> * gcc.misc-tests/output.exp: New

Re: [PATCH] PR36312

2014-11-04 Thread Jeff Law
On 10/25/14 04:20, Anthony Brandon wrote: Hi, Sorry for the delay. Here are the updated diff and changelog. gcc/testsuite/ChangeLog: 2014-10-25 Anthony Brandon PR driver/36312 * gcc.misc-tests/output.exp: New test case for identical input and output files. include/ChangeLog: 20

Re: [PATCH] PR36312

2014-10-25 Thread Anthony Brandon
Hi, Sorry for the delay. Here are the updated diff and changelog. gcc/testsuite/ChangeLog: 2014-10-25 Anthony Brandon PR driver/36312 * gcc.misc-tests/output.exp: New test case for identical input and output files. include/ChangeLog: 2014-10-25 Anthony Brandon PR driver/363

Re: [PATCH] PR36312

2014-10-19 Thread Manuel López-Ibáñez
On 18 October 2014 14:43, Anthony Brandon wrote: > Never mind about functions.texi. I figured out how to do it. > Here is the new diff and changelog. > > libiberty/ChangeLog: > > 2014-10-18 Anthony Brandon > > * filename_cmp.c (filename_eq): No change. Unfortunately mklog is not 100% p

Re: [PATCH] PR36312

2014-10-18 Thread Anthony Brandon
I didn't start the paper work yet. I'll get started on it. For the changelog entry, I didn't change anything to that function, but mklog gives an entry for it. I think it's because the new function is added after it and at the end if the file. On Sat, Oct 18, 2014 at 3:15 PM, Marc Glisse wrote:

Re: [PATCH] PR36312

2014-10-18 Thread Marc Glisse
On Sat, 18 Oct 2014, Anthony Brandon wrote: I'm a new contributor Welcome! I don't yet have a copyright assignment Did you start the paperwork? * filename_cmp.c (filename_eq): No change. I didn't look at the patch, but that's a confusing ChangeLog entry. Did you change somethin

Re: [PATCH] PR36312

2014-10-18 Thread Anthony Brandon
Never mind about functions.texi. I figured out how to do it. Here is the new diff and changelog. libiberty/ChangeLog: 2014-10-18 Anthony Brandon * filename_cmp.c (filename_eq): No change. (canonical_filename_eq): New function to check if file names are the same. * func