[Bug target/67351] Missed optimisation on 64-bit field compared to 32-bit

2015-08-25 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67351 Uroš Bizjak ubizjak at gmail dot com changed: What|Removed |Added CC||rguenth at gcc dot

[gomp4.1] comment some stuff

2015-08-25 Thread Aldy Hernandez
I'm obviously not smart enough to understand libgomp's tasking runtime, and rth and you get 0 for commenting skills ;-). I had some notes scribbled down while reading the code, and figured someone else might read this code some day. It's still in dire need of commenting, but this mildly

Re: [gomp4.1] comment some stuff

2015-08-25 Thread Aldy Hernandez
On 08/25/2015 10:35 AM, Aldy Hernandez wrote: -int gomp_max_task_priority_var = 0; +static int gomp_max_task_priority_var = 0; Sorry I snuck that in there. The variable is unused elsewhere, might as well make it static. Aldy

[Bug target/67351] Missed optimisation on 64-bit field compared to 32-bit

2015-08-25 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67351 --- Comment #4 from Andrew Pinski pinskia at gcc dot gnu.org --- (In reply to Uroš Bizjak from comment #3) (In reply to Uroš Bizjak from comment #2) (In reply to Allan Jensen from comment #0) Gcc will expand and detect field setting on

[gomp-4.1] fix incorrect memory size in goacc_new_thread

2015-08-25 Thread Aldy Hernandez
This is either blatantly wrong or subtly correct, in which case it needs a comment. My guess is the former. OK for branch? commit 330391636113ed9a9067e6eb639755fb0f4723dc Author: Aldy Hernandez al...@redhat.com Date: Tue Aug 25 10:41:28 2015 -0700 * oacc-init.c (goacc_new_thread):

Re: Fix libbacktrace -fPIC breakage from Use libbacktrace in libgfortran

2015-08-25 Thread Ulrich Weigand
Hans-Peter Nilsson wrote: From: Ulrich Weigand uweig...@de.ibm.com Date: Tue, 25 Aug 2015 14:59:05 +0200 The other GCC run-time libraries rely on libtool to figure out that even though -fPIC works, dynamic libraries are still not supported on the platform, and thus compile everything

[Bug target/67351] Missed optimisation on 64-bit field compared to 32-bit

2015-08-25 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67351 --- Comment #5 from Andrew Pinski pinskia at gcc dot gnu.org --- Oh his patch only handled multiplies/divide and not shifts. But it should be easy to add them to match.pd to simplify this at the tree level.

Re: [libgfortran,patch] Remove never-used debugging code

2015-08-25 Thread FX
Turns out I missed some of the dead code. And I now also fixed comments and some formatting. libgfortran/runtime/environ.c is now much more readable than before. The patch is still a no-op, in terms of user functionality. OK to commit to trunk? FX unusedcode.ChangeLog Description: Binary

[nvptx] More gcc testsuite markup

2015-08-25 Thread Nathan Sidwell
I've committed this to markup more test requirements. Most are obvious enough. nvptx doesn't expose a normal stack, so stack-based tests fail. It also requires correct typing on function calls, so lying about that results in assembler errors. Finally, it doesn't accept string constants,

[Bug target/67351] Missed optimisation on 64-bit field compared to 32-bit

2015-08-25 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67351 --- Comment #2 from Uroš Bizjak ubizjak at gmail dot com --- (In reply to Allan Jensen from comment #0) Gcc will expand and detect field setting on 32-bit integers, but for some reason miss the opportunity on 64-bit. The immediates for 64bit

[Bug target/67317] [x86] Silly code generation for _addcarry_u32/_addcarry_u64

2015-08-25 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67317 --- Comment #4 from Uroš Bizjak ubizjak at gmail dot com --- (In reply to Segher Boessenkool from comment #3) Does this need to be an unspec at all? Of course not. We are looking to replace unspecs with standard RTXes. Do you have any

Re: [PATCH] rs6000: Fix PR67344

