Re: [Patch,optimization]: Optimized changes in the estimate register pressure cost.

2015-09-26 Thread Bin.Cheng
On Sat, Sep 26, 2015 at 12:51 PM, Ajit Kumar Agarwal wrote: > I have made the following changes in the estimate_reg_pressure_cost function > used > by the loop invariant and IVOPTS. > > Earlier the estimate_reg_pressure cost uses the cost of n_new variables that > are generated by the Loop Invar

[PATCH] Fix undefined behaviour in cris port

2015-09-26 Thread Jeff Law
Yet another left shift of a negative number. Fixed in the obvious way. Tested by completing the cris-* builds in config-list.mk using a trunk compiler. Installed on the trunk. Jeff commit 6759b0576ae1f474749cc901e0e3d5679f546251 Author: law Date: Sat Sep 26 07:33:05 2015 + [PAT

[PATCH] Fix undefined behaviour in mips port

2015-09-26 Thread Jeff Law
Another instance of left shifting a negative value. Fixed in an obvious way. Verified all the mips configurations in config-list.mk build now using a trunk compiler. Installed on the trunk, Jeff commit fff8087106b686a8c0ac6c7dab3dd7e12cd717ba Author: Jeff Law Date: Sat Sep 26 03:24:00 201

[PATCH] Fix undefined behaviour in SH port,

2015-09-26 Thread Jeff Law
And now the SH port in the obvious way. Verified by building all the sh configurations in config-list.mk with a trunk compiler. Installed on the trunk, Jeff commit ac55233b083fc30445141c530fcf472786f935e7 Author: Jeff Law Date: Sat Sep 26 03:24:07 2015 -0400 [PATCH] Fix undefined

[PATCH] Fix undefined behaviour in arc port

2015-09-26 Thread Jeff Law
And now the ARC port. Tested by building the arc configurations in config-list.mk with a trunk compiler. Installed on the trunk, Jeff commit ce021a249da7da00877d20a4b3dc79b47eec19c1 Author: Jeff Law Date: Sat Sep 26 03:22:53 2015 -0400 [PATCH] Fix undefined behaviour in arc port

Re: [PATCH] Fix undefined behaviour in arc port

2015-09-26 Thread Andreas Schwab
Jeff Law writes: > @@ -9320,7 +9320,9 @@ arc_legitimize_reload_address (rtx *p, machine_mode > mode, int opnum, >if ((scale-1) & offset) > scale = 1; >shift = scale >> 1; > - offset_base = (offset + (256 << shift)) & (-512 << shift); > + offset_base > + = ((of

Re: [PATCH] Fix undefined behaviour in arc port

2015-09-26 Thread Andreas Schwab
Jeff Law writes: > @@ -195,7 +195,7 @@ >"@internal > Unconditional two-address add / sub constant" >(and (match_code "const_int") > - (match_test "ival == -1 << 31 > + (match_test "ival == HOST_WIDE_INT (HOST_WIDE_INT_M1U << 31) Syntax error? Andreas. -- Andreas Schwab

Re: [PATCH] fix PR67700

2015-09-26 Thread Tobias Grosser
On 09/25/2015 10:39 PM, Sebastian Pop wrote: The patch makes the detection of scop parameters in parameter_index_in_region a bit more conservative by discarding scalar variables defined in function of data references defined in the scop. 2015-09-25 Aditya Kumar Sebastian Pop

[Patch, fortran] PR67721 deep copy missing when assigning derived type constructor to an array

2015-09-26 Thread Mikael Morin
Hello, I've just submitted this PR, and the patch as well, which passes the testsuite. The problem is a missing deep copy when the rhs is a (scalar) derived type constructor (with allocatable components) and the lhs an array. The patch removes the nonconstantness condition, so that the deep_c

Re: [Patch, fortran] PR67721 deep copy missing when assigning derived type constructor to an array

2015-09-26 Thread Paul Richard Thomas
Dear Mikael, Apart from the regtesting, this patch is 'obvious'. This is good for trunk and, I would suggest, after a decent interval 5 branch. Thanks for the PR and the patch. Paul On 26 September 2015 at 15:10, Mikael Morin wrote: > Hello, > > I've just submitted this PR, and the patch as we

Re: [PATCH] fortran/67616 -- Fix ICE within a BLOCK construct

2015-09-26 Thread Mikael Morin
Le 25/09/2015 23:41, Steve Kargl a écrit : The attached patch has been built and regression tested on x86_64-*-freebsd. No regression occurred. The patch removes a conditional within an assert() that triggers when a BLOCK construct is encountered. What happens here, I think, is symtree is a ne

RE: [PATCH] Fix undefined behaviour in mips port

2015-09-26 Thread Matthew Fortune
Jeff Law writes: > Another instance of left shifting a negative value. Fixed in an obvious > way. Verified all the mips configurations in config-list.mk build now > using a trunk compiler. > > Installed on the trunk, Thanks Jeff. Matthew

[Patch, fortran/67567] resolve.c: gfc_error called with iface->module == NULL

2015-09-26 Thread Paul Richard Thomas
Fixed as 'obvious' in revision: 228169. Cheers Paul 2013-09-26 Paul Thomas PR fortran/67567 * resolve.c (resolve_fl_procedure): For module procedures, take the parent module name and the submodule name from the name of the namespace.

Re: [PATCH] fortran/67616 -- Fix ICE within a BLOCK construct

2015-09-26 Thread Steve Kargl
On Sat, Sep 26, 2015 at 03:34:22PM +0200, Mikael Morin wrote: > Le 25/09/2015 23:41, Steve Kargl a ??crit : > > The attached patch has been built and regression > > tested on x86_64-*-freebsd. No regression occurred. > > > > The patch removes a conditional within an assert() > > that triggers when

Re: [patch] Enable lightweight checks with _GLIBCXX_ASSERTIONS.

2015-09-26 Thread Jonathan Wakely
On 14/09/15 11:57 +0200, Florian Weimer wrote: On 09/10/2015 06:57 PM, Martin Sebor wrote: There is quite a bit of documentation of _FORTIFY_SOURCE that explains its effect on user code. I think there are only random blog articles discussing aspects of it, most of them slightly incorrect or o

Re: [patch] Enable lightweight checks with _GLIBCXX_ASSERTIONS.

2015-09-26 Thread Florian Weimer
On 09/26/2015 09:52 PM, Jonathan Wakely wrote: > Would changes like this be suitable for _FORTIFY_SOURCE? > diff --git a/libstdc++-v3/include/std/mutex b/libstdc++-v3/include/std/mutex > index 5e5ced1..074bf26 100644 > --- a/libstdc++-v3/include/std/mutex > +++ b/libstdc++-v3/include/std/mutex >

Re: [PATCH] Disable -fno-reorder-blocks-and-partition if no -fprofile-use to avoid unnecessary overhead

2015-09-26 Thread Bernhard Reutner-Fischer
On September 25, 2015 7:59:07 PM GMT+02:00, Andi Kleen wrote: >On Fri, Sep 25, 2015 at 11:01:47AM -0600, Jeff Law wrote: >> On 09/25/2015 10:58 AM, Teresa Johnson wrote: >> >Woops, we crossed wires. I just committed this patch. Would you like >> >me to revert it? >> Leave it. If Andi can include

Re: [PATCH 1/5] Testsuite: add dg-{begin|end}-multiline-output commands

2015-09-26 Thread Bernhard Reutner-Fischer
On September 25, 2015 6:52:37 PM GMT+02:00, Jeff Law wrote: >On 09/22/2015 03:26 PM, David Malcolm wrote: >> This patch is essentially identical to v1 here: >>https://gcc.gnu.org/ml/gcc-patches/2015-09/msg00729.html >> The only change is in the ChangeLog, moving the libgo.exp >> ChangeLog entr

Re: [PATCH] Disable -fno-reorder-blocks-and-partition if no -fprofile-use to avoid unnecessary overhead

2015-09-26 Thread Andi Kleen
> +@item partition-cold-min-freq > +When using doing function partitioning and there is no profile information > +consider edges below this frequency cold. Setting to zero disables > +any function splitting without profile information. > > Trouble parsing "using doing", nice hack otherwise. Th

Re: [RFC PATCH] parse #pragma GCC diagnostic in libcpp

2015-09-26 Thread Manuel López-Ibáñez
On 25 September 2015 at 17:14, Dodji Seketeli wrote: > The caller of do_pragma(), which is destringize_and_run() then detects > that pfile->directive_result.type is set, and then puts the tokens of > the pragma back into the input stream again. So next time the FE > requests more tokens, it's goi

Re: [PATCH] v3 of diagnostic_show_locus and rich_location (was Re: [PATCH 2/5] Reimplement diagnostic_show_locus, introducing rich_location classes (v2))

2015-09-26 Thread Dodji Seketeli
[Note to libcpp, C, and Fortran maintainers: we still need your input :-)] Hello, David Malcolm writes: [...] > Here's the revised comment I put in the attached patch: [...] > + The class caches the lookup of the color codes for the above. > + > + The class also has responsibility for tr

Re: [RFC PATCH] parse #pragma GCC diagnostic in libcpp

2015-09-26 Thread Dodji Seketeli
Manuel López-Ibáñez writes: > On 25 September 2015 at 17:14, Dodji Seketeli wrote: >> The caller of do_pragma(), which is destringize_and_run() then detects >> that pfile->directive_result.type is set, and then puts the tokens of >> the pragma back into the input stream again. So next time the