RE: [PATCH 1/4] [MIPS] Add support for MIPS SIMD Architecture (MSA)

Hi Robert, Next batch of comments for this patch. I've covered mips-msa.md up to the copy patterns and one supporting function from mips.c. >+++ b/gcc/config/mips/mips-msa.md >+;; The attribute gives half modes for vector modes. >+(define_mode_attr VHMODE >+ [(V8HI "V16QI") >+ (V4SI "V8HI")

Re: [PATCH] fix TLS support detection for sh targets

Rich Felker wrote: > Bad patterns caused configure to always disable TLS for big-endian sh > targets and for anything other than sh 3/4. Could you please give a patch for the trunk with an appropriate ChangeLog entry? Regards, kaz

[PATCH] fix TLS support detection for sh targets

Bad patterns caused configure to always disable TLS for big-endian sh targets and for anything other than sh 3/4. Rich --- gcc-5.2.0.base/gcc/configure.ac 2015-08-11 16:23:36.0 + +++ gcc-5.2.0/gcc/configure.ac 2015-09-13 08:17:31.714972082 + @@ -3300,7 +3300,7 @@

[wwwdocs] Buildstat update for 4.9

Latest results for 4.9.x -tgc Testresults for 4.9.3: i386-pc-solaris2.9 i486-sun-solaris2.11 sparc-sun-solaris2.9 sparc-sun-solaris2.10 sparc64-sun-solaris2.9 sparc64-sun-solaris2.10 x86_64-w64-mingw32 Testresults for 4.9.2 x86_64-unknown-linux-gnu Index: buildstat.html

Re: [PATCH] fix TLS support detection for sh targets

On Sun, Sep 13, 2015 at 06:55:56PM +0900, Kaz Kojima wrote: > Rich Felker wrote: > > Bad patterns caused configure to always disable TLS for big-endian sh > > targets and for anything other than sh 3/4. > > Could you please give a patch for the trunk with an appropriate >

Re: Reviving SH FDPIC target

On Fri, Sep 04, 2015 at 06:04:15PM -0500, Segher Boessenkool wrote: > On Fri, Sep 04, 2015 at 04:16:40PM -0400, Rich Felker wrote: > > One thing I've noticed that's odd is that gcc -mfdpic -fPIC produces > > different (less efficient) code from just gcc -mfdpic, which seems > > wrong, but agrees

Re: [PATCH] PR28901 -Wunused-variable ignores unused const initialised variables

On 13/09/15 13:40, Mark Wielaard wrote: On Sun, Sep 13, 2015 at 12:00:51AM +0200, Mark Wielaard wrote: I'll rewrite my patch a little, add some C++ testcases, and update the documentation. Then we can discuss again. Slightly adjusted patch attached. Now it is explicit that the warning is

Re: [PATCH] doc/install.texi: Mention contrib/download_prerequisites