2015-08-25 Thread David Edelsohn
On Tue, Aug 25, 2015 at 1:08 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: The *andmode3_imm_dot_shifted pattern is a define_insn_and_split, like most dot patterns: if its output is not assigned cr0 but some other cr reg, it splits to a non-dot insn and a compare. Unfortunately

[PATCH] Don't ICE on invalid weak decl (PR middle-end/67330)

2015-08-25 Thread Marek Polacek
Here we are ICEing on an invalid code: symtab_node::get asserts that it's dealing with a function or a static or external variable, but an invalid decl is rejected too late. So don't try to mark_weak an invalid decl and also don't duplicate the declared weak after being used check -- that is

Re: [libgfortran,patch] Remove never-used debugging code

2015-08-25 Thread Steve Kargl
On Tue, Aug 25, 2015 at 07:10:23PM +0200, FX wrote: Certainly, the dead code can go. But,is this changing the library ABI? troutmask:fvwm:kargl[764] nm /mnt/sgk/work/6/lib/libgfortran.a | grep show_ T _gfortrani_show_variables t show_boolean

RE: [PATCH] MIPS: If a test in the MIPS testsuite requires standard library support check the sysroot supports the required test options.

2015-08-25 Thread Moore, Catherine
-Original Message- From: Andrew Bennett [mailto:andrew.benn...@imgtec.com] Sent: Tuesday, July 21, 2015 10:15 AM To: gcc-patches@gcc.gnu.org Cc: Matthew Fortune; Moore, Catherine Subject: [PATCH] MIPS: If a test in the MIPS testsuite requires standard library support check the

[gomp4] another routine test

2015-08-25 Thread Nathan Sidwell
I've committed this test to check 2-dimensional loops inside a routine. nathan 2015-08-24 Nathan Sidwell nat...@codesourcery.com * testsuite/libgomp.oacc-c-c++-common/routine-wv-1.c: New. Index: testsuite/libgomp.oacc-c-c++-common/routine-wv-1.c

Expand comment on struct switchstr

2015-08-25 Thread Joseph Myers
This patch, extracted from a larger change on gomp-4_0-branch, expands a comment documenting struct switchstr in gcc.c. Committed. 2015-08-25 Thomas Schwinge tho...@codesourcery.com Joseph Myers jos...@codesourcery.com * gcc.c (struct switchstr): Expand comment. Index:

Re: [AArch64][TLSLE][1/3] Add the option -mtls-size for AArch64

2015-08-25 Thread Jiong Wang
Marcus Shawcroft writes: On 19 August 2015 at 15:26, Jiong Wang jiong.w...@arm.com wrote: 2015-08-19 Jiong Wang jiong.w...@arm.com gcc/ * config/aarch64/aarch64.opt (mtls-size): New entry. * config/aarch64/aarch64.c (initialize_aarch64_tls_size): New function.

Re: Forwarding -foffload=[...] from the driver (compile-time) to libgomp (run-time)

2015-08-25 Thread Joseph Myers
On reviewing in more detail the changes to pass offloading targets from the driver to libgomp at link time to identify the minimal self-contained pieces that can go to trunk, I found that the use of fnmatch to match against target names was completely unnecessary; the ISO C90 functions strstr and

Re: [PATCH, rs6000] Fix vec_shr define_expand

2015-08-25 Thread David Edelsohn
On Tue, Aug 25, 2015 at 10:14 AM, Pat Haugen pthau...@linux.vnet.ibm.com wrote: The following patch fixes the vec_shr expander to do a shift instead of a rotate. CPU2006 benchmark 482.sphinx3 recently started failing due to this issue. Bootstrapped and tested on ppc64/ppc64le with no new

Re: [testsuite] Clean up effective_target cache

2015-08-25 Thread Mike Stump
On Aug 25, 2015, at 1:14 AM, Christophe Lyon christophe.l...@linaro.org wrote: Some subsets of the tests override ALWAYS_CXXFLAGS or TEST_ALWAYS_FLAGS and perform effective_target support tests using these modified flags. This patch adds a new function 'clear_effective_target_cache', which

Re: [PATCH 14/15][ARM/AArch64 Testsuite]Add test of vcvt{,_high}_i{f32_f16,f16_f32}

