[Bug tree-optimization/21278] Move maximum out of a loop

2016-11-16 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=21278 Thomas Koenig changed: What|Removed |Added Last reconfirmed|2013-02-01 18:00:00 |2016-11-16 --- Comment #2 from Thomas

[Bug debug/78363] [7 Regression] ICE in in force_type_die, at dwarf2out.c:24864

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78363 --- Comment #5 from Richard Biener --- diff --git a/gcc/tree-cfg.c b/gcc/tree-cfg.c index e99e102..e23c8c6 100644 --- a/gcc/tree-cfg.c +++ b/gcc/tree-cfg.c @@ -7054,6 +7054,13 @@ replace_block_vars_by_duplicates (tree block, hash_map

[Bug sanitizer/78270] [7 Regression] ICE in gimplify_switch_expr, at gimplify.c:2272

2016-11-16 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78270 Martin Liška changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---

[Bug rtl-optimization/78378] New: [5/6/7 Regression] wrong code when combining shift + mult + zero_extend

2016-11-16 Thread zsojka at seznam dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78378 Bug ID: 78378 Summary: [5/6/7 Regression] wrong code when combining shift + mult + zero_extend Product: gcc Version: 7.0 Status: UNCONFIRMED Keywords:

Re: [PATCH] Fix PR sanitizer/78270 (part 2)

2016-11-16 Thread Martin Liška
On 11/16/2016 12:35 PM, Jakub Jelinek wrote: > On Fri, Nov 11, 2016 at 02:44:40PM +0100, Martin Liška wrote: >> Hello. >> >> Due to a stupid mistake I did, following patch is needed for the test-case >> to properly save previous gimplify_ctxp->live_switch_vars. >> >> Patch can bootstrap on

[Bug sanitizer/78270] [7 Regression] ICE in gimplify_switch_expr, at gimplify.c:2272

2016-11-16 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78270 --- Comment #6 from Martin Liška --- Author: marxin Date: Wed Nov 16 11:56:58 2016 New Revision: 242485 URL: https://gcc.gnu.org/viewcvs?rev=242485=gcc=rev Log: Fix PR sanitizer/78270 (part 2) PR sanitizer/78270 * gimplify.c

[Bug other/78366] target_clones does not generate resovler function

2016-11-16 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78366 Thomas Koenig changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

