Re: [REPOST] Invalid Code when reading from unaligned zero-sized array

2013-12-08 Thread Richard Biener
Eric Botcazou ebotca...@adacore.com wrote: It's not fully fixing the issue as _all_ aggregates that may be accessed beyond their declarations size are broken. Sure, but we don't need to support such nonsense in the general case. And not every language allows it, for example in Ada you cannot

Re: [REPOST] Invalid Code when reading from unaligned zero-sized array

2013-12-08 Thread Richard Biener
Eric Botcazou ebotca...@adacore.com wrote: That being said, the concern is certainly valid so we may want to go for a kludge instead of the fix. The point is that the kludge should do exactly what the fix would have done in the RTL expander and nothing more; it's out of question to pessimize

Re: [wide-int] small cleanup in wide-int.*

2013-12-08 Thread Richard Biener
Richard Sandiford rdsandif...@googlemail.com wrote: Kenneth Zadeck zad...@naturalbridge.com writes: #define WIDE_INT_MAX_ELTS \ - ((4 * MAX_BITSIZE_MODE_ANY_INT + HOST_BITS_PER_WIDE_INT - 1) \ - / HOST_BITS_PER_WIDE_INT) + (((MAX_BITSIZE_MODE_ANY_INT + HOST_BITS_PER_WIDE_INT - 1) \

Remove regexp and expected output from gtest pass/fail line

2013-12-08 Thread Richard Sandiford
I sent a patch to upstream dejagnu so that things like: FAIL: c-c++-common/asan/global-overflow-1.c -Os output pattern test, is ==4731 ==ERROR: AddressSanitizer failed to allocate 0xdfff0001000 (15392894357504) byte s at address 0x02008fff7000 (12) ==4731==ReserveShadowMemoryRange failed while

Re: Remove regexp and expected output from gtest pass/fail line

2013-12-08 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: I sent a patch to upstream dejagnu so that things like: FAIL: c-c++-common/asan/global-overflow-1.c -Os output pattern test, is ==4731 ==ERROR: AddressSanitizer failed to allocate 0xdfff0001000 (15392894357504) byte s at address

[Patch, Fortran] PRs 59103/58676/41724 - honour pure-/elementalness of intrinsics, add elemental checks

2013-12-08 Thread Tobias Burnus
This patch is about rejects-valid and accepts-invalid and does essentially only: a) It ensures that sym-attr.pure/elemental gets set for pure/elemental intrinsics (isym-pure/elemental). b) It rejects dummy procedures / procedure pointers which are ELEMENTAL. * * * To quote (link see PR)

Re: [PATCH] Fix PR 59390

2013-12-08 Thread Uros Bizjak
On Fri, Dec 6, 2013 at 8:33 PM, Sriraman Tallam tmsri...@google.com wrote: Patch updated with two more tests to check if the vfmadd insn is being produced when possible. Thanks Sri On Fri, Dec 6, 2013 at 11:12 AM, Sriraman Tallam tmsri...@google.com wrote: Hi, I have attached a patch

Re: Remove regexp and expected output from gtest pass/fail line