2015-08-25 Thread Alan Lawrence
Sorry - wrong version posted. The hunk for add_options_for_arm_neon_fp16 has moved to the previous patch! This version also fixes some whitespace issues. gcc/testsuite/ChangeLog: * gcc.target/aarch64/advsimd-intrinsics/vcvt_f16.c: New. * lib/target-supports.exp

[Bug middle-end/67351] Missed optimisation on 64-bit field compared to 32-bit

2015-08-25 Thread linux at carewolf dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67351 --- Comment #1 from Allan Jensen linux at carewolf dot com --- Created attachment 36254 -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=36254action=edit Compiled test assembler

[hsa] Fix omp declare target support

2015-08-25 Thread Martin Jambor
Hi, it was brought to my attention that omp declare target functions were not properly translated to HSA functions. Until the grand shceme with an IPA pass is complete, this will do. And having a single predicate to decide what should be an HSA function cannot be bad. Committed to the hsa

[Bug middle-end/67330] ICE handling weak attributes

2015-08-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67330 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug tree-optimization/66372] [6 Regression] ICE on valid code at -O3 on x86_64-linux-gnu

2015-08-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66372 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

[nvptx] fix thinko

2015-08-25 Thread Nathan Sidwell
I've committed this. A thinko regarding what asm_operands wants for an argument. Fixes a couple of tests. nathan 2015-08-25 Nathan Sidwell nat...@acm.org * config/nvptx/nvptx.c (nvptx_write_function_decl): Reformat. (nvptx_reorg_subreg): Pass insn pattern to asm_operands. Index:

[PATCH, rs6000] Fix vec_shr define_expand