RE: [PATCH 3/3] MIPS/GCC: Mark trailing labels with `.insn'

2016-11-16 Thread Maciej W. Rozycki
On Mon, 14 Nov 2016, Matthew Fortune wrote: > > This however requires the correct annotation of branch targets as code, > > because the ISA mode is not relevant for data symbols and is therefore > > not recorded for them. > > I wonder if it would have been possible to add the ISA mode to data >

Re: [Patch, Fortran, F03] PR 77501: ICE in gfc_match_generic, at fortran/decl.c:9429

2016-11-16 Thread Janus Weil
2016-11-12 21:21 GMT+01:00 Janus Weil : Index: gcc/fortran/class.c === --- gcc/fortran/class.c(Revision 242066) +++ gcc/fortran/class.c(Arbeitskopie) @@ -2970,7 +2970,6 @@

[Bug fortran/78377] New: [cleanup] gfc_get_tbp_symtree vs. gfc_get_sym_tree

2016-11-16 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78377 Bug ID: 78377 Summary: [cleanup] gfc_get_tbp_symtree vs. gfc_get_sym_tree Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

RE: [PATCHv2 0/7] ARC: Add support for nps400 variant

2016-11-16 Thread Claudiu Zissulescu
Approved and committed: Committed r24248 //Claudiu > -Original Message- > From: Andrew Burgess [mailto:andrew.burg...@embecosm.com] > Sent: Saturday, April 30, 2016 12:17 AM > To: Claudiu Zissulescu ; Joern Wolfgang > Rennecke > Cc:

[Bug target/78376] invalid ARM 'ubfx' instruction generated

2016-11-16 Thread arnd at linaro dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78376 Arnd Bergmann changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/78364] [7 Regression][ARM] Error: bit-field extends past end of register -- ubfx

2016-11-16 Thread arnd at linaro dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78364 Arnd Bergmann changed: What|Removed |Added CC||arnd at linaro dot org --- Comment #5

Re: [PATCH] Fix PR sanitizer/78270 (part 2)

2016-11-16 Thread Jakub Jelinek
On Fri, Nov 11, 2016 at 02:44:40PM +0100, Martin Liška wrote: > Hello. > > Due to a stupid mistake I did, following patch is needed for the test-case > to properly save previous gimplify_ctxp->live_switch_vars. > > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > I

Re: [PATCH] Fix PR sanitizer/78270 (part 2)

2016-11-16 Thread Martin Liška
I'm pinging this as it breaks ASAN bootstrap. On 11/11/2016 02:44 PM, Martin Liška wrote: > Hello. > > Due to a stupid mistake I did, following patch is needed for the test-case > to properly save previous gimplify_ctxp->live_switch_vars. > > Patch can bootstrap on ppc64le-redhat-linux and

Re: [PATCH] Fix PR78305

2016-11-16 Thread Richard Biener
On Wed, 16 Nov 2016, Marc Glisse wrote: > On Wed, 16 Nov 2016, Richard Biener wrote: > > > I am testing the following to avoid undefined behavior when negating > > a multiplication (basically extending a previous fix to properly handle > > negative power of two). > > > > Bootstrap / regtest

[Bug middle-end/78294] [5/6/7 Regression] -fsanitize=thread broken by ignoring __attribute__((tls_model("initial-exec")))

2016-11-16 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78294 --- Comment #22 from Markus Trippelsdorf --- Author: trippels Date: Wed Nov 16 11:31:18 2016 New Revision: 242483 URL: https://gcc.gnu.org/viewcvs?rev=242483=gcc=rev Log: Fix PR78294 - thread sanitizer broken when using ld.gold When one uses

[Bug libstdc++/78371] list::sort doesn't work with non-default-constructible allocators

2016-11-16 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78371 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug libstdc++/66742] abort on sorting list with custom allocator that is not stateless

2016-11-16 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66742 Jonathan Wakely changed: What|Removed |Added CC||rs2740 at gmail dot com --- Comment

[Bug middle-end/78294] [5/6/7 Regression] -fsanitize=thread broken by ignoring __attribute__((tls_model("initial-exec")))

2016-11-16 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78294 --- Comment #21 from Markus Trippelsdorf --- Author: trippels Date: Wed Nov 16 11:28:57 2016 New Revision: 242482 URL: https://gcc.gnu.org/viewcvs?rev=242482=gcc=rev Log: Fix PR78294 - thread sanitizer broken when using ld.gold When one uses

[Bug target/78376] invalid ARM 'ubfx' instruction generated

2016-11-16 Thread arnd at linaro dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78376 --- Comment #1 from Arnd Bergmann --- Created attachment 40055 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40055=edit preprocessed drivers/net/ethernet/mellanox/mlx5/core/en_rep.c adding gzipped version of the preprocessed source

[Bug middle-end/78294] [5/6/7 Regression] -fsanitize=thread broken by ignoring __attribute__((tls_model("initial-exec")))

2016-11-16 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78294 --- Comment #20 from Markus Trippelsdorf --- Author: trippels Date: Wed Nov 16 11:21:42 2016 New Revision: 242480 URL: https://gcc.gnu.org/viewcvs?rev=242480=gcc=rev Log: Fix PR78294 - thread sanitizer broken when using ld.gold When one uses

[Bug target/78376] New: invalid ARM 'ubfx' instruction generated

2016-11-16 Thread arnd at linaro dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78376 Bug ID: 78376 Summary: invalid ARM 'ubfx' instruction generated Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: target

[Bug middle-end/71441] [7 regression] gcc.dg/pr44023.c FAILs

2016-11-16 Thread ro at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71441 Rainer Orth changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug debug/78363] [7 Regression] ICE in in force_type_die, at dwarf2out.c:24864

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78363 --- Comment #4 from Richard Biener --- Ok, so we eliminate as unused DIE0: DW_TAG_structure_type (0x2c04fb40) abbrev id: 0 offset: 0 mark: 0 DW_AT_name: "" DW_AT_byte_size: 1 DW_AT_decl_file: "t.ii"

[Bug middle-end/71441] [7 regression] gcc.dg/pr44023.c FAILs

2016-11-16 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71441 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #1

[Bug sanitizer/78307] [7 Regression] missing symbols in libubsan without changing the soname

2016-11-16 Thread chefmax at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78307 --- Comment #8 from chefmax at gcc dot gnu.org --- Author: chefmax Date: Wed Nov 16 11:13:19 2016 New Revision: 242478 URL: https://gcc.gnu.org/viewcvs?rev=242478=gcc=rev Log: PR sanitizer/78307 * ubsan/ubsan_handlers.cc

[Bug rtl-optimization/78374] Segfault in cc1 (arm-eabi) on -O1 (or better)

2016-11-16 Thread meisenmann....@fh-salzburg.ac.at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78374 --- Comment #3 from Markus Eisenmann --- Further information on (IMHO) problematic optimization / tests: Option -Og does not cause this segfault (as already written), but the option -Og and -ftree-bit-ccp will

[Bug c++/78373] [7 Regression] error: constant not recomputed when ADDR_EXPR changed

2016-11-16 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78373 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #2

Re: [PATCH][PR sanitizer/78307] Fix missing symbols in libubsan after recent merge.

2016-11-16 Thread Maxim Ostapenko
On 16/11/16 14:00, Jakub Jelinek wrote: On Fri, Nov 11, 2016 at 06:32:43PM +0300, Maxim Ostapenko wrote: Tested on x86_64-unknown-linux-gnu. OK for mainline? Ok (though the LOCAL_PATCHES file is missing in the patch). Yes, I'm going to commit it in separate revision.

Re: [patch, nios2] Fix PR78357, adjust sync builtin initialization

2016-11-16 Thread Sebastian Huber
Thanks, Chung-Lin. This fixes the problem for RTEMS. The TLS support works also. On 16/11/16 10:53, Chung-Lin Tang wrote: This patch adjusts the initialization of __sync built-in functions: instead of conditionalizing on TARGET_LINUX_ABI, directly place the target-hook #define in

[Bug tree-optimization/78319] [7 Regression] PASS->FAIL: gcc.dg/uninit-pred-8_a.c bogus warning (test for bogus messages, line 20)

2016-11-16 Thread prathamesh3492 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78319 --- Comment #8 from prathamesh3492 at gcc dot gnu.org --- (In reply to Richard Biener from comment #7) > (In reply to prathamesh3492 from comment #6) > > (In reply to Richard Biener from comment #5) > > > It's a matter of costs (here BRANCH_COST

Re: [PATCH][PR sanitizer/78307] Fix missing symbols in libubsan after recent merge.

2016-11-16 Thread Jakub Jelinek
On Fri, Nov 11, 2016 at 06:32:43PM +0300, Maxim Ostapenko wrote: > Tested on x86_64-unknown-linux-gnu. OK for mainline? Ok (though the LOCAL_PATCHES file is missing in the patch). > libsanitizer/ChangeLog: > > 2016-11-11 Maxim Ostapenko > > PR sanitizer/78307

Re: [PATCH] Fix PR78305

2016-11-16 Thread Marc Glisse
On Wed, 16 Nov 2016, Richard Biener wrote: I am testing the following to avoid undefined behavior when negating a multiplication (basically extending a previous fix to properly handle negative power of two). Bootstrap / regtest running on x86_64-unknown-linux-gnu. Richard. 2016-11-16

[PATCH] Fix PR78294 - thread sanitizer broken when using ld.gold

2016-11-16 Thread Markus Trippelsdorf
When one uses ld.gold to build gcc, the thread sanitizer doesn't work, because gold is more conservative when applying TLS relaxations than ld.bfd. In this case a missing initial-exec attribute on a declaration causes gcc to assume the general dynamic model. With ld.bfd this gets relaxed to

RE: [PATCH 4/4] MIPS16/GCC: Emit explicit JRC from`casesi_internal_mips16_' insn

