Re: [PATCH] Fix up passing long long in ubsan with -m32 (PR sanitizer/59333)

2013-12-05 Thread Jakub Jelinek
On Thu, Dec 05, 2013 at 06:14:03PM +0100, Marek Polacek wrote: 2013-12-05 Marek Polacek pola...@redhat.com PR sanitizer/59333 PR sanitizer/59397 * ubsan.c: Include rtl.h and expr.h. (ubsan_encode_value): Add new parameter. If expanding, assign a stack slot

Re: [SPARC] Fix PR target/59316

2013-12-05 Thread Joseph S. Myers
On Thu, 5 Dec 2013, Eric Botcazou wrote: + /* We generate the equivalent of feclearexcept (FE_ALL_EXCEPT): + + unsigned int tmp2_var; + __builtin_store_fsr (tmp2_var); + + tmp2_var = ~accrued_exception_mask; + + __builtin_load_fsr (tmp2_var); */ Note that you

RFA: patch to fix 2 testsuite failures for LRA on PPC

2013-12-05 Thread Vladimir Makarov
The following patch fixes two GCC testsuite failures for LRA. The patch makes swap through registers instead of memory for the test cases when LRA is used. There are differences in reload and LRA constraint matching algorithm which results in different alternative choices when the original

Re: [PATCH] Fix up passing long long in ubsan with -m32 (PR sanitizer/59333)