2015-08-25 Thread Pat Haugen
The following patch fixes the vec_shr expander to do a shift instead of a rotate. CPU2006 benchmark 482.sphinx3 recently started failing due to this issue. Bootstrapped and tested on ppc64/ppc64le with no new regressions. Ok for trunk? And ok for 4.9/5 (with equivalent change to vec_shl

Re: [Patch] Add to the libgfortran/newlib bodge to detect ftruncate support in ARM/AArch64/SH

2015-08-25 Thread James Greenhalgh
On Fri, Aug 21, 2015 at 11:05:47AM +0100, James Greenhalgh wrote: On Thu, Aug 20, 2015 at 10:50:47AM +0100, Marcus Shawcroft wrote: On 20 August 2015 at 09:31, James Greenhalgh james.greenha...@arm.com wrote: Hi, Steve's patch in 2013 [1] to fix the MIPS newlib/libgfortran build

Re: [Patch] Add to the libgfortran/newlib bodge to detect ftruncate support in ARM/AArch64/SH

2015-08-25 Thread FX
2015-08-25 James Greenhalgh james.greenha...@arm.com * configure.ac: Auto-detect newlib function support unless we know there are issues when configuring for a host. * configure: Regenerate. Thanks for CC’ing the fortran list. Given that this is newlib-specific code,

[Bug middle-end/67005] [5/6 Regression] ICE: in verify_loop_structure, at cfgloop.c:1647 (loop with header n not in loop tree)

2015-08-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67005 --- Comment #7 from Marek Polacek mpolacek at gcc dot gnu.org --- (In reply to Marek Polacek from comment #6) Hm, adding || (e2-flags EDGE_IRREDUCIBLE_LOOP) doesn't work; the E2 edge doesn't have the EDGE_IRREDUCIBLE_LOOP flag, even if I

Re: [PATCH 1/2] driver: support state cleanup

2015-08-25 Thread Joseph Myers
On Thu, 6 Aug 2015, David Malcolm wrote: gcc/ChangeLog: * gcc-main.c (main): Add params to driver ctor. * gcc.c (class env_manager): New. (env): New global. (env_manager::init): New. (env_manager::get): New. (env_manager::xput): New.

Indirect jumps

2015-08-25 Thread Nathan Sidwell
Ptx is one of those rare (unique?) machines that doesn't have an indirect branch. optabs is prepared for such a target and emits a sorry when an indirect branch is needed. However it then goes on to try and emit such an instruction and ends up ICEing. Fixed thusly, ok? (Or is the right

Re: [PATCH 14/15][ARM/AArch64 Testsuite]Add test of vcvt{,_high}_i{f32_f16,f16_f32}

2015-08-25 Thread Christophe Lyon
On 25 August 2015 at 15:57, Alan Lawrence alan.lawre...@arm.com wrote: Sorry - wrong version posted. The hunk for add_options_for_arm_neon_fp16 has moved to the previous patch! This version also fixes some whitespace issues. This looks OK to me now, thanks. gcc/testsuite/ChangeLog:

PING: PATCH: Mention --enable-default-pie in gcc-6/changes.html

2015-08-25 Thread H.J. Lu
On Thu, May 28, 2015 at 6:49 AM, H.J. Lu hjl.to...@gmail.com wrote: OK to install? H.J. --- Index: gcc-6/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v retrieving revision 1.10 diff -u -p -r1.10

[Bug target/67272] [HSA] register allocator expects that every register must be assigned

2015-08-25 Thread matz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67272 --- Comment #5 from Michael Matz matz at gcc dot gnu.org --- Author: matz Date: Tue Aug 25 16:02:38 2015 New Revision: 227176 URL: https://gcc.gnu.org/viewcvs?rev=227176root=gccview=rev Log: PR target/67272 PR target/67296

[Bug target/67296] [HSA] ICE in register allocator (assignment of this argument in a ctor)

2015-08-25 Thread matz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67296 --- Comment #1 from Michael Matz matz at gcc dot gnu.org --- Author: matz Date: Tue Aug 25 16:02:38 2015 New Revision: 227176 URL: https://gcc.gnu.org/viewcvs?rev=227176root=gccview=rev Log: PR target/67272 PR target/67296

[libgfortran,committed] Fix default SIGN mode on preconnected/internal units

2015-08-25 Thread FX
Preconnected and internal units currently have their sign mode set to SIGN_SUPPRESS, rather than the logical value of SIGN_UNSPECIFIED. This does not matter in most cases, since our chosen processor-dependent behavior is to suppress optional plus signs anyway… … except when one tries to

Re: [libgfortran,patch] Remove never-used debugging code

2015-08-25 Thread Steve Kargl
On Tue, Aug 25, 2015 at 06:17:13PM +0200, FX wrote: Turns out I missed some of the dead code. And I now also fixed comments and some formatting. libgfortran/runtime/environ.c is now much more readable than before. The patch is still a no-op, in terms of user functionality. OK to commit to

Re: [libgfortran,patch] Remove never-used debugging code

2015-08-25 Thread FX
Certainly, the dead code can go. But,is this changing the library ABI? troutmask:fvwm:kargl[764] nm /mnt/sgk/work/6/lib/libgfortran.a | grep show_ T _gfortrani_show_variables t show_boolean t show_integer t show_sep

[gomp4] optimize routine calling

2015-08-25 Thread Nathan Sidwell
When forking to call a partitioned routine there is no need to propagate local state from the active thread to the forked threads. I've committed this patch to implement that optimization. nathan 2015-08-25 Nathan Sidwell nat...@codesourcery.com * config/nvptx/nvptx.c (nvptx_emit_forking):

[Bug middle-end/67351] New: Missed optimisation on 64-bit field compared to 32-bit

2015-08-25 Thread linux at carewolf dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67351 Bug ID: 67351 Summary: Missed optimisation on 64-bit field compared to 32-bit Product: gcc Version: 5.2.1 Status: UNCONFIRMED Severity: enhancement Priority: P3

[Bug target/67296] [HSA] ICE in register allocator (assignment of this argument in a ctor)

2015-08-25 Thread matz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67296 Michael Matz matz at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

Re: Fix libbacktrace -fPIC breakage from Use libbacktrace in libgfortran

2015-08-25 Thread Hans-Peter Nilsson
TL;DR: See last... From: Ulrich Weigand uweig...@de.ibm.com Date: Tue, 25 Aug 2015 14:59:05 +0200 However, the compiler actually does accept -fPIC. If the flag is present, we attempt to generate relocatable code, but only to the extent the compiler can do that without support for run-time

[PATCH] Update wwwdocs for --with-advance-toolchain=at

2015-08-25 Thread Michael Meissner
I installed the following patch on wwwdocs to document the --with-advance-toolchain=at option I added in June: 2015-08-25 Michael Meissner meiss...@linux.vnet.ibm.com * changes.html (PowerPC options): Document new configure option --with-advance-toolchain=at. Index:

[PATCH] rs6000: Fix PR67346

2015-08-25 Thread Segher Boessenkool
*iormode_mask is a define_insn_and_split, so it shouldn't use can_create_pseudo in its instruction condition, because IRA can then create such an instruction, and the condition becomes false before the insn is split. Use a scratch instead. Bootstrapped and tested on powerpc64-linux; okay for

[Bug middle-end/64544] ../../gcc-svn/gcc/cgraphunit.c:2183:1: internal compiler error: in check_probability, at basic-block.h:581

2015-08-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64544 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[PATCH] rs6000: Fix PR67344

2015-08-25 Thread Segher Boessenkool
The *andmode3_imm_dot_shifted pattern is a define_insn_and_split, like most dot patterns: if its output is not assigned cr0 but some other cr reg, it splits to a non-dot insn and a compare. Unfortunately that non-dot insn will clobber cr0 as well. We could add another clobber (with =X,x), but

Re: [PATCH 14/15][ARM/AArch64 Testsuite]Add test of vcvt{,_high}_{f16_f32,f32_f16}

2015-08-25 Thread Alan Lawrence
Christophe Lyon wrote: On 28 July 2015 at 13:27, Alan Lawrence alan.lawre...@arm.com wrote: gcc/testsuite/ChangeLog: * gcc.target/aarch64/advsimd-intrinsics/advsimd-intrinsics.exp: set additional flags for neon-fp16 support. *

RE: [PATCH ppc64,aarch64,alpha 00/15] Improve backend constant generation

2015-08-25 Thread Wilco Dijkstra
Richard Henderson wrote: On 08/12/2015 08:59 AM, Wilco Dijkstra wrote: I looked at the statistics of AArch64 immediate generation a while ago. The interesting thing is ~95% of calls are queries, and the same query is on average repeated 10 times in a row. So (a) it is not important to

Re: [PATCH] rs6000: Fix PR67346

2015-08-25 Thread David Edelsohn
On Tue, Aug 25, 2015 at 1:09 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: *iormode_mask is a define_insn_and_split, so it shouldn't use can_create_pseudo in its instruction condition, because IRA can then create such an instruction, and the condition becomes false before the insn

Re: Moving to git

2015-08-25 Thread Eric S. Raymond
Jason Merrill ja...@redhat.com: git-svn find-rev takes r123456 and returns a commit hash based on the git-svn-id in the git log; I don't see why we would need to break that moving forward, though I'm not sure how well it would work without reference to an actual SVN server. It won't work at

Re: [PATCH 3/5] Build ARRAY_REFs when the base is of ARRAY_TYPE.

2015-08-25 Thread Bin.Cheng
On Wed, Aug 26, 2015 at 3:50 AM, Jeff Law l...@redhat.com wrote: On 08/25/2015 05:06 AM, Alan Lawrence wrote: When SRA completely scalarizes an array, this patch changes the generated accesses from e.g. MEM[(int[8] *)a + 4B] = 1; to a[1] = 1; This overcomes a limitation in dom2, that

Re: Moving to git

2015-08-25 Thread Jason Merrill
On 08/24/2015 11:49 AM, Jeff Law wrote: On 08/24/2015 09:43 AM, Jakub Jelinek wrote: Not to mention we should keep the existing r123456 comments in bugzilla working, and I'm not convinced keeping a SVN version of the repository (frozen) for that purpose is the best idea. I'd like to keep the

Re: Moving to git

2015-08-25 Thread Jason Merrill
On 08/24/2015 11:54 AM, Richard Earnshaw wrote: Why not use the output of 'git show -s --format=%ct-%h'? $ git show -s --format=%ct-%h master 1440153969-f57da59 That gives you a unix timestamp for the commit, followed by the hash. Now you've got a fully ordered way of referring to the commit,

[Bug tree-optimization/67349] [5 regression] ICE on optimization

2015-08-25 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67349 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added Target|h8300 |

[Bug tree-optimization/67055] [5/6 Regression] Segmentation fault in fold_builtin_alloca_with_align in tree-ssa-ccp.c

2015-08-25 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67055 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added CC||ysato

[Bug tree-optimization/67349] [5 regression] ICE on optimization

2015-08-25 Thread ysato at users dot sourceforge.jp
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67349 --- Comment #3 from Yoshinori Sato ysato at users dot sourceforge.jp --- I tested gcc version 6.0.0 20150710 (experimental) (GCC) I'll trying latest trunk. Thanks,

[Bug inline-asm/67317] [x86] Silly code generation for _addcarry_u32/_addcarry_u64

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67317 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |WAITING

[Bug middle-end/67340] [6 Regression] ICE: in convert_move, at expr.c:279

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67340 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target||hppa*-*-*

[PATCH, ARM] List Cs and US constraints as being used

2015-08-25 Thread Thomas Preud'homme
Hi, The header in gcc/config/arm/constraints.md list all the ARM-specific constraints defined and for which targets they are but miss a couple of them. This patch add the missing Cs and US constraints to the list. Patch was tested by verifying that arm-none-eabi-gcc cross-compiler can still

[Bug bootstrap/66038] [5 regression] (stage 2) build/genmatch issue (gcc/hash-table.h|c) with --disable-checking [ introduced by r218976 ]

2015-08-25 Thread kumba at gentoo dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66038 --- Comment #27 from Joshua Kinard kumba at gentoo dot org --- (In reply to Richard Biener from comment #26) Don't hold your breath. Basically somebody who can reproduce it has to find the root-cause and a fix. 4.9.3 works, and the problem

[Bug middle-end/67005] [5/6 Regression] ICE: in verify_loop_structure, at cfgloop.c:1647 (loop with header n not in loop tree)

2015-08-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67005 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added CC||mpolacek at

[Bug tree-optimization/67349] [5 regression] ICE on optimization

2015-08-25 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67349 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added Status|NEW

[Bug tree-optimization/67055] [5 Regression] Segmentation fault in fold_builtin_alloca_with_align in tree-ssa-ccp.c

2015-08-25 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67055 Markus Trippelsdorf trippels at gcc dot gnu.org changed: What|Removed |Added Summary|[5/6 Regression]|[5

[Bug bootstrap/66038] [5 regression] (stage 2) build/genmatch issue (gcc/hash-table.h|c) with --disable-checking [ introduced by r218976 ]

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66038 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target|powerpc-darwin

[testsuite] Clean up effective_target cache

2015-08-25 Thread Christophe Lyon
Hi, Some subsets of the tests override ALWAYS_CXXFLAGS or TEST_ALWAYS_FLAGS and perform effective_target support tests using these modified flags. In case these flags conflict with the effective_target tests, it means that subsequent tests will be UNSUPPORTED even though

[Bug c++/67313] [6 Regression] ICE: in vague_linkage_p, at cp/decl2.c:1878 with -fno-weak and variadic template

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67313 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |6.0

[Bug c++/67315] [4.9 Regression] Strange 'this' pointer behavior when calling virtual function with different optimization attributes.

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67315 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |4.9.4

[Bug c++/67315] [4.9 Regression] Strange 'this' pointer behavior when calling virtual function with different optimization attributes.

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67315 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added CC||hubicka at

[Bug tree-optimization/67326] [5/6 Regression] -ftree-loop-if-convert-stores does not vectorize conditional assignment (anymore)

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67326 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW

[Bug c++/67315] [4.9 Regression] Strange 'this' pointer behavior when calling virtual function with different optimization attributes.

2015-08-25 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67315 --- Comment #3 from Jonathan Wakely redi at gcc dot gnu.org --- PR 54068 ?

Re: [AArch64] [TLSIE][1/2] Rename test source file for reuse

2015-08-25 Thread Marcus Shawcroft
On 19 June 2015 at 10:15, Jiong Wang jiong.w...@arm.com wrote: Rename test source from tlsle.c into tls.c for reuse purpose. tls.c will be used as test source file for all TLS test, we just need to specify different tls options in different testcases. 2015-06-19 Jiong Wang

[Bug tree-optimization/67312] [6 Regression] ICE: SIGSEGV in expand_expr_real_1 (expr.c:9561) with -ftree-coalesce-vars

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67312 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |6.0 ---

[Bug middle-end/67118] gcc and gfortran started crashing recently

2015-08-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67118 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |WAITING

[Bug middle-end/67298] [6 Regression] 254.gap in SPEC CPU 2000 is miscompiled

2015-08-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67298 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |WAITING

[Bug middle-end/66984] ICE: fold_binary changes type of operand, causing failure in verify_gimple_assign_binary

2015-08-25 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66984 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

[Bug target/67349] ICE on optimization

2015-08-25 Thread miyuki at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67349 Mikhail Maltsev miyuki at gcc dot gnu.org changed: What|Removed |Added CC||miyuki at gcc

[Bug middle-end/67341] [ICE] libgo build failure: in mark_stmt_if_obviously_necessary, at tree-ssa-dce.c:278

2015-08-25 Thread miyuki at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67341 Mikhail Maltsev miyuki at gcc dot gnu.org changed: What|Removed |Added CC||miyuki at gcc

[Bug tree-optimization/37021] Fortran Complex reduction / multiplication not vectorized

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=37021 --- Comment #21 from Richard Biener rguenth at gcc dot gnu.org --- (In reply to Bill Schmidt from comment #20) We still don't vectorize the original code example on Power. It appears that this is being disabled because of an alignment issue.

[Bug tree-optimization/67328] range test rather than single bit test for code testing enum values

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67328 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |WAITING

[Bug c++/67318] [6 regression] Parsing error when using abbreviated integral type names in template parameter pack declaration

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67318 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |6.0

[Bug debug/67293] Very large DW_AT_const_value produced

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67293 --- Comment #3 from Richard Biener rguenth at gcc dot gnu.org --- (In reply to Jakub Jelinek from comment #2) I'm fine with introducing some limit on the size of const values, with a param. As for the other question, I think you've answered

[Bug target/67349] New: ICE on optimization

2015-08-25 Thread ysato at users dot sourceforge.jp
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67349 Bug ID: 67349 Summary: ICE on optimization Product: gcc Version: 6.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: target Assignee:

Re: Moving to git

2015-08-25 Thread Andreas Schwab
Jakub Jelinek ja...@redhat.com writes: On Mon, Aug 24, 2015 at 10:22:22AM +0200, Andreas Schwab wrote: Jakub Jelinek ja...@redhat.com writes: And for those really identifying them by sha1 hashes is significantly worse than using monotonically increasing small number, sha1 hashes are

Re: [PATCH 1/5] Refactor completely_scalarize_var

2015-08-25 Thread Jeff Law
On 08/25/2015 05:06 AM, Alan Lawrence wrote: This is a small refactoring/renaming patch, it just moves the call to completely_scalarize_record out from completely_scalarize_var, and renames the latter to create_total_scalarization_access. This is because the next patch needs to drop the _record

Re: Indirect jumps

2015-08-25 Thread Jeff Law
On 08/25/2015 08:11 AM, Nathan Sidwell wrote: Ptx is one of those rare (unique?) machines that doesn't have an indirect branch. optabs is prepared for such a target and emits a sorry when an indirect branch is needed. However it then goes on to try and emit such an instruction and ends up

[gomp4] add reduction lock initializer

2015-08-25 Thread Nathan Sidwell
Cesar discovered another quirk of PTX. Inspite of PTX documenting that static variables can be initialized and default to zero, there's a little note that it doesn't work for .shared variables. Thus we need code to initialize the worker lock variable used for reductions. This implements a

[Bug c++/67350] New: auto deduction error in variable template lambda

2015-08-25 Thread norbert.pfeiler+gcc.gnu.org/bugzilla at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67350 Bug ID: 67350 Summary: auto deduction error in variable template lambda Product: gcc Version: 5.2.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Re: [AArch64] [TLSIE][2/2] Implement TLS IE for tiny model

2015-08-25 Thread Marcus Shawcroft
On 19 June 2015 at 10:15, Jiong Wang jiong.w...@arm.com wrote: Currently, TLS IE is supported on small model only. This patch implement TLS Initial-exec model support for AArch64 tiny memory model. Under tiny model, we only allow 1M loadable segment size, one single ldr instruction is enough

[PATCH, PR other/67320] Fix wide add standard names

2015-08-25 Thread Michael Collison
The standard names for signed and unsigned vector wide adds are wrong in the documentation. OK for trunk? 2015-08-25 Michael Collison michael.colli...@linaro.org PR other/67320 * doc/md.texi: Rename [su]sum_widen to widen_[su]sum to reflect correct standard names diff --git

Re: [PATCH][AARCH64]Fix for branch offsets over 1 MiB

2015-08-25 Thread Andrew Pinski
On Tue, Aug 25, 2015 at 5:50 PM, Andrew Pinski pins...@gmail.com wrote: On Tue, Aug 25, 2015 at 5:37 PM, Andre Vieira andre.simoesdiasvie...@arm.com wrote: Conditional branches have a maximum range of [-1048576, 1048572]. Any destination further away can not be reached by these. To be able to

[PATCH] Fix PR67306

2015-08-25 Thread Richard Biener
The following fixes ICEs due to the genmatch generated code for GENERIC not verifying if builtin_decl_implicit returns non-NULL. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-08-25 Richard Biener rguent...@suse.de PR middle-end/67306 *

[Bug middle-end/67005] [5/6 Regression] ICE: in verify_loop_structure, at cfgloop.c:1647 (loop with header n not in loop tree)

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67005 --- Comment #4 from Richard Biener rguenth at gcc dot gnu.org --- Note we already do /* If we made a BB unconditionally exit a loop then this transform alters the set of BBs in the loop. Schedule a

[Bug tree-optimization/67323] Use non-unit stride loads by preference when applicable

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67323 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED

Re: [PATCH][AARCH64]Fix for branch offsets over 1 MiB

2015-08-25 Thread Andrew Pinski
On Tue, Aug 25, 2015 at 5:37 PM, Andre Vieira andre.simoesdiasvie...@arm.com wrote: Conditional branches have a maximum range of [-1048576, 1048572]. Any destination further away can not be reached by these. To be able to have conditional branches in very large functions, we invert the

[Bug c++/67345] -Woverloaded-virtual false negative: Does not warn on overloaded virtual function

2015-08-25 Thread EisahLee at gmx dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67345 --- Comment #2 from EisahLee at gmx dot de --- I see the hiding as a potential design error, or however that is called: A shortcoming of the way the methods were named. Clang 4.5 does not warn until there is such a bad call. Is there a compiler

[Bug tree-optimization/67306] Patterns ICEs when moved using simplify and match

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67306 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug tree-optimization/67306] Patterns ICEs when moved using simplify and match

2015-08-25 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67306 --- Comment #5 from Richard Biener rguenth at gcc dot gnu.org --- Author: rguenth Date: Tue Aug 25 10:29:09 2015 New Revision: 227163 URL: https://gcc.gnu.org/viewcvs?rev=227163root=gccview=rev Log: 2015-08-25 Richard Biener rguent...@suse.de

Re: [PATCH 12/15][AArch64] Add vcvt(_high)?_f32_f16 intrinsics, with BE RTL fix

2015-08-25 Thread Alan Lawrence
James Greenhalgh wrote: - VAR1 (UNOP, vec_unpacks_hi_, 10, v4sf) + VAR2 (UNOP, vec_unpacks_hi_, 10, v4sf, v8hf) Should this not use the appropriate BUILTIN_... iterator? Indeed; BUILTIN_VQ_HSF it is. VAR1 (BINOP, float_truncate_hi_, 0, v4sf) VAR1 (BINOP, float_truncate_hi_, 0,

  1   2   >