gcc-7-20160925 is now available

2016-09-25 Thread gccadmin
Snapshot gcc-7-20160925 is now available on ftp://gcc.gnu.org/pub/gcc/snapshots/7-20160925/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 7 SVN branch with the following options: svn://gcc.gnu.org/svn/gcc/trunk revision

Re: Converting to LRA (calling all maintainers)

2016-09-25 Thread Eric Botcazou
> There is no hurry to kill old reload. As you say, many targets will > not be converted soon. But one day it will be removed. Not in GCC 7, > not in GCC 8 almost certainly. But one day. Certainly not in GCC 8, the top priority is IMO the cc0 thing and you cannot really do both at the same

Successful bootstrap and install of gcc (GCC) 6.2.0 on powerpc64-unknown-linux-gnu

2016-09-25 Thread Aaro Koskinen
Hi, Here's a report of a successful build and install of GCC: $ gcc-6.2.0/config.guess powerpc64-unknown-linux-gnu $ newcompiler/bin/gcc -v Using built-in specs. COLLECT_GCC=newcompiler/bin/gcc

sprintf warning on overlapping output

2016-09-25 Thread Bernd Edlinger
Hi Martin, in the past I have seen (and fixed) code like sprintf(buf, "%s %d", buf, x); that may possibly work by chance, but usually produces undefined results. Do you see a way to enhance the warning for cases where the output buffer overlaps an input buffer? Thanks Bernd.

Re: Converting to LRA (calling all maintainers)

2016-09-25 Thread Segher Boessenkool
On Sun, Sep 25, 2016 at 10:46:55AM +0200, Eric Botcazou wrote: > > There is no hurry to kill old reload. As you say, many targets will > > not be converted soon. But one day it will be removed. Not in GCC 7, > > not in GCC 8 almost certainly. But one day. > > Certainly not in GCC 8, the top

Re: Converting to LRA (calling all maintainers)

2016-09-25 Thread Segher Boessenkool
On Sun, Sep 25, 2016 at 04:28:04PM +0900, Oleg Endo wrote: > > > ANd yes, I see this as a way to deprecating those cc0 targets like > > > the  > > > m68k :-) > > Would be a shame to see m68k go.  There still is time... > > Indeed.  68K is a perfect candidate for addressing mode optimization >

Re: sprintf warning on overlapping output

2016-09-25 Thread Eric Gallager
On 9/25/16, Bernd Edlinger wrote: > Hi Martin, > > in the past I have seen (and fixed) code like > > sprintf(buf, "%s %d", buf, x); > > that may possibly work by chance, but usually > produces undefined results. > > Do you see a way to enhance the warning for cases >

Re: Converting to LRA (calling all maintainers)

2016-09-25 Thread Paul.Koning
> On Sep 25, 2016, at 4:46 AM, Eric Botcazou wrote: > >> There is no hurry to kill old reload. As you say, many targets will >> not be converted soon. But one day it will be removed. Not in GCC 7, >> not in GCC 8 almost certainly. But one day. > > Certainly not in

[Bug target/77738] New: Invalid initialisation of ar.lc register

2016-09-25 Thread sch...@linux-m68k.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77738 Bug ID: 77738 Summary: Invalid initialisation of ar.lc register Product: gcc Version: 7.0 Status: UNCONFIRMED Keywords: wrong-code Severity: normal Priority:

[Bug middle-end/68733] [6/7 Regression] FAIL: libgomp.c/target-29.c (internal compiler error)

2016-09-25 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68733 --- Comment #10 from John David Anglin --- Similar fails: FAIL: c-c++-common/gomp/clauses-2.c (test for errors, line 51) FAIL: c-c++-common/gomp/clauses-2.c (test for excess errors) FAIL: c-c++-common/gomp/clauses-2.c -std=c++98 (test for

[Bug testsuite/77735] New: FAIL: gcc.dg/tree-ssa/builtin-sprintf-warn-1.c (test for warnings, line 358)

2016-09-25 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77735 Bug ID: 77735 Summary: FAIL: gcc.dg/tree-ssa/builtin-sprintf-warn-1.c (test for warnings, line 358) Product: gcc Version: 7.0 Status: UNCONFIRMED Severity:

Re: [PATCH, 5.x/6.x/7.x] Be more conservative in early inliner if FDO is enabled