2013-12-05 Thread Marek Polacek
On Thu, Dec 05, 2013 at 06:22:02PM +0100, Jakub Jelinek wrote: As a follow-up, please see if all the dg-skip-if -flto ubsan markings can be removed now. Unfortunately, not yet. With -flto, we fail with cclAoIBG.o:(.text+0x14): undefined reference to `.Lubsan_data0.2616'^M collect2: error: ld

Re: [buildrobot] Re: [PATCH] Split -fisolate-erroneous-paths into two options

2013-12-05 Thread Ian Lance Taylor
On Thu, Dec 5, 2013 at 9:03 AM, Jeff Law l...@redhat.com wrote: Q. Now that we're in stage3, presumably GO imports onto the trunk are stopping? If so would it make sense to enable GO by default? The Go library will no longer be updated except for bug fixes. If there is a Go 1.2.1 release,

[C++11] DR1479 - Literal operators and default arguments

2013-12-05 Thread Ed Smith-Rowland
This patch rejects literal operators with defaulted arguments with an extra note to that effect. Not a big deal but it responds to a malformed program statement in the draft. Builds and tests clean on x86_64-linux. OK? Ed CL_udlit_nodefault Description: Binary data

[PATCH] Use libbacktrace for libsanitizer symbolization (take 2, PR sanitizer/59136)

2013-12-05 Thread Jakub Jelinek
Hi! This is a second attempt at libsanitizer symbolization using libbacktrace. The compiler-rt maintained bit have been already added by the recent merge from compiler-rt, so this patch is mostly configury/Makefile stuff. Rather than using libbacktrace.la built in libbacktrace directory

Re: [PING]: [GOMP4] [PATCH] SIMD-Enabled Functions (formerly Elemental functions) for C

2013-12-05 Thread Aldy Hernandez
On 11/30/13 20:38, Iyer, Balaji V wrote: Hello Aldy, Some of the middle end changes I made in the previous patch was not flying for the C++. Here is a fixed patch where the middle-end changes will work for both C and C++. With this email, I am attaching the patch for C along

Re: [PATCH] Add -mtune=ia support

2013-12-05 Thread Uros Bizjak
On Thu, Dec 5, 2013 at 7:22 PM, H.J. Lu hongjiu...@intel.com wrote: We'd like to add a new -mtune=ia option for x86 to optimize for both Haswell and Silvermont. Currently, -mtune=ia is aliased to -mtune=slm. We will improve it further for Haswell and Silvermont. Later, we will update it to

Re: [PATCH] Fixinclude linux/vt.h problem breaking libsanitizer

2013-12-05 Thread Jeff Law
On 12/05/13 03:18, Richard Biener wrote: This fixes the issue in linux/vt.h that appears in SUSE SLE11 kernel headers which contain a pre-release variant that is broken and not compatible with C++ (using the 'new' keyword). The following fix simply replaces that (and only that) field with

Re: [PATCH] Add -mtune=ia support

2013-12-05 Thread H.J. Lu
On Thu, Dec 5, 2013 at 12:38 PM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Dec 5, 2013 at 7:22 PM, H.J. Lu hongjiu...@intel.com wrote: We'd like to add a new -mtune=ia option for x86 to optimize for both Haswell and Silvermont. Currently, -mtune=ia is aliased to -mtune=slm. We will

Re: [PATCH, doc] Document -fsanitize=signed-integer-overflow

2013-12-05 Thread Jeff Law
On 12/05/13 02:50, Marek Polacek wrote: Ping. The implementation has been commited. On Thu, Nov 28, 2013 at 01:32:24PM +0100, Marek Polacek wrote: As promised, this patch on top of this patch by Tobias: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03082.html adds the documentation for

Re: [SPARC] Fix PR target/59316

2013-12-05 Thread Eric Botcazou
Note that you could probably just reload the tmp1_var value rather than needing to store the fsr value again to clear exceptions from it, unless there's an architecture-specific reason the tmp1_var value might no longer be the right value to load to clear exceptions. (This is what x86 does

Re: _Cilk_spawn and _Cilk_sync for C++

2013-12-05 Thread Jason Merrill
On 12/04/2013 02:45 PM, Jason Merrill wrote: + error_at (input_location, _Cilk_sync cannot be used without enabling + Cilk Plus); + cp_lexer_consume_token (parser-lexer); + if (parser-in_statement IN_CILK_SPAWN) + parser-in_statement = parser-in_statement

Re: [PATCH] Add -mtune=ia support

2013-12-05 Thread Patrick Marlier
Hi, On 12/05/2013 07:22 PM, H.J. Lu wrote: We'd like to add a new -mtune=ia option for x86 to optimize for both Haswell and Silvermont. Currently, -mtune=ia is aliased to -mtune=slm. We will improve it further for Haswell and Silvermont. Later, we will update it to future Intel processors.

Re: [PATCH] Add -mtune=ia support

2013-12-05 Thread H.J. Lu
On Thu, Dec 5, 2013 at 1:02 PM, Patrick Marlier patrick.marl...@gmail.com wrote: Hi, On 12/05/2013 07:22 PM, H.J. Lu wrote: We'd like to add a new -mtune=ia option for x86 to optimize for both Haswell and Silvermont. Currently, -mtune=ia is aliased to -mtune=slm. We will improve it

[buildrobot] score: Silence warnings to fix config-list.mk build

2013-12-05 Thread Jan-Benedict Glaw
Hi! This patch silences warnings (unused static functions, ambiguous `else') for the score target. 2013-12-05 Jan-Benedict Glaw jbg...@lug-owl.de * config/score/score.c (score_force_temporary): Delete function. (score_split_symbol): Ditto. * config/score/score.h

Re: [buildrobot] score: Silence warnings to fix config-list.mk build

