Re: C++ PATCH for c++/68586 (rejects-valid with enum in C++11)

2016-01-18 Thread Marek Polacek
On Mon, Jan 18, 2016 at 01:06:46PM -0500, Jason Merrill wrote: > Yes, and let's also clear the fold_cache at the same time. Ah, that one as well. Thus: Bootstrapped/regtested on x86_64-linux, ok for trunk? 2016-01-18 Marek Polacek PR c++/68586 * constexpr.c (clear_cv_cache):

Re: [SMS] Schedule normalization after scheduling branch

2016-01-18 Thread Roman Zhuykov
Hello, 4 years ago when I create some SMS patches nobody cares about SMS failures on ia64. Now ia64 is even more dead but at least one of bugs appears on powerpc - PR69252. Proposed patch is here: https://gcc.gnu.org/ml/gcc-patches/2011-12/txt00266.txt and it even suits current trunk without modi

Re: [patch, fortran] Inline MATMUL(A,TRANSPOSE(B)), PR 66094

2016-01-18 Thread Thomas Koenig
Hi Toon, It will also perform the following tests (minus the "inline_matmul_13.f90" one, which wasn't included in the attachements :-) Well, here it is. Regards Thomas ! { dg-do run } ! { dg-options "-ffrontend-optimize -fdump-tree-original -Wrealloc-lhs" } ! PR 66094: Check functio

Re: [patch, fortran] Inline MATMUL(A,TRANSPOSE(B)), PR 66094

2016-01-18 Thread Toon Moene
On 01/18/2016 11:14 PM, Thomas Koenig wrote: Hi Toon, It will also perform the following tests (minus the "inline_matmul_13.f90" one, which wasn't included in the attachements :-) Well, here it is. Included, thanks, -- Toon Moene - e-mail: t...@moene.org - phone: +31 346 214290 Saturnusho

Re: genattrab.c generate switch

2016-01-18 Thread Jesper Broge Jørgensen
On 18/01/16 18:39, Manuel López-Ibáñez wrote: On 18/01/16 14:39, Jesper Broge Jørgensen wrote: No i have not gone through copyright assignment. This is my first time trying to contribute to a GNU project so i have tried following the "Contributing to GCC"@ https://gcc.gnu.org/contribute.html

Re: [PATCH v2] sanitize paths used in regular expression

2016-01-18 Thread Zach Welch
On 01/15/2016 10:48 AM, Mike Stump wrote: > On Jan 15, 2016, at 10:40 AM, Zachary T Welch wrote: >> Does this version look better? > > Ok. > >> I am not sure if this the right place to put the new helper, so let me know >> if there is a better spot for it. > > So, someone that wants to rehome

[PATCH, testsuite]: Disable LTO for gcc.c-torture/execute/builtins/{memops,strstr}-asm.c

2016-01-18 Thread Uros Bizjak
Hello! As explained by Honza in the PR, these two tests are not suitable for LTO tests. Also, the dg-options directives are not effective in this directory. 2016-01-19 Uros Bizjak PR testsuite/68820 * gcc.c-torture/execute/builtins/memops-asm.x: New file. * gcc.c-torture/execute/b

[PATCH] Fix ICE with asm "m" (stmt-expr) operand (PR middle-end/67653)

2016-01-18 Thread Jakub Jelinek
Hi! Here is an attempt to fix ICE on statement expression in "m" asm input operand. The problem is that gimplify_asm_expr attempts to mark it addressable, but that can be just too late, a temporary the stmt-expression gimplifies to might not be addressable and may be used already in the gimplifie

[PATCH] Fix debug info handling in prepare_shrink_wrap (PR debug/65779)

2016-01-18 Thread Jakub Jelinek
Hi! On the following testcase with -mrelocatable on ppc32 we get assembly that contains undefined reference to a local .LC* symbol. The problem is that prepare_shrink_wrap attempts to schedule some instructions from the entry block to later basic blocks, if they set a register that is only used in

[PATCH] Fix RTL DSE (PR rtl-optimization/68955, take 2)

2016-01-18 Thread Jakub Jelinek
On Mon, Jan 18, 2016 at 11:40:23AM +0100, Eric Botcazou wrote: > > So, do you suggest to tweak get_addr like the patch below, and remove the > > mem_addr = get_addr (mem_addr); > > line above and the comment? > > Yes, exactly. And if that doesn't easily work, then go for your solution and > ad

Re: [PATCH 1/2] fix memory chunk corruption for opts_obstack (PR jit/68446)

2016-01-18 Thread Jakub Jelinek
On Fri, Jan 15, 2016 at 03:04:33PM -0500, David Malcolm wrote: > OK for trunk? (assuming it bootstraps®rtests by itself) > > gcc/ChangeLog: > PR jit/68446 > * gcc.c (driver::decode_argv): Add call to > init_opts_obstack before init_options_struct. > * opts.c (init_opts_obs

Re: reject decl with incomplete struct/union type in check_global_declaration()

2016-01-18 Thread Joseph Myers
On Sat, 16 Jan 2016, Prathamesh Kulkarni wrote: > > There's a GNU C extension allowing forward declarations of enums, and it > > seems that > > > > static enum e x; > > > > doesn't get diagnosed either with -fsyntax-only. Thus I think you should > > cover that case as well. > Done in the attached

-z bndplt documentation in GCC manual

2016-01-18 Thread Sandra Loosemore
I think the documentation relating to '-z bndplt' in the GCC manual description of -fcheck-pointer-bounds is incorrect. It looks like, as of r225862, the GCC driver is supposed to emit an error message if GCC was configured with a linker that doesn't support this option and you pass -mmpx with

[PATCH] fix #69251 - [6 Regression] ICE in unify_array_domain on a flexible array member

2016-01-18 Thread Martin Sebor
The attached is a minimal patch to avoid the ICE. The patch doesn't fix the type substitution of flexible array members as that seems more involved and is, strictly speaking, outside the scope of this bug. Type substitution of flexible array is wrong in 5.3.0 (which treats flexible array members

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-18 Thread H.J. Lu
On Thu, Dec 24, 2015 at 3:55 AM, Alan Lawrence wrote: > This version changes the test cases to fix failures on some platforms, by > rewriting the initializers so that they aren't pushed out to the constant > pool. > > gcc/ChangeLog: > > * tree-ssa-scopedtables.c (avail_expr_hash): Hash ME

[PR66726] Fixe regression caused by Factor conversion out of COND_EXPR

2016-01-18 Thread Kugan
Hi, This is an updated version of https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02196.html. Patch to fix PR66726 missed optimization, factor conversion out of COND_EXPR caused a regression for targets with branch cost greater than i.e., testcase gcc.dg/pr46309.c failed for these targets. I posted

[PATCH] Fix PR c++/69283 (auto deduction fails when ADL is required)

2016-01-18 Thread Patrick Palka
mark_used is wrongly diagnosing a use of a TEMPLATE_DECL (e.g. the call to f1 in function f3 of auto-fn29.C below) for having an undeduced 'auto' return type. This doesn't make sense, because an 'auto' used inside a template doesn't get deduced until after the template is instantiated. So for a T

[PATCH] Fix for PR67896 (C++ FE cannot distinguish __Poly{8,16,64,128}_t types)

2016-01-18 Thread Roger Ferrer Ibáñez
Hi, aarch64-builtins.c defines several SIMD builtin types. Among these SIMD types there are the polynomials __Poly{8,16,64,128}_t. These are built by a call to build_distinct_type_copy (unsigned_int{Q,H,D,T}I_type_node), respectively, i.e. they are not VECTOR_TYPEs. A later loop, traverses an arra

Re: [PATCH 1/3, libgomp] Resolve libgomp plugin deadlock on exit, libgomp proper parts

2016-01-18 Thread Chung-Lin Tang
Ping x 2. On 2016/1/5 11:22 PM, Chung-Lin Tang wrote: > Patch has been updated to accommodate the gomp_fini_device() removal changes. > And ping. > > On 2015/12/14 11:47 PM, Chung-Lin Tang wrote: >> [sorry, forgot to C gcc-patches in last send] >> >> Hi Jakub, >> these patches are a revision of

Re: [PATCH] OpenACC use_device clause ICE fix

2016-01-18 Thread Chung-Lin Tang
Ping. On 2016/1/5 10:15 PM, Chung-Lin Tang wrote: > Hi, > we've been encountering an ICE for OpenACC host_data sections, which > has a use_device() clause similar to OpenMP use_device_ptr. > > The ICE happens in make_decl_rtl() for scan-created variables, which IIUC, > should not be entered at al

[PATCH] [tree-optimization/69320] Optimize tests of boolean ranged objects against constants not 0 or 1

2016-01-18 Thread Jeff Law
The problem with 69320 (and its various duplicates) is via various paths we can end up with tests of a boolean ranged object against a constant outside a boolean range. ie if (x != 23) Where x is a boolean ranged object. That in turn ran afoul of code in record_edge_info which assumed that

<    1   2