Re: [GSoC] Addition of __isl_give to declarations

2014-07-05 Thread Tobias Grosser
On 05/07/2014 08:58, Roman Gareev wrote: This patch adds __isl_give to declarations of the following funcions: generate_isl_context, generate_isl_schedule, scop_to_isl_ast Is it fine for trunk? The ChangeLog should be per-function. Otherwise LGTM. Tobias

Re: [PATCH] Fix arrays in rtx.u + add minor rtx verification

2014-07-05 Thread Richard Biener
On July 4, 2014 10:50:01 PM CEST, Jakub Jelinek wrote: >On Mon, Jun 23, 2014 at 04:40:43PM +0200, Richard Biener wrote: >> Can we instead refactor expmed.c to avoid allocating rtx_def >directly? >> Like by using rtx in init_expmed_rtl and allocating from an obstack >> (or not care and GC-allocate

Re: [PATCH] [ARM] [RFC] Fix longstanding push_minipool_fix ICE (PR49423, lp1296601)

2014-07-05 Thread Charles Baylis
On 3 July 2014 15:26, Richard Earnshaw wrote: > So OK, but if you're considering back-ports, I suggest you let it bake a > while on trunk first. Committed as r212303.

Re: [wwwdocs,Fortran] Announce IEEE intrinsic modules support for Fortran

2014-07-05 Thread Gerald Pfeifer
On Fri, 4 Jul 2014, FX wrote: > Sounds good to me. Okay; just committed per the patch below. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.926 diff -u -r1.926 index.html --- index.

[Patch, testsuite] PR61453 gfortran.dg/bind_c_array_params_2.f90 for targets where a call insn isn't "call"

2014-07-05 Thread Dominique Dhumieres
Is the following patch OK for trunk? 2014-07-05 Dominique d'Humieres PR testsuite/61453 * gfortran.dg/gfortran.dg/bind_c_array_params_2.f90: Adjust regexp for more targets. --- ../_clean/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 2014-05-24 16:17:53.

[PATCH, gfortran testsuite] Minor cleanup for the gfortran test suite, v2.

2014-07-05 Thread Dominique Dhumieres
On Mon, 05 May 2014, I have posted https://gcc.gnu.org/ml/fortran/2014-05/msg00012.html. On IRC Tobias Burnus remarked that the *.mod file in gfortran.dg/vect/fast-math-real8-pr40801.f90 should be cleaned automatically. This is not done because the cleaning is done in gfortran-dg-runtest and not

Re: [Patch, testsuite] PR61453 gfortran.dg/bind_c_array_params_2.f90 for targets where a call insn isn't "call"

2014-07-05 Thread John David Anglin
On 5-Jul-14, at 11:56 AM, Dominique Dhumieres wrote: Is the following patch OK for trunk? 2014-07-05 Dominique d'Humieres PR testsuite/61453 * gfortran.dg/gfortran.dg/bind_c_array_params_2.f90: Adjust regexp for more targets. --- ../_clean/gcc/testsuite/gfortran.dg/

Re: [Patch, testsuite] PR61453 gfortran.dg/bind_c_array_params_2.f90 for targets where a call insn isn't "call"

2014-07-05 Thread Dominique Dhumieres
> "myBindC,%r2" will work on hppa*-*-*. I don't believe preceding stuff > is needed on hppa. Sorry, the relation with -flto has been lost in my repost of this patch. Without something before 'myBindC', the symbol appears twice when compiled with -flto, once in the assembly itself and once in the

Make declaration of wide_int_storage match its definition

2014-07-05 Thread Gerald Pfeifer
Farther below in wide-int.h, we define wide_int_storage as a class: class GTY(()) wide_int_storage { private: HOST_WIDE_INT val[WIDE_INT_MAX_ELTS]; unsigned int len; unsigned int precision; : The patch below, which I applied as obvious after a full bootstrap on i386-unknown-fr

Re: [Patch, testsuite] PR61453 gfortran.dg/bind_c_array_params_2.f90 for targets where a call insn isn't "call"

2014-07-05 Thread John David Anglin
On 5-Jul-14, at 1:55 PM, Dominique Dhumieres wrote: "myBindC,%r2" will work on hppa*-*-*. I don't believe preceding stuff is needed on hppa. Sorry, the relation with -flto has been lost in my repost of this patch. Without something before 'myBindC', the symbol appears twice when compile

Re: Make declaration of wide_int_storage match its definition

2014-07-05 Thread pinskia
> On Jul 5, 2014, at 11:42 AM, Gerald Pfeifer wrote: > > Farther below in wide-int.h, we define wide_int_storage as a class: > > class GTY(()) wide_int_storage > { > private: >HOST_WIDE_INT val[WIDE_INT_MAX_ELTS]; >unsigned int len; >unsigned int precision; > : > > The patch b

Re: Make declaration of wide_int_storage match its definition

2014-07-05 Thread Gerald Pfeifer
On Sat, 5 Jul 2014, pins...@gmail.com wrote: > This patch is fine but c++ allows class and struct be used > interchangeable. If there is a compilers which does not it is broken > and should be report to them. Yes that means clang is broken. Clang does allow for it (it actually is the stage 1 co

Re: Make declaration of wide_int_storage match its definition

2014-07-05 Thread Jakub Jelinek
On Sat, Jul 05, 2014 at 09:25:50PM +0200, Gerald Pfeifer wrote: > On Sat, 5 Jul 2014, pins...@gmail.com wrote: > > This patch is fine but c++ allows class and struct be used > > interchangeable. If there is a compilers which does not it is broken > > and should be report to them. Yes that means

Re: [wwwdocs,Fortran] Announce IEEE intrinsic modules support for Fortran

2014-07-05 Thread Gerald Pfeifer
On Sat, 5 Jul 2014, Gerald Pfeifer wrote: > Okay; just committed per the patch below. Oops. This makes it consistent with other entries. Gerald

Re: [GSoC] Addition of ISL AST generation to Graphite

2014-07-05 Thread Rainer Orth
Rainer Orth writes: > Roman Gareev writes: > >>> It seems the patch1/patch2 files you attach have the Content-Type: >>> application/octet-stream. This makes it impossible to view them inline. >>> Could you send them as text files? Just calling them patch1.patch or >>> patch1.txt should make this

RE: [PATCH, aarch64] Add prefetch support

2014-07-05 Thread Gopalasubramanian, Ganesh
PING! From: Gopalasubramanian, Ganesh Sent: Friday, July 04, 2014 5:57 AM To: gcc-patches@gcc.gnu.org Cc: marcus.shawcr...@arm.com; richard.earns...@arm.com Subject: [PATCH, aarch64] Add prefetch support Hi, Attached is a patch that implements * Pref

Re: [fortran, patch] IEEE intrinsic modules (ping)

2014-07-05 Thread Rainer Orth
Steve Kargl writes: > On Tue, Jun 24, 2014 at 10:26:27PM +0200, FX wrote: >> >> 3. Does the attached updated patch (libgfortran only, without >> >> regenerated files) fix the problem? >> > >> > I'll test it when my regtesting is completed. But, a scan of >> > the configure.host re-arrangement s

[wwwdocs] Rotate news

2014-07-05 Thread Gerald Pfeifer
Applied. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.928 diff -u -r1.928 index.html --- index.html 5 Jul 2014 19:46:32 - 1.928 +++ index.html 5 Jul 2014 21:14:05 -

Re: [PATCH 3/5] IPA ICF pass

2014-07-05 Thread Gerald Pfeifer
On Fri, 20 Jun 2014, Trevor Saunders wrote: >> +@item -fipa-icf >> +@opindex fipa-icf >> +Perform Identical Code Folding for functions and read-only variables. >> +Behavior is similar to Gold Linker ICF optimization. Symbols proved >> +as semantically equivalent are redirected to corresponding symb

[wwwdocs] bugs/index.html: remove some links and make some relative

2014-07-05 Thread Gerald Pfeifer
Ever since the following change to maintainer-scripts/gcc_release 2007-08-21 Paul Brook Nathan Sidwell Mark Mitchell Joseph Myers * gcc_release: Do not include copies of bugs.html in releases. * update_web_docs_svn: Set VERSION_

Re: AARCH64 configure check for gas -mabi support

2014-07-05 Thread Gerald Pfeifer
On Mon, 30 Jun 2014, Yufeng Zhang wrote: > Looks good to me. Thanks for the fix. >> 2014-06-30 Gerald Pfeifer >> >> * doc/install.texi (Specific, aarch64*-*-*): Fix markup. Reword a bit. I also pushed this to the GCC 4.9 branch now. Gerald

Re: [wwwdocs,Fortran] Announce IEEE intrinsic modules support for Fortran

2014-07-05 Thread Gerald Pfeifer
On Sat, 5 Jul 2014, Gerald Pfeifer wrote: > Oops. This makes it consistent with other entries. Something happened when I made that edit resulting in a weird date, plus I failed to include the patch. Thanks to a friendly voice noting this to me! Gerald Index: index.html ===

Re: [PATCH, libbacktrace]: fix pointer from integer without a cast

2014-07-05 Thread Ian Lance Taylor
On Thu, Jul 3, 2014 at 2:34 AM, Duncan Mac-Vicar P. wrote: > > the following patch fixes some post-build-checks from our distro build > system, better to upstream it: > > [ 4077s] E: rust 64bit-portability-issue > /home/abuild/rpmbuild/BUILD/rust-0.11.0+git.1403898616.aa1163b/src/libbacktrace/dwar

Re: [PATCH 3/5] IPA ICF pass

2014-07-05 Thread Jan Hubicka
> On Fri, 20 Jun 2014, Trevor Saunders wrote: > >> +@item -fipa-icf > >> +@opindex fipa-icf > >> +Perform Identical Code Folding for functions and read-only variables. I would perhaps explicitly say that the optimizations reduce code size and may disturb unwind stacks by replacing a function by eq

[wwwdocs,Java] java/news.html

2014-07-05 Thread Gerald Pfeifer
2014-07-06 Gerald Pfeifer * news.html: Avoid link into our SVN repository. Fix grammar. Applied. Index: news.html === RCS file: /cvs/gcc/wwwdocs/htdocs/java/news.html,v retrieving revision 1.18 retrieving revision 1.19 d

Re: [wwwdocs] Buildstat update for 4.7

2014-07-05 Thread Gerald Pfeifer
On Thu, 3 Jul 2014, Tom G. Christensen wrote: > Testresults for 4.7.4: > i386-pc-solaris2.8 > i386-pc-solaris2.9 > sparc-sun-solaris2.8 > sparc-sun-solaris2.9 > sparc64-sun-solaris2.8 > x86_64-apple-darwin13.2.0 > x86_64-pc-linux-gnu Applied, thank you! Gerald

[PATCH] gcc/config/microblaze/microblaze.md: Remove redundant '@' to avoid compiling warning

2014-07-05 Thread Chen Gang
* microblaze/microblaze.md: Remove redundant '@' to avoid compiling warning. The related warning: ../../gcc/gcc/config/microblaze/microblaze.md:516: '@' is redundant for output template with single alternative Signed-off-by: Chen Gang --- gcc/config/microblaze/microblaze.md | 3 +-- 1

Re: [GSoC] Addition of ISL AST generation to Graphite

2014-07-05 Thread Roman Gareev
Sorry, I was going to fix this. Thank you very much! -- Cheers, Roman Gareev