On Thu, 10 Sep 2015, Markus Trippelsdorf wrote: > * doc/invoke.texi (Downloading GCC): Mention > contrib/download_prerequisites script. This is a follow-up patch I apllied on top. Gerald 2015-09-13 Gerald Pfeifer * doc/install.texi (Downloading the

Re: [PATCH, PR67405, committed] Avoid NULL pointer dereference

On Mon, Sep 7, 2015 at 2:39 PM, Ilya Enkovich wrote: > 2015-09-02 15:35 GMT+03:00 Richard Biener : >> >> DECL_FIELD_BIT_OFFSET should be never NULL. Whoever created that >> FIELD_DECL created an invalid one. >> >> Richard. >> > >

[wwwdocs] Buildstat update for 4.8

Latest results for 4.8.x -tgc Testresults for 4.8.5: i386-pc-solaris2.9 powerpc64le-unknown-linux-gnu s390-ibm-linux-gnu (3) s390x-ibm-linux-gnu (3) sparc-sun-solaris2.9 sparc64-sun-solaris2.9 x86_64-w64-mingw32 Index: buildstat.html

Re: [PATCH] PR28901 -Wunused-variable ignores unused const initialised variables

On Sun, Sep 13, 2015 at 12:00:51AM +0200, Mark Wielaard wrote: > I'll rewrite my patch a little, add some C++ testcases, and update the > documentation. Then we can discuss again. Slightly adjusted patch attached. Now it is explicit that the warning is enabled by -Wunused-variable for C, but not

[wwwdocs] Buildstat update for 5.x

Latest results for 5.x -tgc Testresults for 5.2.0: i686-unknown-linux-gnu powerpc-unknown-linux-gnu sparc-sun-solaris2.10 sparc64-sun-solaris2.10 x86_64-unknown-linux-gnu (4) x86_64-w64-mingw32 Testresults for 5.1.0: hppa64-hp-hpux11.00 Index: buildstat.html

[committed] Fix warning from config/pa/fptr.c

The attached patch fixes a warning caused by left shifting a signed constant. Tested on hppa-unknown-linux-gnu. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2015-09-13 John David Anglin * config/pa/fptr.c (SIGN_EXTEND): Cast -1 to

Re: [Darwin, driver] Make our sysroot handling the same as clang's.

On Sep 12, 2015, at 8:02 AM, Iain Sandoe wrote: > OK for trunk? So, does this work on 10.3? ppc? ppc64? You modify darwin.h, and that file is used everywhere. If it really works everywhere, Ok. If it doesn’t, then it should be in a file that is used only on the

Re: [PATCH] PR28901 -Wunused-variable ignores unused const initialised variables

On Sun, Sep 13, 2015 at 02:50:53PM +0200, Manuel López-Ibáñez wrote: > On 13/09/15 13:40, Mark Wielaard wrote: > >Slightly adjusted patch attached. Now it is explicit that the warning is > >enabled by -Wunused-variable for C, but not C++. There are testcases for > >both C and C++ to check the

[PATCH] GFortran testsuite visibility dependency

A number of tests in the gfortran testsuite depend on the visibility feature of GCC but do not specify the requirement in the test. This causes a large number of testsuite failure on AIX. This patch adds dg-require-visibility to the relevant tests, bringing the gfortran testsuite failures down

[wwwdocs] Rotate news

Applied. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.981 diff -u -r1.981 index.html --- index.html 16 Jul 2015 11:53:23 - 1.981 +++ index.html 12 Sep 2015 22:18:35

Re: [Darwin, driver] Make our sysroot handling the same as clang's.

Hi Mike, On 13 Sep 2015, at 21:32, Mike Stump wrote: > On Sep 12, 2015, at 8:02 AM, Iain Sandoe wrote: >> OK for trunk? chose this comment carefully : " for something that's essentially a constant for the versions of ld that work with current GCC. " > So, does this

[testsuite] Minor tweaks for visium

The FPU of the GR5 doesn't preserve NaNs so the new signbit test added to type-generic-1.c fails. Same as ARM for Wno-frame-address.c. The range of offset in addresses is quite limited so there is one more IV in loop-8.c. Tested on visium-elf, applied on the mainline. 2015-09-13 Eric

Re: New power of 2 hash policy

On 11/09/2015 15:23, Jonathan Wakely wrote: > On 11/09/15 14:18 +0100, Jonathan Wakely wrote: >> On 11/09/15 15:11 +0200, Michael Matz wrote: >>> Hi, >>> >>> On Thu, 10 Sep 2015, François Dumont wrote: >>> Here is a patch to offer an alternative hash policy. This one is using power of

Re: [Darwin, Driver/specs] A bit more TLC, factor version-min code, lose some dead specs.

On Sep 12, 2015, at 7:33 AM, Iain Sandoe wrote: > e) If a user elects to call a compiler (cc1*, f951, etc.) without a > version-min, the init provided in (d) will kick in, and stop the compiler > from segv-ing. However, we should warn the User that a default was used, >

[visium] Add --with-cpu configure option

Self-explanatory, tested on visium-elf, applied on the mainline. 2015-09-13 Olivier Hainque Eric Botcazou * config.gcc (visium-*-*): Enable --with-cpu option, accept gr5 and gr6 as possible values, defaulting to gr5.

Re: [Darwin, driver] Make our sysroot handling the same as clang's.

On Sep 13, 2015, at 1:59 PM, Iain Sandoe wrote: > chose this comment carefully : " for something that's essentially a constant > for the versions of ld that work with current GCC. “ > WDYT? I can revise the patch if you think there's a Better Way to do it. So, I tried

[PATCH diagnostics/67460] Replace some_warnings_are_errors with diagnostic_kind_count (context, DK_WERROR)

The flag diagnostic_context::some_warnings_are_errors controls whether to give the message "all warnings being treated as errors". However, when warnings are buffered and then discarded, this flag is not reset. It turns out we do not need this flag at all, since we already count explicitly how

Re: [PATCH] fix TLS support detection for sh targets

Rich Felker wrote: > I'm pretty sure this will still apply to trunk, but I can check that > and add the changelog entry. Is there something I should read on the > form or just follow the example from my last patch where you added it? The latter would be enough for this, though