2013-12-08 Thread Jakub Jelinek
On Sun, Dec 08, 2013 at 10:31:56AM +, Richard Sandiford wrote: --- gcc/testsuite/lib/asan-dg.exp (revision 205654) +++ gcc/testsuite/lib/asan-dg.exp (working copy) @@ -288,9 +288,10 @@ if { $status == fail } { pass $testname execution test

Re: [Patch, Fortran] PRs 59103/58676/41724 - honour pure-/elementalness of intrinsics, add elemental checks

2013-12-08 Thread Janus Weil
Hi Tobias, first off: I assume the first PR number in the subject line is wrong, since I don't see how it is related to your patch. I guess you meant 58099? This patch is about rejects-valid and accepts-invalid and does essentially only: a) It ensures that sym-attr.pure/elemental gets set

Re: Remove regexp and expected output from gtest pass/fail line

2013-12-08 Thread Richard Sandiford
Jakub Jelinek ja...@redhat.com writes: On Sun, Dec 08, 2013 at 10:31:56AM +, Richard Sandiford wrote: --- gcc/testsuite/lib/asan-dg.exp(revision 205654) +++ gcc/testsuite/lib/asan-dg.exp(working copy) @@ -288,9 +288,10 @@ if { $status == fail } {

[wwwdocs] Re: Joseph Myers joins GCC Steering Committee

2013-12-08 Thread Gerald Pfeifer
And here the web page update for Mark's departure and Joseph joining. Applied. Gerald Index: steering.html === RCS file: /cvs/gcc/wwwdocs/htdocs/steering.html,v retrieving revision 1.36 diff -u -3 -p -r1.36 steering.html ---

Re: Remove regexp and expected output from gtest pass/fail line

2013-12-08 Thread Jakub Jelinek
On Sun, Dec 08, 2013 at 07:18:37PM +, Richard Sandiford wrote: gcc/testsuite/ * lib/asan-dg.exp (asan-gtest): Remove actual and expected output from the pass/fail line and add it to the log instead. Looks ok to me, except the ChangeLog, all I'm seeing is that you are removing

Re: [Patch, Fortran] PRs 59103/58676/41724 - honour pure-/elementalness of intrinsics, add elemental checks

2013-12-08 Thread Tobias Burnus
Hi Janus, Janus Weil wrote: first off: I assume the first PR number in the subject line is wrong, since I don't see how it is related to your patch. I guess you meant 58099? Yes. Well spotted. a) It ensures that sym-attr.pure/elemental gets set for pure/elemental intrinsics

[PATCH, testsuite ]: Cleanup dumps

2013-12-08 Thread Uros Bizjak
Hello! 2013-12-08 Uros Bizjak ubiz...@gmail.com * gcc.dg/macro-fusion-1.c: Cleanup sched2 rtl dump. * gcc.dg/macro-fusion-2.c: Ditto. * gcc.dg/vect/vect-simd-clone-10a.c: Cleanup vect tree dump. * gcc.dg/vect/vect-simd-clone-12a.c: Ditto. Tested on x86_64-pc-linux-gnu,

fixinclude patch for fenv.h on Ubuntu

2013-12-08 Thread Bruce Korb
Adjusted for Richard Biener's patch Index: fixincludes/ChangeLog === --- fixincludes/ChangeLog (revision 205790) +++ fixincludes/ChangeLog (working copy) @@ -1,3 +1,14 @@ +2013-12-07 Bruce Korb bk...@gnu.org + + * inclhack.def:

Re: configure check for flex

2013-12-08 Thread Bruce Korb
On 12/08/13 13:06, Gerald Pfeifer wrote: Lovely. Thank you very much! $ svn diff Index: configure.ac === --- configure.ac(revision 205790) +++ configure.ac(working copy) @@ -1319,10 +1319,17 @@ # Used for

Re: 3 libstdc++ tests fail at random

2013-12-08 Thread Jonathan Wakely
On 7 December 2013 17:28, H.J. Lu hjl.to...@gmail.com wrote: On Sat, Dec 7, 2013 at 9:26 AM, H.J. Lu hjl.to...@gmail.com wrote: On Sat, Dec 7, 2013 at 9:09 AM, Jakub Jelinek ja...@redhat.com wrote: FAIL: 30_threads/async/async.cc execution test async.exe:

[SH, committed] Deprecate mcbranchdi and mcmpeqdi options.

2013-12-08 Thread Oleg Endo
Hello, The attached patch deprecates the SH options mcbranchdi and mcmpeqdi, since they turn out to be not so useful. For example, disabling the cbranchdi pattern results in worse code for -Os than -O1 and enabling the cmpeqdi patterns seems to trigger broken/incomplete code paths. Effectively

[wwwdocs] SH 4.9 changes update

2013-12-08 Thread Oleg Endo
Hello, This is a summary of what has been happening on the SH side during 4.9. Applied. Cheers, Oleg ? sh_changes_49_1.patch Index: htdocs/gcc-4.9/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.9/changes.html,v retrieving

Re: [PATCH PR41488]Recognize more induction variables by simplifying PEELED chrec in scalar evolution

2013-12-08 Thread Jeff Law
On 12/06/13 19:44, Bin.Cheng wrote: Right. Based on reading the archives, it looks like this stuff is/was generated by PRE. I also suspect jump threading can create them. There was talk of throttling PRE to leave things in a form that the IV analysis could more easily digest, but I'm not sure

Re: [REPOST] Invalid Code when reading from unaligned zero-sized array

2013-12-08 Thread Jeff Law
On 12/07/13 03:44, Eric Botcazou wrote: I'd certainly be concerned. Ports have (for better or worse) keyed on BLKmode rather than looking at the underlying types. So if something which was previously SImode or DImode is now BLKmode, there's a nonzero chance we're going to change how it gets

Re: Two build != host fixes

2013-12-08 Thread Alan Modra
On Tue, Dec 03, 2013 at 11:44:46PM -0500, DJ Delorie wrote: Alan Modra amo...@gmail.com writes: Bootstrapped etc. powerpc64-linux. OK mainline and 4.8 branch? * configure.ac (BUILD_CXXFLAGS) Don't use ALL_CXXFLAGS for build != host. recursive call for build != host: Clear

Re: [PATCH PR41488]Recognize more induction variables by simplifying PEELED chrec in scalar evolution

2013-12-08 Thread Bin.Cheng
On Mon, Dec 9, 2013 at 12:00 PM, Jeff Law l...@redhat.com wrote: On 12/06/13 19:44, Bin.Cheng wrote: Right. Based on reading the archives, it looks like this stuff is/was generated by PRE. I also suspect jump threading can create them. There was talk of throttling PRE to leave things in a

Re: [Patch, RTL] Eliminate redundant vec_select moves.

2013-12-08 Thread Kirill Yukhin
Hello, On 05 Dec 16:40, Kirill Yukhin wrote: On 05 Dec 05:30, H.J. Lu wrote: Kirill, can you take a look why it doesn't work for x86? Okay, I'll look at this. I've looked at this. It seems that `CANNOT_CHANGE_MODE_CLASS' is too conservative for x86. In rtlanal.c we have

Re: Two build != host fixes

2013-12-08 Thread Jakub Jelinek
On Mon, Dec 09, 2013 at 03:04:44PM +1030, Alan Modra wrote: On Tue, Dec 03, 2013 at 11:44:46PM -0500, DJ Delorie wrote: Alan Modra amo...@gmail.com writes: Bootstrapped etc. powerpc64-linux. OK mainline and 4.8 branch? * configure.ac (BUILD_CXXFLAGS) Don't use ALL_CXXFLAGS for