2016-11-16 Thread Matthew Fortune
Maciej Rozycki writes: > gcc/ > * config/mips/mips.md (casesi_internal_mips16_): > Explicitly switch between JR and JRC for the table jump. Adjust > instruction count. > --- > OK to apply? OK. Thanks, Matthew

RE: [PATCH 3/4] MIPS16/GCC: Improve `casesi_internal_mips16_'s instruction count estimate

2016-11-16 Thread Matthew Fortune
Maciej Rozycki writes: > gcc/ > * config/mips/mips.md (casesi_internal_mips16_): Set > `insn_count' to 11 rather than 16. > --- > OK to apply? Good catch again. OK. Thanks, Matthew

RE: [PATCH 2/4] MIPS16/GCC: Correct `casesi_internal_mips16_'s RTL pattern

2016-11-16 Thread Matthew Fortune
Maciej Rozycki writes: > gcc/ > * config/mips/mips.md (casesi_internal_mips16_): Use the > `ltu' rather than `leu' operation in the RTL pattern > --- > OK to apply? Good spot. OK to commit. Thanks, Matthew

RE: [PATCH 1/4] MIPS16/GCC: Fix DImode `casesi_internal_mips16_' assembly instructions

2016-11-16 Thread Matthew Fortune
Maciej Rozycki writes: > gcc/ > config/mips/mips.md (casesi_internal_mips16_): Add missing >instruction prefixes throughout. Correct formatting. > > gcc/testsuite/ > * gcc.target/mips/code-readable-4.c (dg-final): Expect `dla' >

