Re: [patch, Fortran] Fix some string temporaries

2016-07-17 Thread Mikael Morin
Hello, Le 16/07/2016 à 15:38, Thomas Koenig a écrit : Hello world, this fixes PR 71902. The recent fix for PR 71783 introduced a performance and code size regression a string temporary was created for the test case when it was not actually needed. I also took the opportunity of renaming the mi

Re: [PATCH] Avoid invoking ranlib on libbackend.a

2016-07-17 Thread David Edelsohn
> Patrick Palka writes: > Hmm, this is only a problem due to command line length limits right? > Fortunately the change makes the longest command line only about 10% > longer than the previous longest command line. With the change to > avoid building libbackend.a altogether, the longest comma

Re: [RFC][IPA-VRP] Add support for IPA VRP in ipa-cp/ipa-prop

2016-07-17 Thread Prathamesh Kulkarni
On 15 July 2016 at 05:46, kugan wrote: > Hi, > > > > This patch extends ipa-cp/ipa-prop infrastructure to handle propagation of > VR. Hi Kugan, Just a small nit - perhaps you should modify ipa_print_node_jump_functions_for_edge () to pretty-print value ranges associated with the jump function. Th

Re: [PATCH] Avoid invoking ranlib on libbackend.a

2016-07-17 Thread Patrick Palka
On Sun, Jul 17, 2016 at 9:15 AM, David Edelsohn wrote: >> Patrick Palka writes: > >> Hmm, this is only a problem due to command line length limits right? >> Fortunately the change makes the longest command line only about 10% >> longer than the previous longest command line. With the change t

Re: [PATCH] Avoid invoking ranlib on libbackend.a

2016-07-17 Thread Manuel López-Ibáñez
On 17/07/16 15:43, Patrick Palka wrote: On Sun, Jul 17, 2016 at 9:15 AM, David Edelsohn wrote: You repeatedly are making bad assumptions and assertions without having studied much about GCC. You assume that GNU ar is the only archiver in use. You propose removing libbackend.a without having inv

Re: RFA: new pass to warn on questionable uses of alloca() and VLAs

2016-07-17 Thread Manuel López-Ibáñez
On 15/07/16 18:05, Aldy Hernandez wrote: +case OPT_Walloca_larger_than_: + if (!value) + inform (loc, "-Walloca-larger-than=0 is meaningless"); + break; + +case OPT_Wvla_larger_than_: + if (!value) + inform (loc, "-Wvla-larger-than=0 is meaningless"); + bre

Re: Importing gnulib into the gcc tree

2016-07-17 Thread Manuel López-Ibáñez
On 16 July 2016 at 10:54, ayush goel wrote: > Hi, > Thanks for the feedbacks. > > —> I’m already configuring gcc with multiple languages and multilib enabled > > —> The changes have been bootstrapped and regression tested (complete check, > make -k -j20 check). > > —> As mentioned, I have locally

Re: [patch, Fortran] Fix some string temporaries

2016-07-17 Thread Thomas Koenig
Hi Mikael, Do we actually want to backport this? Technically, it is a regression, but people are not likely to notice much. It is not an ICE, neither a code correctness issue as far as I can see, so I would rather not backport. Fine with me. +case GFC_DEP_FORWARD: + retur

Re: [PATCH] Avoid invoking ranlib on libbackend.a

2016-07-17 Thread David Edelsohn
On Sun, Jul 17, 2016 at 10:43 AM, Patrick Palka wrote: > On Sun, Jul 17, 2016 at 9:15 AM, David Edelsohn wrote: >>> Patrick Palka writes: >> >>> Hmm, this is only a problem due to command line length limits right? >>> Fortunately the change makes the longest command line only about 10% >>> lo

Re: [patch, fortran] PR66310 Problems with intrinsic repeat for large number of copies

2016-07-17 Thread Jerry DeLisle
On 07/12/2016 06:26 AM, Dominique d'Humières wrote: >>> 2016-07-11 Jerry DeLisle >>> >>> PR fortran/66310 >>> * simplify.c (gfc_simplify_repeat): Set max repeat to huge - 1 to allow >>> one byte for null terminating the resulting string constant. >> >> OK, thanks > Please hold on. I

Re: [PATCH] Avoid invoking ranlib on libbackend.a

2016-07-17 Thread Patrick Palka
On Sun, 17 Jul 2016, David Edelsohn wrote: > On Sun, Jul 17, 2016 at 10:43 AM, Patrick Palka wrote: > > On Sun, Jul 17, 2016 at 9:15 AM, David Edelsohn wrote: > >>> Patrick Palka writes: > >> > >>> Hmm, this is only a problem due to command line length limits right? > >>> Fortunately the cha

Re: Fix for PR70909 in Libiberty Demangler (4)

2016-07-17 Thread Marcel Böhme
Hi, This patch is still pending a full review. Best regards, - Marcel > On 30 Jun 2016, at 12:09 AM, Pedro Alves wrote: > > On 06/29/2016 08:43 AM, Marcel Böhme wrote: >> Hi Jason, >> >> These test cases are generated by fuzzing which produces a lot of >> nonsensical input data. >> I think,

Re: Fix fir PR71696 in Libiberty Demangler (6)

2016-07-17 Thread Marcel Böhme
Hi, This patch is still pending a full review. Best regards, - Marcel > On 4 Jul 2016, at 8:54 PM, Bernd Schmidt wrote: > > On 06/30/2016 08:46 AM, Marcel Böhme wrote: >> The attached patch fixes the stack overflow in the demangler due to >> cycles in the references of “remembered” mangled typ

Re: [PATCH] Avoid invoking ranlib on libbackend.a

2016-07-17 Thread Trevor Saunders
On Sun, Jul 17, 2016 at 10:04:13PM -0400, Patrick Palka wrote: > On Sun, 17 Jul 2016, David Edelsohn wrote: > > > On Sun, Jul 17, 2016 at 10:43 AM, Patrick Palka > > wrote: > > > On Sun, Jul 17, 2016 at 9:15 AM, David Edelsohn wrote: > > >>> Patrick Palka writes: > > >> > > >>> Hmm, this is

Re: [patch,avr] make progmem work on AVR_TINY, use TARGET_ADDR_SPACE_DIAGNOSE_USAGE

2016-07-17 Thread Denis Chertykov
2016-07-15 18:26 GMT+03:00 Georg-Johann Lay : > This patch needs new hook TARGET_ADDR_SPACE_DIAGNOSE_USAGE: > https://gcc.gnu.org/ml/gcc-patches/2016-07/msg00839.html > > This patch turns attribute progmem into a working feature for AVR_TINY > cores. > > It boils down to adding 0x4000 to all symbol