2016-09-25 Thread Yuan, Pengfei
Hi, May I ask if there is any decision? Regards, Yuan, Pengfei > > > I also like a new param better as it avoids a new magic constant and > > > makes playing with > > > it easier (your patch removes the ability to do statistics like you did > > > via the > > > early-inlining-insns parameter by

[Bug bootstrap/77593] [7 Regression] Bootstrap failure with configure-target-libgfortran " cygwin64 Windows 10 anniversary

2016-09-25 Thread n8tm at aol dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77593 --- Comment #10 from n8tm at aol dot com --- Jerry , Thanks for the efforts and apparent progress. I will return to wired internet territory and the win10 box next weekend. I have the win8.1 laptop here. Sent via the ASUS PadFone X mini, an AT

[Bug c++/77733] New: Add fixit hint suggesting to use std::move

2016-09-25 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77733 Bug ID: 77733 Summary: Add fixit hint suggesting to use std::move Product: gcc Version: 7.0 Status: UNCONFIRMED Keywords: diagnostic Severity: normal Priority:

[v3 PATCH] PR libstdc++/77727

2016-09-25 Thread Ville Voutilainen
Tested on Linux-x64. 2016-09-26 Ville Voutilainen PR libstdc++/77727 * include/std/optional (optional(const optional<_Up>&)): Default-initialize the base and use emplace. (optional(optional<_Up>&&)): Likewise. *

Re: [PR77719] Fix ICE in pp_string, at pretty-print.c:955

2016-09-25 Thread kugan
On 25/09/16 04:50, kugan wrote: Hi, In make_new_ssa_for_def (tree-reassoc.c) we should use gimple_get_lhs to get lhs instead of gimple_assign_lhs as stmt can be builtins too. Attached patch fixes this. Testcase from PR (attached) seems to fail. I dont any fortran so I didnt try fixing it.

[Bug libstdc++/77727] Unwrapping std::optional constructor is not working for non-transferable object

2016-09-25 Thread ville.voutilainen at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77727 Ville Voutilainen changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug libstdc++/77727] Unwrapping std::optional constructor is not working for non-transferable object

2016-09-25 Thread ville.voutilainen at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77727 --- Comment #2 from Ville Voutilainen --- Patch available: https://gcc.gnu.org/ml/gcc-patches/2016-09/msg01777.html

[Bug middle-end/71509] Bitfield causes load hit store with larger store than load

2016-09-25 Thread anton at samba dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71509 --- Comment #4 from Anton Blanchard --- Created attachment 39683 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39683=edit Another bitop LHS test case Here's another issue found in the Linux kernel. Seems like this should be a single

[Bug c++/77731] New: Parameter pack expansion doesn't work when used to define argument list

2016-09-25 Thread michele.caini at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77731 Bug ID: 77731 Summary: Parameter pack expansion doesn't work when used to define argument list Product: gcc Version: 6.2.0 Status: UNCONFIRMED Severity:

[Bug bootstrap/77593] [7 Regression] Bootstrap failure with configure-target-libgfortran " cygwin64 Windows 10 anniversary

2016-09-25 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77593 --- Comment #9 from Jerry DeLisle --- (In reply to tprince from comment #8) > I show my configure parameters in my test results posts. At some time in > the past, each of them has been important. I don't know if the parameters > quoted by

[Bug target/77730] Fortran performance on aarch64 (6/7 regression heads-up)

2016-09-25 Thread tulipawn at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77730 --- Comment #5 from PeteVine --- OK, I hacked the last two to use -mcpu=cortex-a53 (the first one got switched to -mtune) and the result looks like this: http://openbenchmarking.org/result/1609258-LO-FORTRANAA63 Not exactly the original issues

[Bug testsuite/77736] New: FAIL: gcc.dg/tree-ssa/builtin-sprintf-warn-2.c (test for warnings, line 50)

2016-09-25 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77736 Bug ID: 77736 Summary: FAIL: gcc.dg/tree-ssa/builtin-sprintf-warn-2.c (test for warnings, line 50) Product: gcc Version: 7.0 Status: UNCONFIRMED Severity:

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