[Bug tree-optimization/78319] [7 Regression] PASS->FAIL: gcc.dg/uninit-pred-8_a.c bogus warning (test for bogus messages, line 20)

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78319 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug rtl-optimization/78374] Segfault in cc1 (arm-eabi) on -O1 (or better)

2016-11-16 Thread meisenmann....@fh-salzburg.ac.at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78374 --- Comment #2 from Markus Eisenmann --- Created attachment 40054 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40054=edit Preprocessed file Preprocessed file, as requested by ktkachov

RE: [PATCH] microMIPS/GCC: Fix PIC call relaxation

2016-11-16 Thread Matthew Fortune
Maciej Rozycki writes: > Fix `-mrelax-pic-calls' support for microMIPS code where the relocation > produced is supposed to be R_MICROMIPS_JALR rather than R_MIPS_JALR. > The lack of short delay support comes from a missed update to this code > for microMIPS support and

[PATCH] [ARC] Add support for QuarkSE processor.

2016-11-16 Thread Claudiu Zissulescu
Updated QuarkSE patch. Ok to apply? Claudiu gcc/ 2016-05-25 Claudiu Zissulescu * config/arc/arc-arches.def: Add FPX quarkse instruction as valid for arcem. * config/arc/arc-c.def (__ARC_FPX_QUARK__): Define. * config/arc/arc-cpus.def

Re: [PATCH, GCC/ARM] Fix PR77904: callee-saved register trashed when clobbering sp

2016-11-16 Thread Kyrill Tkachov
Hi Thomas, On 03/11/16 16:52, Thomas Preudhomme wrote: Hi, When using a callee-saved register to save the frame pointer the Thumb-1 prologue fails to save the callee-saved register before that. For ARM and Thumb-2 targets the frame pointer is handled as a special case but nothing is done for

Re: An alternative fix for PR70944

2016-11-16 Thread Segher Boessenkool
On Wed, Nov 16, 2016 at 10:16:16AM +, Richard Sandiford wrote: > Segher Boessenkool writes: > > On Tue, Nov 15, 2016 at 12:33:06PM +, Richard Sandiford wrote: > >> The transformations made by make_compound_operation apply > >> only to scalar integer modes. The

Re: Add more subreg offset helpers

2016-11-16 Thread Eric Botcazou
> subreg_offset_from_lsb was supposed to be the inverse operation of > subreg_lsb, which also returns a bit number. It would have helped the reviewer to state it in the function comment. ;-) > Should I change that to return a byte number as well? Both functions are fine as-is, but mention that

[Bug tree-optimization/78319] [7 Regression] PASS->FAIL: gcc.dg/uninit-pred-8_a.c bogus warning (test for bogus messages, line 20)

2016-11-16 Thread prathamesh3492 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78319 --- Comment #6 from prathamesh3492 at gcc dot gnu.org --- (In reply to Richard Biener from comment #5) > It's a matter of costs (here BRANCH_COST and its uses in fold and ifcombine). > > You don't mention what IL differences your patch causes

[Bug tree-optimization/70965] [7 Regression] ICE on released SSA name during IPA SRA

2016-11-16 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70965 --- Comment #8 from Jakub Jelinek --- Created attachment 40053 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40053=edit gcc7-pr70965.patch This untested patch fixes the ICE, but the question is if it is the right spot to do that, or if

[PATCH 0/4] [ARC] Series of small patches to fix dejagnu found issues.

2016-11-16 Thread Claudiu Zissulescu
Hi, This is a series of four pathes that are solving a number of dejagnu found errors for ARC600 and ARCHS. 1st patch solves ARCHS issues when ifconverting movsi.ne instructions. 2nd patch cleans/refactors a bit our implementation. 3rd patch reimplements the mul64 support resolving all dejagnu

[PATCH 3/4] [ARC] Refurbish mul64 support.

2016-11-16 Thread Claudiu Zissulescu
gcc/ 2016-07-04 Claudiu Zissulescu * config/arc/arc.md (mulsidi_600): Changed. (umulsidi_600): Likewise. (mul64): New pattern. (mulu64): Likewise. (mulsidi3): Changed. (umulsidi3): Likewise. --- gcc/config/arc/arc.md | 64

[PATCH 4/4] [ARC] Fix compilation issue in pr71872.

2016-11-16 Thread Claudiu Zissulescu
gcc/ 2016-07-21 Claudiu Zissulescu * config/arc/arc.c (arc_ccfsm_post_advance): Handle return instruction type. --- gcc/config/arc/arc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/gcc/config/arc/arc.c b/gcc/config/arc/arc.c

[PATCH 2/4] [ARC] Cleanup implementation.

2016-11-16 Thread Claudiu Zissulescu
gcc/ 2016-06-30 Claudiu Zissulescu * config/arc/arc-protos.h (insn_is_tls_gd_dispatch): Remove. * config/arc/arc.c (arc_unspec_offset): New function. (arc_finalize_pic): Change. (arc_emit_call_tls_get_addr): Likewise.

Re: Rework subreg_get_info

2016-11-16 Thread Eric Botcazou
> This isn't intended to change the behaviour, just rewrite the > existing logic in a different (and hopefully clearer) way. Yes, I agree that it's an improvement. A few remarks below. > diff --git a/gcc/rtlanal.c b/gcc/rtlanal.c > index ca6cced..7c0acf5 100644 > --- a/gcc/rtlanal.c > +++

[PATCH 1/4] [ARC] Various fixes.

2016-11-16 Thread Claudiu Zissulescu
The ifconversion was failing because a move involving the lp_count was not match by movsi_ne. This patch updates the constraints such that movsi_ne will match. The failing test is dg-torture.exp=pr68955.c for archs and without small data. gcc/ 2016-07-11 Claudiu Zissulescu

[Bug tree-optimization/70965] [7 Regression] ICE on released SSA name during IPA SRA

2016-11-16 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70965 Jakub Jelinek changed: What|Removed |Added CC||hubicka at gcc dot gnu.org,

Re: An alternative fix for PR70944

2016-11-16 Thread Richard Sandiford
Segher Boessenkool writes: > On Tue, Nov 15, 2016 at 12:33:06PM +, Richard Sandiford wrote: >> The transformations made by make_compound_operation apply >> only to scalar integer modes. The fix for PR70944 had enforced >> that by returning early for vector modes

[Bug tree-optimization/78319] [7 Regression] PASS->FAIL: gcc.dg/uninit-pred-8_a.c bogus warning (test for bogus messages, line 20)

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78319 --- Comment #5 from Richard Biener --- It's a matter of costs (here BRANCH_COST and its uses in fold and ifcombine). You don't mention what IL differences your patch causes (I'll check soon myself). For the issue at hand I suggest to XFAIL for

[committed] Fix pdp11 build

2016-11-16 Thread Richard Sandiford
Needed this to test the effect of the SVE patches on other targets. Tested on pdp11 and committed as obvious. Thanks, Richard gcc/ * config/pdp11/pdp11.c: Include dbxout.h. diff --git a/gcc/config/pdp11/pdp11.c b/gcc/config/pdp11/pdp11.c index 1dc3eef..8e2de14 100644 ---

[Bug rtl-optimization/78374] Segfault in cc1 (arm-eabi) on -O1 (or better)

2016-11-16 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78374 ktkachov at gcc dot gnu.org changed: What|Removed |Added CC||ktkachov at gcc dot gnu.org

[ARC] Fix missing brackets in arc.c

2016-11-16 Thread Richard Sandiford
The old code still built thanks to the brackets in the definition of XVECEXP. Tested on arc-elf and committed as obvious. Thanks, Richard gcc/ * config/arc/arc.c (arc_loop_hazard): Add missing brackets. diff --git a/gcc/config/arc/arc.c b/gcc/config/arc/arc.c index 3bce7ef..98c7298

Re: [1/9][RFC][DWARF] Reserve three DW_OP numbers in vendor extension space

2016-11-16 Thread Jiong Wang
On 15/11/16 19:25, Richard Earnshaw (lists) wrote: On 15/11/16 16:48, Jiong Wang wrote: On 15/11/16 16:18, Jakub Jelinek wrote: I know nothing about the aarch64 return address signing, would all 3 or say 2 usually appear together without any separate pc advance, or are they all going to appear

[Bug tree-optimization/78332] [ARM] Negative costs of ivopts groups

2016-11-16 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78332 amker at gcc dot gnu.org changed: What|Removed |Added CC||amker at gcc dot gnu.org ---

Re: [PATCH, vec-tails] Support loop epilogue vectorization

2016-11-16 Thread Richard Biener
On Tue, 15 Nov 2016, Yuri Rumyantsev wrote: > Hi All, > > Here is patch for non-masked epilogue vectoriziation. > > Bootstrap and regression testing did not show any new failures. > > Is it OK for trunk? Ok for trunk. I believe we ultimatively want to remove the new --param and enable this

Re: Add more subreg offset helpers

2016-11-16 Thread Richard Sandiford
Eric Botcazou writes: >> Provide versions of subreg_lowpart_offset and subreg_highpart_offset >> that work on mode sizes rather than modes. Also provide a routine >> that converts an lsb position to a subreg offset. > > The position is in bits but all the other parameters

[patch, nios2] Fix PR78357, adjust sync builtin initialization

2016-11-16 Thread Chung-Lin Tang
This patch adjusts the initialization of __sync built-in functions: instead of conditionalizing on TARGET_LINUX_ABI, directly place the target-hook #define in config/nios2/linux.h. This appears to be in line with other similar ports, e.g. m68k. Sebastian, this should solve your issue of not

[Bug debug/78375] New: Fortran, pointer to structure, data_member_location

2016-11-16 Thread bernhard.heckel at intel dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78375 Bug ID: 78375 Summary: Fortran, pointer to structure, data_member_location Product: gcc Version: 6.2.1 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug sanitizer/78267] [7 Regression] libsanitizer breaks bootstrap on x86_64-apple-darwin16 at r241977

2016-11-16 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78267 --- Comment #51 from Iain Sandoe --- (In reply to r...@cebitec.uni-bielefeld.de from comment #50) > > --- Comment #49 from Iain Sandoe --- > [...] > > I can do darwin14 (I built 242408 last night with the patches-in-progress + > > __BLOCKS__)

Re: [PATCH] Fix PR78306

2016-11-16 Thread Richard Biener
On Tue, 15 Nov 2016, Richard Biener wrote: > > Appearantly for some unknown reason we refuse to inline anything into > functions calling cilk_spawn. That breaks fortified headers and > all other always-inline function calls (intrinsics come to my mind as > well). > > Bootstrapped and tested

[Bug rtl-optimization/78374] New: Segfault in cc1 (arm-eabi) on -O1 (or better)

2016-11-16 Thread meisenmann....@fh-salzburg.ac.at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78374 Bug ID: 78374 Summary: Segfault in cc1 (arm-eabi) on -O1 (or better) Product: gcc Version: 5.4.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Re: [PATCH, GCC/ARM] Fix ICE when compiling empty FIQ interrupt handler in ARM mode

2016-11-16 Thread Kyrill Tkachov
On 09/11/16 16:19, Thomas Preudhomme wrote: Hi, This patch fixes the following ICE when building when compiling an empty FIQ interrupt handler in ARM mode: empty_fiq_handler.c:5:1: error: insn does not satisfy its constraints: } ^ (insn/f 13 12 14 (set (reg/f:SI 13 sp) (plus:SI

[PATCH] Fix PR78333

2016-11-16 Thread Richard Biener
Since GCC 4.6 we aggressively prune bodies of GNU extern inline functions which means that instrumenting them via -finstrument-functions doesn't work because that takes the address of the function. Fixed by not instrumenting those functions (we still instrument regular always-inline functions

[Patch, testsuite] Fix bogus Wlogical-op-1.c test failure for avr

2016-11-16 Thread Senthil Kumar Selvaraj
Hi, The below patch fixes c-c++-common/Wlogical-op-1.c for avr by explicitly typedef'ing __INT32_TYPE for int and __INT16_TYPE__ for short if the target's int size is less than 4 bytes. The test assumes short is always smaller than int, and therefore does not expect a warning when the

[PATCH] Fix PR78305

2016-11-16 Thread Richard Biener
I am testing the following to avoid undefined behavior when negating a multiplication (basically extending a previous fix to properly handle negative power of two). Bootstrap / regtest running on x86_64-unknown-linux-gnu. Richard. 2016-11-16 Richard Biener PR

[Bug tree-optimization/78305] Wrong constant folding

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78305 --- Comment #5 from Richard Biener --- So it's negating a * -1073741824 via case MULT_EXPR: if (TYPE_UNSIGNED (type)) break; /* INT_MIN/n * n doesn't overflow while negating one operand it does if n is a power

[Bug sanitizer/78267] [7 Regression] libsanitizer breaks bootstrap on x86_64-apple-darwin16 at r241977

2016-11-16 Thread ro at CeBiTec dot Uni-Bielefeld.DE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78267 --- Comment #50 from ro at CeBiTec dot Uni-Bielefeld.DE --- > --- Comment #49 from Iain Sandoe --- [...] > I can do darwin14 (I built 242408 last night with the patches-in-progress + > __BLOCKS__) but that's a little bit more than the minimum >

[Bug sanitizer/78267] [7 Regression] libsanitizer breaks bootstrap on x86_64-apple-darwin16 at r241977

2016-11-16 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78267 --- Comment #49 from Iain Sandoe --- (In reply to r...@cebitec.uni-bielefeld.de from comment #48) > > --- Comment #47 from Maxim Ostapenko --- > [...] > > Rainer, sorry for a dumb question: are you going to commit your fix for > >

[Bug c++/78323] pathological code generation for long logical expression with temporary objects

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78323 Richard Biener changed: What|Removed |Added Keywords||missed-optimization

[Bug target/78364] [7 Regression][ARM] Error: bit-field extends past end of register -- ubfx

2016-11-16 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78364 ktkachov at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug tree-optimization/78305] Wrong constant folding

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78305 Richard Biener changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug sanitizer/78267] [7 Regression] libsanitizer breaks bootstrap on x86_64-apple-darwin16 at r241977

2016-11-16 Thread ro at CeBiTec dot Uni-Bielefeld.DE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78267 --- Comment #48 from ro at CeBiTec dot Uni-Bielefeld.DE --- > --- Comment #47 from Maxim Ostapenko --- [...] > Rainer, sorry for a dumb question: are you going to commit your fix for > darwin_availabilityinternal part? Or should I just apply a

[Bug c/78365] [7 Regression] ICE in determine_value_range, at tree-ssa-loo p-niter.c:413

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78365 --- Comment #5 from Richard Biener --- IPA has to deal with argument mismatches (I think I've said this elsewhere).

[Bug target/78364] [7 Regression][ARM] Error: bit-field extends past end of register -- ubfx

2016-11-16 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78364 --- Comment #3 from ktkachov at gcc dot gnu.org --- Author: ktkachov Date: Wed Nov 16 09:02:18 2016 New Revision: 242471 URL: https://gcc.gnu.org/viewcvs?rev=242471=gcc=rev Log: [ARM] PR target/78364: Add proper restrictions to zero and

[Bug middle-end/78333] [5/6/7 Regression] always-inline gnu-inline functions break -finstrument-functions

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78333 Richard Biener changed: What|Removed |Added Status|NEW |ASSIGNED Known to work|

[Bug middle-end/78306] [CilkPlus] "inlining failed in call to always_inline ‘memset’: function not inlinable" with -fcilkplus

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78306 Richard Biener changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug other/78366] target_clones does not generate resovler function

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78366 --- Comment #2 from Richard Biener --- Note it is created in the caller context, thus double foo(double *, double *, int) __attribute__((target_clones("avx,default"))); double x; int main() { double a,b; x = foo (, , 1); } creates it in

[Bug other/78366] target_clones does not generate resovler function

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78366 --- Comment #1 from Richard Biener --- The resolver is only created when there is a call to the function.

[Bug tree-optimization/78348] [7 REGRESSION] 15% performance drop for coremark-pro/nnet-test after r242038

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78348 Richard Biener changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[PATCH][ARM] PR target/78364: Add proper restrictions to zero and sign_extract patterns operands

2016-11-16 Thread Kyrill Tkachov
Hi all, This patch fixes the arm build failure due to out of range ubfx operands. Combine now more aggressively generates zero_extracts and it's up to the backend to reject invalid bit offsets and widths. And arm seems to suffer from the same problems as aarch64 and s390 did in PR 77822. My

[Bug tree-optimization/78348] [7 REGRESSION] 15% performance drop for coremark-pro/nnet-test after r242038

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78348 --- Comment #7 from Richard Biener --- Author: rguenth Date: Wed Nov 16 08:42:20 2016 New Revision: 242470 URL: https://gcc.gnu.org/viewcvs?rev=242470=gcc=rev Log: 2016-11-16 Richard Biener PR

[Bug middle-end/78370] Missing uninitialzed warning

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78370 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/78373] [7 Regression] error: constant not recomputed when ADDR_EXPR changed

2016-11-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78373 Richard Biener changed: What|Removed |Added Target Milestone|--- |7.0

[PATCH] Fix PR78348

2016-11-16 Thread Richard Biener
This should fix a performance regression caused by recent loop distribution improvements. It trivially uses dependence analysis (in addition to the existing alias oracle query) to determine if we can use memcpy instead of memmove (no attempt is made yet to cover the case where the dependence

[Bug sanitizer/77823] [7 Regression] ICE: in ubsan_encode_value, at ubsan.c:137 with -fsanitize=undefined and vector types

2016-11-16 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77823 --- Comment #8 from Jakub Jelinek --- Author: jakub Date: Wed Nov 16 08:28:50 2016 New Revision: 242469 URL: https://gcc.gnu.org/viewcvs?rev=242469=gcc=rev Log: PR sanitizer/77823 * ubsan.c (ubsan_build_overflow_builtin): Add

Re: [PATCH] Support -fsanitize=integer-arith-overflow even for vectors (PR sanitizer/77823)

2016-11-16 Thread Richard Biener
On Tue, 15 Nov 2016, Jakub Jelinek wrote: > Hi! > > On Mon, Nov 14, 2016 at 10:58:51AM +0100, Jakub Jelinek wrote: > > Working virtually out of Samoa. > > > > The following patch is an attempt to handle -fsanitize=undefined > > for vectors. We already diagnose out of bounds accesses for vector

[Bug fortran/78299] [6/7 Regression] ICE in expand_omp_for_static_nochunk, at omp-low.c:9622

2016-11-16 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78299 Jakub Jelinek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug sanitizer/78267] [7 Regression] libsanitizer breaks bootstrap on x86_64-apple-darwin16 at r241977

2016-11-16 Thread m.ostapenko at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78267 --- Comment #47 from Maxim Ostapenko --- (In reply to r...@cebitec.uni-bielefeld.de from comment #45) > > --- Comment #44 from Maxim Ostapenko --- > [...] > >> Otherwise the definition of SANITIZER_OS_TRACE results in > >>

Re: Add more subreg offset helpers

2016-11-16 Thread Eric Botcazou
> Provide versions of subreg_lowpart_offset and subreg_highpart_offset > that work on mode sizes rather than modes. Also provide a routine > that converts an lsb position to a subreg offset. The position is in bits but all the other parameters and the return value are in bytes so IMO this is a

[Bug fortran/78059] Pointer to variable length array, debug info

2016-11-16 Thread bernhard.heckel at intel dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78059 Bernhard Heckel changed: What|Removed |Added Status|WAITING |RESOLVED Resolution|---

[Bug fortran/24546] [meta-bug] gfortran debugging problems

2016-11-16 Thread bernhard.heckel at intel dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24546 Bug 24546 depends on bug 78059, which changed state. Bug 78059 Summary: Pointer to variable length array, debug info https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78059 What|Removed |Added

<    1   2   3   4   5   6