2013-12-05 Thread Jeff Law
On 12/05/13 14:17, Jan-Benedict Glaw wrote: Hi! This patch silences warnings (unused static functions, ambiguous `else') for the score target. 2013-12-05 Jan-Benedict Glaw jbg...@lug-owl.de * config/score/score.c (score_force_temporary): Delete function.

RE: _Cilk_spawn and _Cilk_sync for C++

2013-12-05 Thread Iyer, Balaji V
-Original Message- From: Jason Merrill [mailto:ja...@redhat.com] Sent: Thursday, December 5, 2013 4:00 PM To: Iyer, Balaji V; gcc-patches@gcc.gnu.org Cc: Jeff Law Subject: Re: _Cilk_spawn and _Cilk_sync for C++ On 12/04/2013 02:45 PM, Jason Merrill wrote: + error_at

Re: [PATCH, libgfortran]: Fix PR59313, gfortran.dg/erf_3.F90 FAILs

2013-12-05 Thread Rainer Orth
Uros Bizjak ubiz...@gmail.com writes: Currently, gfortran.dg/erf_3.F90 FAILs on targets with 128bit (quadruple) long double, since high-precision erfc_scaled_r16 gets defined only for __float128 quadruple precision. Attached patch defines this function also for 128bit long double targets.

Re: [PATCH] Time profiler - phase 2

2013-12-05 Thread Jan Hubicka
Hello, thank you for the trick in ipa-split.c. It really helped! I Good!, this patch is pre-approved after testing. prepared 2 tests for Inkscape, first was just with my function reordering pass. And for the second, I enable also -freorder-blocks-and-partition (note: files ending with

Re: [buildrobot] score: Silence warnings to fix config-list.mk build

2013-12-05 Thread Jan-Benedict Glaw
On Thu, 2013-12-05 14:22:46 -0700, Jeff Law l...@redhat.com wrote: On 12/05/13 14:17, Jan-Benedict Glaw wrote: diff --git a/gcc/config/score/score.h b/gcc/config/score/score.h index 5ab7875..e3bb7f0 100644 --- a/gcc/config/score/score.h +++ b/gcc/config/score/score.h @@ -757,11 +757,12 @@

Re: [PATCH, libgfortran]: Fix PR59313, gfortran.dg/erf_3.F90 FAILs

2013-12-05 Thread Tobias Burnus
Rainer Orth wrote: Unfortunately, this breaks Solaris/SPARC bootstrap: /vol/gcc/src/hg/trunk/local/libgfortran/intrinsics/erfc_scaled.c:64:22: error: 'M_2_SQRTPIl' undeclared (first use in this function) # define _M_2_SQRTPI M_2_SQRTPIl It seems M_2_SQRTPI[lq] are GNU extensions, thus

Re: [PATCH, libgfortran]: Fix PR59313, gfortran.dg/erf_3.F90 FAILs

2013-12-05 Thread Rainer Orth
Tobias Burnus bur...@net-b.de writes: Rainer Orth wrote: Unfortunately, this breaks Solaris/SPARC bootstrap: /vol/gcc/src/hg/trunk/local/libgfortran/intrinsics/erfc_scaled.c:64:22: error: 'M_2_SQRTPIl' undeclared (first use in this function) # define _M_2_SQRTPI M_2_SQRTPIl It seems

Re: [SPARC] Fix PR target/59316

2013-12-05 Thread Joseph S. Myers
On Thu, 5 Dec 2013, Eric Botcazou wrote: Note that you could probably just reload the tmp1_var value rather than needing to store the fsr value again to clear exceptions from it, unless there's an architecture-specific reason the tmp1_var value might no longer be the right value to load

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

2013-12-05 Thread Jakub Jelinek
On Wed, Dec 04, 2013 at 08:14:43AM -0800, H.J. Lu wrote: --- /dev/null +++ b/gcc/testsuite/gcc.dg/vect/vect-nop-move.c @@ -0,0 +1,64 @@ +/* { dg-do run } */ +/* { dg-require-effective-target vect_float } */ +/* { dg-options -O3 -fdump-rtl-combine-details } */ Please change dg-options

Re: [C++ Patch] Avoid pairs of error calls in duplicate_decls

2013-12-05 Thread Jakub Jelinek
On Tue, Dec 03, 2013 at 11:49:03AM +0100, Paolo Carlini wrote: 2013-12-03 Paolo Carlini paolo.carl...@oracle.com * decl.c (duplicate_decls): Replace pairs of errors and permerrors with error + inform (permerror + inform, respectively). This broke g++.dg/warn/pr15774-1.C test:

C++ PATCH for c++/59044 (partial specialization of member class template)

2013-12-05 Thread Jason Merrill
If a non-type template parameter of a member class template depends on template parameters from the enclosing class template, we need to use the partially instantiated version of the member template in deduction. Tested x86_64-pc-linux-gnu, applying to trunk and 4.8. commit

Re: [SPARC] Fix PR target/59316

2013-12-05 Thread Eric Botcazou
Generically, each iteration of the compare-and-exchange loop should be as if the previous iterations hadn't happened, so restoring all the previous FP state is right in that sense. I see. With regard to the implementation, the insn patterns for the SPARC builtins should specify that the

Re: [C++ Patch] Avoid pairs of error calls in duplicate_decls

2013-12-05 Thread Paolo Carlini
Hi, On 12/05/2013 11:41 PM, Jakub Jelinek wrote: On Tue, Dec 03, 2013 at 11:49:03AM +0100, Paolo Carlini wrote: 2013-12-03 Paolo Carlini paolo.carl...@oracle.com * decl.c (duplicate_decls): Replace pairs of errors and permerrors with error + inform (permerror + inform,

Re: [C++ Patch] Avoid pairs of error calls in duplicate_decls

2013-12-05 Thread Jakub Jelinek
On Fri, Dec 06, 2013 at 12:19:04AM +0100, Paolo Carlini wrote: 2013-12-05 Paolo Carlini paolo.carl...@oracle.com * g++.dg/warn/pr15774-1.C: Adjust expected message. Thanks. Jakub

Re: [RFC] Old school parallelization of WPA streaming

2013-12-05 Thread Jan Hubicka
On Thu, 21 Nov 2013, Jan Hubicka wrote: Why do you need an additional -fparallelism? Wouldn't -fwpa=... be a better match, matching -flto=...? As we already pass down a -fwpa option to WPA this would make things easier, no? My plan was to possibly use same option later for

PR c++/58567: Fix ICE on invalid code with -fopenmp in cp/pt.c

2013-12-05 Thread Tobias Burnus
A rather simple fix for an ICE on invalid bug (low-priority 4.8/4.9 regression). Bootstrapped and regtested without new failure on x86-64-gnu-linux. OK for the trunk and 4.8? Tobias 2013-06-12 Tobias Burnus bur...@net-b.de PR c++/58567 * pt.c (tsubst_omp_for_iterator): Early return for

Re: [PATCH][10/10] -fuse-caller-save - Add test-case

2013-12-05 Thread Tom de Vries
On 27-04-13 12:01, Richard Sandiford wrote: Tom de Vries t...@codesourcery.com writes: +/* { dg-do run } */ +/* { dg-options -fuse-caller-save -save-temps } */ +/* { dg-skip-if { *-*-* } { * } { -Os } } */ +/* Testing -fuse-caller-save optimization option. */ + +static int

Re: [PATCH][IRA] Analysis of register usage of functions for usage by IRA.

2013-12-05 Thread Tom de Vries
On 14-03-13 10:34, Tom de Vries wrote: I thought about implementing your optimization for LRA by myself. But it is ok if you decide to work on it. At least, I am not going to start this work for a month. I'm also currently looking at how to use the analysis in LRA. AFAIU, in lra-constraints.c

Re: [PATCH] Hoist loop invariant statements containing data refs with zero-step during loop-versioning in vectorization.

2013-12-05 Thread Cong Hou
Hi Richard You mentioned that Micha has a patch pending that enables of zero-step stores. What is the status of this patch? I could not find it through searching Micha. Thank you! Cong On Wed, Oct 16, 2013 at 2:02 AM, Richard Biener rguent...@suse.de wrote: On Tue, 15 Oct 2013, Cong Hou

Re: [PATCH][05/10] -fuse-caller-save - Implement TARGET_FN_OTHER_HARD_REG_USAGE hook for ARM

2013-12-05 Thread Tom de Vries
On 30-03-13 18:11, Tom de Vries wrote: Richard, This patch series adds analysis of register usage of functions for usage by IRA. The original post is here ( http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01234.html ). This patch implements the target hook TARGET_FN_OTHER_HARD_REG_USAGE for ARM.

Re: [PATCH][07/10] -fuse-caller-save - Use collected register usage information

2013-12-05 Thread Tom de Vries
On 30-03-13 18:11, Tom de Vries wrote: Paolo, This patch series adds analysis of register usage of functions for usage by IRA. The original post is here ( http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01234.html ). This patch uses the information of which registers are clobbered by a call in

Re: [buildrobot] score: Silence warnings to fix config-list.mk build

2013-12-05 Thread Jeff Law
On 12/05/13 14:33, Jan-Benedict Glaw wrote: On Thu, 2013-12-05 14:22:46 -0700, Jeff Law l...@redhat.com wrote: On 12/05/13 14:17, Jan-Benedict Glaw wrote: diff --git a/gcc/config/score/score.h b/gcc/config/score/score.h index 5ab7875..e3bb7f0 100644 --- a/gcc/config/score/score.h +++

Re: [PATCH, doc] Document -fsanitize=signed-integer-overflow

2013-12-05 Thread Jeff Law
On 12/05/13 02:50, Marek Polacek wrote: Ping. The implementation has been commited. On Thu, Nov 28, 2013 at 01:32:24PM +0100, Marek Polacek wrote: As promised, this patch on top of this patch by Tobias: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03082.html adds the documentation for

Re: [buildrobot] score: Silence warnings to fix config-list.mk build

2013-12-05 Thread Jan-Benedict Glaw
On Thu, 2013-12-05 20:49:06 -0700, Jeff Law l...@redhat.com wrote: On 12/05/13 14:33, Jan-Benedict Glaw wrote: On Thu, 2013-12-05 14:22:46 -0700, Jeff Law l...@redhat.com wrote: On 12/05/13 14:17, Jan-Benedict Glaw wrote: [...] I'd just change this one to be correct for the GNU style. If

Re: [PE-POST] Adjust Bit-region in expand_assignment

2013-12-05 Thread Jeff Law
On 12/04/13 00:49, Bernd Edlinger wrote: Regarding the consistency of bitregion_start/end , they should either both be zero, or bitregion_start = bitpos bitpos+bitsize-1 = bitregion_end Presumably to satisfy the consecutive bitfields are a single memory location stuff from C++11. Thus the

Re: [buildrobot] score: Silence warnings to fix config-list.mk build

2013-12-05 Thread Jeff Law
On 12/05/13 21:10, Jan-Benedict Glaw wrote: On Thu, 2013-12-05 20:49:06 -0700, Jeff Law l...@redhat.com wrote: On 12/05/13 14:33, Jan-Benedict Glaw wrote: On Thu, 2013-12-05 14:22:46 -0700, Jeff Law l...@redhat.com wrote: On 12/05/13 14:17, Jan-Benedict Glaw wrote: [...] I'd just change

Re: [PING^2][PATCH][2 of 2] RTL expansion for zero sign extension elimination with VRP

2013-12-05 Thread Kugan
tem = (char) 255 + (char) 1; tem is always of type 'char' in GIMPLE (even if later promoted via PROMOTE_MODE) the value-range is a 'char' value-range and thus never will exceed [CHAR_MIN, CHAR_MAX]. The only way you can use that directly is if you can rely on undefined behavior

[PATCH][RFA][PR middle-end/59285] Handle BARRIERS between blocks in rtl_merge_blocks

2013-12-05 Thread Jeff Law
builtin-unreachable-6.c when compiled for armv7l has a conditional where the fall-thru block has no successors (due to __builtin_unreachable) and is immediately followed by the jump-to block. In this situation there will be a BARRIER after the fall-thru block (remember, it has no

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

2013-12-05 Thread Jeff Law
On 12/04/13 01:16, Bernd Edlinger wrote: Looking for some more time your patch may be indeed the easiest without big re-factoring. Richard (or Bernd), can you comment on why? Something seems off here. Why do we need to handle inner references here specially? If feels like we're catering

Re: [PING^2] [PATCH] PR59063

2013-12-05 Thread Jeff Law
On 12/03/13 03:08, Yury Gribov wrote: The situation hasn't changed in the last four days. Thanks. Do you think you can check the patch in question? It looks like Jakub took care of it. jeff

Re: [PING^2] [PATCH] PR59063

2013-12-05 Thread Yury Gribov
Jeff Law wrote: It looks like Jakub took care of it. True.

Re: [Ping]Two pending IVOPT patches

2013-12-05 Thread Jeff Law
On 11/26/13 03:52, Bin.Cheng wrote: On Tue, Nov 26, 2013 at 6:06 AM, Jeff Law l...@redhat.com wrote: On 11/25/13 02:11, Bin.Cheng wrote: Slightly tune to make iv cand choosing algorithm more accurate: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg01574.html It would help if you had some

[PATCH] Handle PIEs in libbacktrace

2013-12-05 Thread Jakub Jelinek
Hi! With the new tsan tests, I've noticed that libbacktrace symbolization doesn't work when the binary is a PIE. The problem is that in that case we obviously can't use base_address of 0, the PIE typically will not have 0 bias, that is actually the sole point of PIEs that their base address is

<    1   2