2016-09-25 Thread Andris Pavenis
On 09/25/2016 09:10 PM, Arnaud Charlet wrote: Is the above change really necessary? Would be nice to get rid of this extra code. The rest looks OK to me. It is required for support of environment when LFN support is either not available or disabled for some reason (for example FreeDOS without

[Bug tree-optimization/77737] New: FAIL: gcc.dg/tree-ssa/scev-5.c scan-tree-dump-times optimized "" 1

2016-09-25 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77737 Bug ID: 77737 Summary: FAIL: gcc.dg/tree-ssa/scev-5.c scan-tree-dump-times optimized "" 1 Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal

[Bug ipa/77732] New: FAIL: gcc.dg/ipa/ipcp-cstagg-7.c scan-ipa-dump-times cp "Discovered an indirect call to a known target" 3

2016-09-25 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77732 Bug ID: 77732 Summary: FAIL: gcc.dg/ipa/ipcp-cstagg-7.c scan-ipa-dump-times cp "Discovered an indirect call to a known target" 3 Product: gcc Version: 7.0 Status:

[Bug c++/77733] Add fixit hint suggesting to use std::move

2016-09-25 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77733 --- Comment #1 from Jonathan Wakely --- When template argument deduction is involved and the list of candidates is displayed, the fixit becomes more useful, as the relevant function can get lost in the list of failed overload resolution

[Bug testsuite/77734] New: FAIL: gcc.dg/plugin/must-tail-call-1.c -fplugin=./must_tail_call_plugin.so (test for excess errors)

2016-09-25 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77734 Bug ID: 77734 Summary: FAIL: gcc.dg/plugin/must-tail-call-1.c -fplugin=./must_tail_call_plugin.so (test for excess errors) Product: gcc Version: 7.0

[Bug c++/77739] New: internal compiler error: in create_tmp_var, at gimple-expr.c:524

2016-09-25 Thread vcato1701 at yahoo dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77739 Bug ID: 77739 Summary: internal compiler error: in create_tmp_var, at gimple-expr.c:524 Product: gcc Version: 5.4.0 Status: UNCONFIRMED Severity: normal

[Bug target/51244] [SH] Inefficient conditional branch and code around T bit

2016-09-25 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51244 --- Comment #87 from Oleg Endo --- Author: olegendo Date: Sun Sep 25 06:59:37 2016 New Revision: 240471 URL: https://gcc.gnu.org/viewcvs?rev=240471=gcc=rev Log: This fixes a fallout that actually goes back to 5.0 but went unnoticed. The costs

[Bug libstdc++/65122] std::vector doesn't honor element alignment

2016-09-25 Thread glisse at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65122 --- Comment #19 from Marc Glisse --- (In reply to Andrew Pinski from comment #17) > I think this is fixed for GCC 7 with -std=c++17 support. No, it isn't. new T[10] will give suitably aligned memory, but not std::allocator. Only the core part

[Bug tree-optimization/58122] loops are not evaluated at compile time if loop count > 17

2016-09-25 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58122 --- Comment #5 from Oleg Endo --- This issue seems to be working just fine. Not sure what kind of test case to add for this though... just scanning final assembler code for some expected hex or dec constant?

[PATCH, PR77558] Remove RECORD_TYPE special-casing in std_canonical_va_list_type

2016-09-25 Thread Tom de Vries
Hi, this patch fixes PR77558, an ice-on-invalid-code 6/7 regression. The fix for PR71602 introduced the invalid-code test-case c-c++-common/va-arg-va-list-type.c: ... __builtin_va_list *pap; void fn1 (void) { __builtin_va_arg (pap, double); /* { dg-error "first argument to 'va_arg' not of

[Bug middle-end/77558] [6/7 regression] c-c++-common/va-arg-va-list-type.c fails for arm/aarch64

2016-09-25 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77558 Tom de Vries changed: What|Removed |Added Keywords||patch --- Comment #7 from Tom de Vries

[PATCH] Make CHECKING_P a boolean flag again

2016-09-25 Thread Bernd Edlinger
Hi! Currently CHECKING_P is not a boolean flag but a ternary option. However the _P in the name implies it is a boolean. That should be cleaned up again IMHO. So this patch splits CHECKING_P into CHECKING_P and a new flag ENABLE_EXTRA_CHECKING. All uses of CHECKING_P are actually of the form

[Bug c++/77725] An example from the standard regarding member lookup fails to compile

2016-09-25 Thread tamiraviv at mail dot tau.ac.il
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77725 Tamir Aviv changed: What|Removed |Added CC||tamiraviv at mail dot tau.ac.il ---

[Bug tree-optimization/56365] [5 Regression] Missed opportunities for smin/smax standard name patterns when compiling as C++

2016-09-25 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56365 --- Comment #14 from Oleg Endo --- Richi, if you're not going to backport any patches, maybe close this one as fixed?

[Bug tree-optimization/77719] [7 Regression] ICE in pp_string, at pretty-print.c:955

2016-09-25 Thread Joost.VandeVondele at mat dot ethz.ch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77719 --- Comment #6 from Joost VandeVondele --- (In reply to kugan from comment #5) > Sent a patch to fix this at > https://gcc.gnu.org/ml/gcc-patches/2016-09/msg01760.html. Thanks, add this line before the first IF statement to silence the

[Bug rtl-optimization/77714] Wrong code generation for gcc.c-torture/execute/pr51447.c

2016-09-25 Thread edlinger at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77714 --- Comment #3 from Bernd Edlinger --- because a different pattern is choosen for this insn the reg-note is overwritten here: Choosing alt 2 in insn 48: (0) l (1) l (2) lL {*thumb1_addsi3} Creating newreg=129, assigning class

[SH][committed] Fix cset_zero pattern regressions

2016-09-25 Thread Oleg Endo
Hi, This fixes a fallout that actually goes back to 5.0 but went unnoticed. The costs for movt and movrt type of insns were not correctly reported and ifcvt thus made some bad choices for SH.  A new cset_zero pattern variant is also required to fix the matching for some recent changes in the

[PATCH] Make -Wint-in-bool-context warn on suspicious shift ops

2016-09-25 Thread Bernd Edlinger
Hi! This patch makes -Wint-in-bool-context warn on suspicious integer left shifts, when the integer is signed, which is most likely some kind of programming error, for instance using "<<" instead of "<". The warning is motivated by the fact, that an overflow on integer shift left is undefined

[Bug target/71767] Endless stream of warnings when using GCC with -Wa,-q and Clang Integrated Assembler

2016-09-25 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71767 --- Comment #24 from Dominique d'Humieres --- The patch in comment 20 (and SDK 10.9) causes the failures of the tests gcc.dg/torture/darwin-cfstring-3.c and g++.dg/torture/darwin-cfstring-3.C with -m32, see

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread j...@jak-linux.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 --- Comment #3 from Julian Andres Klode --- Created attachment 39677 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39677=edit arm (thumb2) output at -O2

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread j...@jak-linux.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 --- Comment #2 from Julian Andres Klode --- Created attachment 39676 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39676=edit Aarch64 output at -O2

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread j...@jak-linux.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 --- Comment #1 from Julian Andres Klode --- Created attachment 39675 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39675=edit C reproducer

Re: [PATCH 6/17][ARM] Add data processing intrinsics for float16_t.

2016-09-25 Thread Christophe Lyon
Hi Matthew, On 27 July 2016 at 15:59, Ramana Radhakrishnan wrote: > On Tue, May 17, 2016 at 3:31 PM, Matthew Wahab > wrote: >> The ACLE specifies a number of intrinsics for manipulating vectors >> holding values in most of the integer and

[PATCH] x86_64: fix 'gather' avx2 builtin names

2016-09-25 Thread Sergei Trofimovich
From: Sergei Trofimovich Today I traced AVX2 optimisation bug in gcc and distilled it down to '__builtin_ia32_gatheraltdiv4si256' generated by gcc. When I attempted to use this builtin directly in a simple program gcc refused to recognise it as known: #include void a

[v3 PATCH] PR libstdc++/77717

2016-09-25 Thread Ville Voutilainen
Tested on Linux-x64. 2016-09-25 Ville Voutilainen PR libstdc++/77717 * testsuite/21_strings/basic_string_view/operations/compare/char/1.cc: Fix an out-of-bounds access. diff --git

[Bug target/77729] New: aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread j...@jak-linux.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 Bug ID: 77729 Summary: aarch64 inserts unneeded uxtb after ldrb, orr ...32 Product: gcc Version: 6.1.1 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 Andrew Pinski changed: What|Removed |Added CC||pinskia at gcc dot gnu.org --- Comment

[Bug testsuite/77411] object-size-9.c -fpic -m32 failure

2016-09-25 Thread edlinger at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77411 --- Comment #4 from Bernd Edlinger --- Author: edlinger Date: Sun Sep 25 15:12:05 2016 New Revision: 240472 URL: https://gcc.gnu.org/viewcvs?rev=240472=gcc=rev Log: 2016-09-25 Bernd Edlinger backport from

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread fw at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 Florian Weimer changed: What|Removed |Added CC||fw at gcc dot gnu.org --- Comment #7

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 --- Comment #8 from Andrew Pinski --- This looks like missing removal of casts. Note in C, char_var|32 is really the same as ((int)char_var)|32

[Bug middle-end/77730] Fortran performance on aarch64 (6/7 regression heads-up)

2016-09-25 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77730 --- Comment #1 from Andrew Pinski --- One thing I noticed is the generic vector cost model sucks for almost all cores. It causes to emit ld2/ld3 in cases where it would be more expensive than if doing it scalar.

[Bug target/77730] Fortran performance on aarch64 (6/7 regression heads-up)

2016-09-25 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77730 Andrew Pinski changed: What|Removed |Added Component|middle-end |target --- Comment #2 from Andrew

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread j...@jak-linux.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 --- Comment #5 from Julian Andres Klode --- Created attachment 39678 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39678=edit ppc64le Hmm, AFAICT the same seems to happen on powerpc64le: lbz 9,0(3)

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread j...@jak-linux.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 --- Comment #6 from Julian Andres Klode --- (In reply to Andrew Pinski from comment #4) > Note this testcase needs to be improved as I have a patch which converts a > switch with just one case and a default into anew if

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread j...@jak-linux.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 --- Comment #9 from Julian Andres Klode --- (In reply to Andrew Pinski from comment #8) > This looks like missing removal of casts. > > Note in C, char_var|32 is really the same as ((int)char_var)|32 Well. The loads of the

parallel/algo.h cleanup

2016-09-25 Thread François Dumont
Hi Trivial patch to cleanup parallel/algo.h by generalizing usage of std::__iterator_category. * include/parallel/algo.h: Generalize usage of std::__iterator_category. Adjust whitespaces. Tested under Linux x86_64 parallel mode. François Index: include/parallel/algo.h

[Bug middle-end/77730] New: Fortran performance on aarch64 (6/7 regression heads-up)

2016-09-25 Thread tulipawn at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77730 Bug ID: 77730 Summary: Fortran performance on aarch64 (6/7 regression heads-up) Product: gcc Version: 6.2.1 Status: UNCONFIRMED Severity: normal

[Bug target/77729] aarch64 inserts unneeded uxtb after ldrb, orr ...32

2016-09-25 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77729 Andrew Pinski changed: What|Removed |Added Keywords||missed-optimization

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

2016-09-25 Thread Arnaud Charlet
> int > __gnat_get_maximum_file_name_length (void) > { > +#if defined (__DJGPP__) > + return (_use_lfn(".")) ? -1 : 8; > +#else >return -1; > +#endif > } Is the above change really necessary? Would be nice to get rid of this extra code. The rest looks OK to me. Arno

Re: [PATCH] x86_64: fix 'gather' avx2 builtin names

2016-09-25 Thread Jakub Jelinek
On Sun, Sep 25, 2016 at 02:48:19PM +0100, Sergei Trofimovich wrote: > From: Sergei Trofimovich > > Today I traced AVX2 optimisation bug in gcc and > distilled it down to '__builtin_ia32_gatheraltdiv4si256' > generated by gcc. > > When I attempted to use this builtin

[Bug tree-optimization/77621] [6 Regression] Internal compiler error for mtune=atom + msse2

2016-09-25 Thread uros at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77621 --- Comment #22 from uros at gcc dot gnu.org --- Author: uros Date: Sun Sep 25 17:07:37 2016 New Revision: 240475 URL: https://gcc.gnu.org/viewcvs?rev=240475=gcc=rev Log: Backport from mainline 2016-09-21 Richard Biener

[Bug tree-optimization/77621] [6 Regression] Internal compiler error for mtune=atom + msse2

2016-09-25 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77621 Uroš Bizjak changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/77730] Fortran performance on aarch64 (6/7 regression heads-up)

2016-09-25 Thread tulipawn at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77730 --- Comment #3 from PeteVine --- What about ARMv7? http://openbenchmarking.org/result/1609253-LO-MERGE859124

[Bug target/77730] Fortran performance on aarch64 (6/7 regression heads-up)

2016-09-25 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77730 --- Comment #4 from Andrew Pinski --- (In reply to PeteVine from comment #3) > What about ARMv7? > > http://openbenchmarking.org/result/1609253-LO-MERGE859124 Similar point there. I have seen cortex-a7 and a9 acting way different. Also

[Bug target/71767] Endless stream of warnings when using GCC with -Wa,-q and Clang Integrated Assembler

2016-09-25 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71767 --- Comment #25 from Iain Sandoe --- (In reply to Dominique d'Humieres from comment #24) > The patch in comment 20 (and SDK 10.9) causes the failures of the tests > gcc.dg/torture/darwin-cfstring-3.c and g++.dg/torture/darwin-cfstring-3.C > with

[Bug fortran/77694] ICE in optimize_binop_array_assignment, at fortran/frontend-passes.c:1080

2016-09-25 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77694 --- Comment #5 from kargl at gcc dot gnu.org --- Author: kargl Date: Sun Sep 25 17:30:27 2016 New Revision: 240476 URL: https://gcc.gnu.org/viewcvs?rev=240476=gcc=rev Log: 2016-09-22 Steven G. Kargl PR

[Committed] PR fortran/77694

2016-09-25 Thread Steve Kargl
Committed as "obvious" even though thomas OK patch in bugzilla audit trail. 2016-09-25 Steven G. Kargl PR fortran/77694 * frontend-passes.c (optimize_binop_array_assignment): Check pointer for NULL. 2016-09-25 Steven G. Kargl

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

2016-09-25 Thread Andris Pavenis
On 09/25/2016 07:25 PM, Arnaud Charlet wrote: int __gnat_get_maximum_file_name_length (void) { +#if defined (__DJGPP__) + return (_use_lfn(".")) ? -1 : 8; +#else return -1; +#endif } Is the above change really necessary? Would be nice to get rid of this extra code. The rest looks

[Bug fortran/77694] ICE in optimize_binop_array_assignment, at fortran/frontend-passes.c:1080

2016-09-25 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77694 kargl at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

2016-09-25 Thread Arnaud Charlet
> >Is the above change really necessary? Would be nice to get rid of this > >extra code. The rest looks OK to me. > It is required for support of environment when LFN support is either > not available or disabled for some reason (for example FreeDOS > without LFN support loaded). Does it matter

Re: [PATCH] x86_64: fix 'gather' avx2 builtin names

2016-09-25 Thread Sergei Trofimovich
On Sun, 25 Sep 2016 19:01:14 +0200 Jakub Jelinek wrote: > On Sun, Sep 25, 2016 at 02:48:19PM +0100, Sergei Trofimovich wrote: > > From: Sergei Trofimovich > > > > Today I traced AVX2 optimisation bug in gcc and > > distilled it down to

[Bug tree-optimization/76957] FAIL: gcc.dg/graphite/scop-dsyr2k.c scan-tree-dump-times graphite "number of SCoPs

2016-09-25 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=76957 --- Comment #4 from John David Anglin --- Similar fail on hppa-unknown-linux-gnu: FAIL: gcc.dg/graphite/scop-dsyrk.c scan-tree-dump-times graphite "number of SCoPs: 1" 1

[Bug fortran/77429] ICE in gfc_check_dependency, at fortran/dependency.c:1261

2016-09-25 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77429 --- Comment #3 from kargl at gcc dot gnu.org --- Author: kargl Date: Sun Sep 25 18:46:28 2016 New Revision: 240477 URL: https://gcc.gnu.org/viewcvs?rev=240477=gcc=rev Log: 2016-09-25 Steven G. Kargl PR

[Committed] PR fortran/77429

2016-09-25 Thread Steve Kargl
Committed as obvious. 2016-09-25 Steven G. Kargl PR fortran/77429 * dependency.c (gfc_check_dependency): Convert gcc_assert() to a conditional and possible call to gfc_internal_error(). 2016-09-25 Steven G. Kargl PR

[Bug fortran/77429] ICE in gfc_check_dependency, at fortran/dependency.c:1261

2016-09-25 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77429 kargl at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED