Re: [PATCH] Use libbacktrace as libsanitizer's symbolizer

2013-11-19 Thread Jakub Jelinek
On Mon, Nov 18, 2013 at 09:09:03AM -0800, Ian Lance Taylor wrote: 2) for tsan querying of data symbols, apparently the classes want to see not just the symbol name and start value, but also size. libbacktrace has all this info available, just doesn't pass it down to the callback.

Re: [GOMP4] Generation tables with omp-functions addresses for offloading.

2013-11-19 Thread Michael V. Zolotukhin
Hi Jakub, Thanks for the remarks. Updated patch is attached, and my answers are below. This will add into the table all omp declare target functions, but you actually want there only the outlined #pragma omp target bodies. The question is how to find them here reliably. At least ignoring

Re: [PATCH] S/390: More htm testcases plus some fixes

2013-11-19 Thread Andreas Krebbel
On 18/11/13 17:09, Peter Bergner wrote: On Mon, 2013-11-18 at 10:05 +0100, Andreas Krebbel wrote: With the patch the htm-nofloat-2 testcase fails. Due to the returns_twice flag on tbegin the optimizers fail to fold the compares of the condition code and the s390_optimize_nonescaping_tx

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Paolo Bonzini
Il 18/11/2013 20:09, Jan Hubicka ha scritto: this patch switches the default for fat-lto-objects as was documented for a while. -ffat-lto-objects doubles compilation time and often makes users to not notice that LTO was not used at all (because they forgot to use gcc-ar/gcc-nm

[PATCH] Parameters added for coverage_compute_cfg_checksum

2013-11-19 Thread Martin Liška
Hello, I will need for my further patch more general usage of coverage_compute_cfg_checksum function, so that I added new argument. Is the patch OK? Thank you, Martin diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d759d4c..32ffc2e 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8

Re: [PATCH] Eliminate n_basic_blocks macro (was Re: [PATCH] Avoid some unnecessary set_cfun calls)

2013-11-19 Thread Richard Biener
On Mon, 18 Nov 2013, David Malcolm wrote: On Fri, 2013-11-15 at 20:38 -0500, David Malcolm wrote: On Wed, 2013-11-13 at 14:44 +0100, Richard Biener wrote: On Wed, 13 Nov 2013, David Malcolm wrote: On Wed, 2013-11-13 at 13:53 +0100, Richard Biener wrote: On Wed, 13 Nov 2013,

Re: Add value range support into memcpy/memset expansion

2013-11-19 Thread Richard Biener
On Tue, 19 Nov 2013, Jan Hubicka wrote: Hi, this patch fixes two issues with memcpy testcase - silences warning and updates the template as suggested by Uros in the PR. The testcase still fails on i386. This is because we end up with: ;; Function t (t, funcdef_no=0, decl_uid=1763,

Re: [patch][RFC] make lra.c:check_rtl set maybe_hot_insn_p

2013-11-19 Thread Yvan Roux
yep, all good performance-wise :) Great, Thanks Kyrill. Ok for trunk ? Yvan

Re: [AArch64] Remove v8type attribute.

2013-11-19 Thread Marcus Shawcroft
On 14 November 2013 17:25, James Greenhalgh james.greenha...@arm.com wrote: Now, every insn has a type, we don't need v8type anymore. This patch removes v8type. Tested on aarch64-none-elf with no regression. OK? OK /Marcus

Re: [PATCH] Generate a label for the split cold function while using -freorder-blocks-and-partition

2013-11-19 Thread Eric Botcazou
Thanks! Do I need any other maintainer's approval or can I go ahead and commit, after checking for test regressions and boot-strap parity? Steven's approval is sufficient here. -- Eric Botcazou

Re: [PATCH i386 4/8] [AVX512] [2/n] Add substed patterns: mask scalar subst.

2013-11-19 Thread Kirill Yukhin
Hello, On 15 Nov 20:03, Kirill Yukhin wrote: Ping? Ping? -- Thanks, K

Re: [PATCH i386 4/8] [AVX512] [4/n] Add substed patterns: `sd' subst.

2013-11-19 Thread Kirill Yukhin
Hello, On 15 Nov 20:05, Kirill Yukhin wrote: Is it ok for trunk? Ping. Ping. -- Thanks, K

Re: [PATCH i386 4/8] [AVX512] [5/8] Add substed patterns: rounding subst.

2013-11-19 Thread Kirill Yukhin
Hello, On 15 Nov 20:06, Kirill Yukhin wrote: Ping. Ping. -- Thanks, K

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

2013-11-19 Thread Jakub Jelinek
On Mon, Nov 18, 2013 at 10:35:50PM +, Iyer, Balaji V wrote: Attached, please find a patch that will implement SIMD enabled functions for C targeting the gomp-4_0-branch. Here are the ChangeLog entries. Is this OK to install? Have you tested say: int func9 (int x, int y) __attribute__

Re: [PATCH i386 4/8] [AVX512] [4/n] Add substed patterns: `sd' subst.

2013-11-19 Thread Kirill Yukhin
Adding back Richard. On 19 Nov 12:07, Kirill Yukhin wrote: Hello, On 15 Nov 20:05, Kirill Yukhin wrote: Is it ok for trunk? Ping. Ping. -- Thanks, K

Re: [PATCH i386 4/8] [AVX512] [6/8] Add substed patterns: `sae' subst.

2013-11-19 Thread Kirill Yukhin
Hello, On 15 Nov 20:07, Kirill Yukhin wrote: Is it ok for trunk? Ping. Ping. -- Thanks, K

Re: [PATCH i386 4/8] [AVX512] [8/8] Add substed patterns: `sae-only for expand' subst.

2013-11-19 Thread Kirill Yukhin
Hello, On 15 Nov 20:09, Kirill Yukhin wrote: Is it ok for trunk? Ping. Ping. -- Thanks, K

Re: Pass floating point values on powerpc64 as per ABI

2013-11-19 Thread Andreas Schwab
Alan Modra amo...@gmail.com writes: On Tue, Nov 19, 2013 at 11:16:26AM +1030, Alan Modra wrote: On Tue, Nov 19, 2013 at 01:27:39AM +0100, Andreas Schwab wrote: Where does it call a varargs function? printf Sorry that wasn't such a helpful response. Here, really: res =

Re: [PATCH i386 5/8] [AVX-512] Extend vectorizer hooks.

2013-11-19 Thread Kirill Yukhin
Hello, On 15 Nov 20:10, Kirill Yukhin wrote: Is it ok to commit to main trunk? Ping. Ping. -- Thanks, K

[PATCH] Fix PR57517

2013-11-19 Thread Richard Biener
This fixes a predcom ICE where it commons a combination of two loads but with the combination being conditionally executed. It's not prepared to handle this situation, disabled with the following. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk and 4.8 branch for now.

Re: [PATCH/AARCH64] Emit brk #0 for __builtin_trap

2013-11-19 Thread Marcus Shawcroft
On 13 November 2013 00:04, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: Hi all, This patch implements the trap pattern for the AARCH64 back-end. I used the brk #0 instruction as that is the breakpoint instruction that GDB uses. I looked at what other targets did when the instruction

Re: Add value range support into memcpy/memset expansion

2013-11-19 Thread Jakub Jelinek
On Tue, Nov 19, 2013 at 09:50:56AM +0100, Richard Biener wrote: this patch fixes two issues with memcpy testcase - silences warning and updates the template as suggested by Uros in the PR. The testcase still fails on i386. This is because we end up with: ;; Function t (t,

Re: [Patch, AArch64] Make reduc_* operations bigendian-safe.

2013-11-19 Thread Marcus Shawcroft
On 15 November 2013 16:52, Tejas Belagod tbela...@arm.com wrote: Hi, The attached patch fixes all the reduc_* expansions to be BE-safe by moving the scalar result to the LSB where RTL expects it. While moving it also adds patterns that will give gcc the freedom to choose between

Re: [PATCH] aarch64 gcc.c-torture/execute/20101011-1.c failures

2013-11-19 Thread Marcus Shawcroft
On 18 November 2013 18:02, Cesar Philippidis ce...@codesourcery.com wrote: gcc.c-torture/execute/20101011-1.c test on aarch64. The reason why this test fails is because aarch64 does not trap on integer division by zero. Is this OK for trunk? If so, please commit it because I do not have an

Re: [RS6000] strict alignment for little-endian

2013-11-19 Thread Eric Botcazou
It's not desirable since gcc easily loses track of alignment, for instance with -mstrict-align void foo (char *p, char *q) { __builtin_memcpy (p, q, 4); } generates lbz 7,0(4) lbz 8,1(4) lbz 10,2(4) lbz 9,3(4) stb 7,0(3) stb 8,1(3)

Re: [PATCH] Support addsub/subadd as non-isomorphic operations for SLP vectorizer.

2013-11-19 Thread Richard Biener
On Mon, 18 Nov 2013, Cong Hou wrote: I tried your method and it works well for doubles. But for float, there is an issue. For the following gimple code: c1 = a - b; c2 = a + b; c = VEC_PERM c1, c2, [0,5,2,7] It needs two instructions to implement the VEC_PERM operation in

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-19 Thread Basile Starynkevitch
On Mon, Nov 18, 2013 at 10:50:10PM +, Joseph S. Myers wrote: On Mon, 18 Nov 2013, Basile Starynkevitch wrote: @@ -43,6 +44,7 @@ TARGET_OPTF. */ #include tm_p.h /* For C_COMMON_OVERRIDE_OPTIONS. */ + #ifndef DOLLARS_IN_IDENTIFIERS

Re: [PATCH GCC]Compute, cache and use cost of auto-increment rtx patterns in IVOPT

2013-11-19 Thread Bin.Cheng
On Tue, Nov 19, 2013 at 10:09 AM, bin.cheng bin.ch...@arm.com wrote: -Original Message- From: Bernd Schmidt [mailto:ber...@codesourcery.com] Sent: Monday, November 18, 2013 8:05 PM To: Bin Cheng Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH GCC]Compute, cache and use cost of

Re: [PATCH] Support addsub/subadd as non-isomorphic operations for SLP vectorizer.

2013-11-19 Thread Richard Earnshaw
On 18/11/13 20:19, Cong Hou wrote: On Fri, Nov 15, 2013 at 10:18 AM, Richard Earnshaw rearn...@arm.com wrote: On 15/11/13 02:06, Cong Hou wrote: Hi This patch adds the support to two non-isomorphic operations addsub and subadd for SLP vectorizer. More non-isomorphic operations can be added

Re: Ping Re: [gomp4] Dumping gimple for offload.

2013-11-19 Thread Ilya Tocar
On 14 Nov 11:27, Richard Biener wrote: + /* Set when symbol needs to be dumped for lto/offloading. */ + unsigned need_dump : 1; + That's very non-descriptive. What's offloading? But yes, something like this is what I was asking for. I've changed it into: Set when symbol needs to be

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Markus Trippelsdorf
On 2013.11.19 at 09:44 +0100, Paolo Bonzini wrote: Il 18/11/2013 20:09, Jan Hubicka ha scritto: this patch switches the default for fat-lto-objects as was documented for a while. -ffat-lto-objects doubles compilation time and often makes users to not notice that LTO was not

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Jan Hubicka
On 2013.11.19 at 09:44 +0100, Paolo Bonzini wrote: Il 18/11/2013 20:09, Jan Hubicka ha scritto: this patch switches the default for fat-lto-objects as was documented for a while. -ffat-lto-objects doubles compilation time and often makes users to not notice that LTO

Re: [RFA/RFC patch]: Follow-up on type-demotion pass ...

2013-11-19 Thread Kai Tietz
- Original Message - This is not a review, but: * What do you need from rtl.h? It's generally best for GIMPLE passes to avoid rtl.h where possible (and if you can avoid it, the next question is whether you can also avoid tm.h). Yes, for now rtl.h, tm.h and tm_p.h headers can be

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Paolo Bonzini
Il 19/11/2013 11:05, Markus Trippelsdorf ha scritto: On 2013.11.19 at 09:44 +0100, Paolo Bonzini wrote: Il 18/11/2013 20:09, Jan Hubicka ha scritto: this patch switches the default for fat-lto-objects as was documented for a while. -ffat-lto-objects doubles compilation time and often makes

[RFC] [PATCH, AARCH64] Machine descriptions to support stack smashing protection

2013-11-19 Thread Venkataramanan Kumar
Hi Maintainers, This is RFC patch that adds machine descriptions to support stack smashing protection in AArch64. I have written a very simple patch that prints stack set and stack test as template of instructions. I had 2 assumptions. 1) For stack_protect_set and stack_protect_test, I used

[PING] Fix PR58115

2013-11-19 Thread Bernd Edlinger
PING... this patch still needs review: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00133.html Thanks. on i686-pc-linux-gnu the test case gcc.target/i386/intrinsics_4.c fails because of an internal compiler error, see PR58155. The reason for this is that the optab CODE_FOR_movv8sf is

Re: [RFC] [PATCH, AARCH64] Machine descriptions to support stack smashing protection

2013-11-19 Thread Jakub Jelinek
On Tue, Nov 19, 2013 at 04:30:21PM +0530, Venkataramanan Kumar wrote: This is RFC patch that adds machine descriptions to support stack smashing protection in AArch64. Most of the testsuite changes look wrong. The fact that aarch64 gets stack protector support doesn't mean all other targets do

Re: [PATCH, PR 10474] Take two on splitting live-ranges of function arguments to help shrink-wrapping

2013-11-19 Thread James Greenhalgh
*Ping* This is one of many bugs blocking bootstrap on ARM. It would be helpful if the fix could be reviewed and, if correct, for it to go in. http://gcc.gnu.org/ml/gcc-patches/2013-11/msg01820.html Thanks, James On Fri, Nov 15, 2013 at 03:10:49PM +, Martin Jambor wrote: Perfect, thanks a

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Markus Trippelsdorf
On 2013.11.19 at 11:54 +0100, Paolo Bonzini wrote: Il 19/11/2013 11:05, Markus Trippelsdorf ha scritto: On 2013.11.19 at 09:44 +0100, Paolo Bonzini wrote: Il 18/11/2013 20:09, Jan Hubicka ha scritto: this patch switches the default for fat-lto-objects as was documented for a while.

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Paolo Bonzini
Il 19/11/2013 12:19, Markus Trippelsdorf ha scritto: So, maybe it is just time to upgrade libtool everywhere in gnu-land? Yes, that would be better but no need to do that now. So would Patches 1 and 2 be OK in the interim? Yes. And Jan's answer suggests that patch 3 is not necessary

[PATCH][ARM] Implement CRC32 intrinsics for AArch32 in ARMv8-A

2013-11-19 Thread Kyrill Tkachov
Hi all, This patch implements the CRC32 intrinsics that map down to the optional CRC32 instructions in ARMv8-A as defined by ACLE. They are exposed by a new header file: arm_acle.h which can be included in user programs similarly to the existing arm_neon.h header. To enable the use of these

[PATCH, testsuite]: Add add-dg-options ieee to gcc.dg/c11-complex-1.c

2013-11-19 Thread Uros Bizjak
Hello! 2013-11-19 Uros Bizjak ubiz...@gmail.com * gcc.dg/c11-complex-1.c: Use dg-add-options ieee. Tested on alphaev68-pc-linux-gnu, committed to mainline SVN. Uros. Index: gcc.dg/c11-complex-1.c === ---

Re: [PING] [PATCH] Optional alternative base_expr in finding basis for CAND_REFs

2013-11-19 Thread Yufeng Zhang
Hi Richard, Can I get an approval or some feedback from you about the patch? Regards, Yufeng On 11/13/13 23:25, Yufeng Zhang wrote: On 11/13/13 20:54, Bill Schmidt wrote: Hi Yufeng, The second version of your original patch is ok with me with the following changes. Thanks a lot for the

[0/4] Make more use of tree_to_[su]hwi

2013-11-19 Thread Richard Sandiford
Dull series, sorry, but this is another change taken from wide-int. After checking host_integerp (now tree_fits_[su]hwi_p), the preferred way of getting the HWI seemed to be tree_low_cst (now tree_to_[su]hwi). This series mops up some cases where TREE_INT_CST_LOW was used directly. Tested on

[1/4] Use tree_to_uhwi with an inlined tree_fits_uhwi_p test

2013-11-19 Thread Richard Sandiford
check_function_arguments_recurse has an assert that is equivalent to tree_fits_uhwi_p. The extraction can then use tree_to_uhwi. Asserting here makes the intent obvious, but tree_to_uhwi also asserts for the same thing, so an alternative would be to use tree_to_uhwi on its own. Thanks, Richard

Re: [PATCH] RTEMS: Add LEON3/SPARC multilibs

2013-11-19 Thread Sebastian Huber
Hello Eric, On 2013-09-19 09:23, Eric Botcazou wrote: I don't expect that this will be back ported to GCC 4.8. You also need Binutils 2.24 for this. From a SPARC maintainership viewpoint, I'd think that this is backportable for the upcoming 4.8.2 release, and the patches are essentially

[2/4] Use tree_to_uhwi when folding (x c) c

2013-11-19 Thread Richard Sandiford
The (x c) c folding has: tree_fits_shwi_p (arg1) TREE_INT_CST_LOW (arg1) prec tree_fits_shwi_p (TREE_OPERAND (arg0, 1)) TREE_INT_CST_LOW (TREE_OPERAND (arg0, 1)) prec) At first glance the use of tree_fits_shwi_p rather than tree_fits_uhwi_p made me

Re: [PATCH, MPX, 2/X] Pointers Checker [8/25] Languages support

2013-11-19 Thread Richard Biener
On Mon, Nov 18, 2013 at 5:45 PM, Jeff Law l...@redhat.com wrote: On 11/08/13 02:02, Ilya Enkovich wrote: Hi, Here is an updated patch version with no langhook. Regarding TLS objects issue - I do not think compiler should compensate the absence of instrumentation in libraries. Compiler

Re: [PATCH, MPX, 2/X] Pointers Checker [14/25] Function splitting

2013-11-19 Thread Richard Biener
On Mon, Nov 18, 2013 at 8:12 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/18 Jeff Law l...@redhat.com: On 11/18/13 11:27, Ilya Enkovich wrote: How does pointer passed to regular function differ from pointer passed to splitted function? How do I know then which pointer is to be

[3/4] Avoid undefined operation in overflow check

2013-11-19 Thread Richard Sandiford
This is a case where tree_to_shwi can be used instead of TREE_INT_CST_LOW. I separated it out because it was using a signed x * y / y == x to check whether x * y overflows a HWI, which relies on undefined behaviour. Thanks, Richard gcc/ * tree-ssa-alias.c

[4/4] The rest of the tree_to_[su]hwi changes

2013-11-19 Thread Richard Sandiford
This patch just changes TREE_INT_CST_LOW to tree_to_[su]hwi in cases where there is already a protecting tree_fits_[su]hwi_p. I've upped the number of context lines in case that helps, but there are still some hunks where the tree_fits_* call is too high up. Thanks, Richard gcc/ada/ 2013-11-19

Re: [PATCH, MPX, 2/X] Pointers Checker [8/25] Languages support

2013-11-19 Thread Ilya Enkovich
On 19 Nov 13:00, Richard Biener wrote: On Mon, Nov 18, 2013 at 5:45 PM, Jeff Law l...@redhat.com wrote: On 11/08/13 02:02, Ilya Enkovich wrote: Hi, Here is an updated patch version with no langhook. Regarding TLS objects issue - I do not think compiler should compensate the

Re: [PATCH] add auto_vec

2013-11-19 Thread Richard Biener
On Mon, Nov 18, 2013 at 10:08 PM, Trevor Saunders tsaund...@mozilla.com wrote: On Mon, Nov 18, 2013 at 10:03:53PM +0100, Marc Glisse wrote: On Mon, 18 Nov 2013, Trevor Saunders wrote: This patch adds a class auto_vec which releases its internal storage in its destructor, but unlike stack_vec

Re: [PATCH, MPX, 2/X] Pointers Checker [14/25] Function splitting

2013-11-19 Thread Ilya Enkovich
2013/11/19 Richard Biener richard.guent...@gmail.com: On Mon, Nov 18, 2013 at 8:12 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/18 Jeff Law l...@redhat.com: On 11/18/13 11:27, Ilya Enkovich wrote: How does pointer passed to regular function differ from pointer passed to splitted

Re: [PATCH, MPX, 2/X] Pointers Checker [15/25] IPA Propagation

2013-11-19 Thread Martin Jambor
Hi, On Mon, Nov 18, 2013 at 10:38:49PM +0400, Ilya Enkovich wrote: 2013/11/18 Martin Jambor mjam...@suse.cz: On Mon, Nov 18, 2013 at 02:28:58PM +0400, Ilya Enkovich wrote: Hi, Here is a patch to disable propagation of bounded values. Why do ypu need to do this? If the problem is

Re: [PATCH] add auto_vec

2013-11-19 Thread Jakub Jelinek
On Mon, Nov 18, 2013 at 02:46:19PM -0500, Trevor Saunders wrote: 2013-11-18 Trevor Saunders tsaund...@mozilla.com gcc/ * vec.h (auto_vec): New class. * cfganal.c cfgloop.c cgraphunit.c config/i386/i386.c dwarf2out.c function.c genautomata.c gimple.c haifa-sched.c ipa-inline.c

Re: Change warnings for unsupported alignment to errors

2013-11-19 Thread Richard Biener
On Tue, Nov 19, 2013 at 2:45 AM, Joseph S. Myers jos...@codesourcery.com wrote: When implementing C11 _Alignas in http://gcc.gnu.org/ml/gcc-patches/2011-11/msg00841.html, I noted that I has omitted checks that alignment was supported (which, as constraints in C11, should be errors or pedwarns

Re: Factor unrelated declarations out of tree.h (1/2)

2013-11-19 Thread Diego Novillo
On Tue, Nov 19, 2013 at 12:17 AM, Jeff Law l...@redhat.com wrote: It looks OK to me. Thanks. Committed as rev 205023. Ian, the Go front end will need that patch committed now. Diego.

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Markus Trippelsdorf
On 2013.11.19 at 11:21 +0100, Jan Hubicka wrote: On 2013.11.19 at 09:44 +0100, Paolo Bonzini wrote: Il 18/11/2013 20:09, Jan Hubicka ha scritto: this patch switches the default for fat-lto-objects as was documented for a while. -ffat-lto-objects doubles compilation time and

[PATCH i386 8/8] [AVX-512] Add SHA support.

2013-11-19 Thread Kirill Yukhin
Hello, This patch introduces new SHA instructions described in [1] along with tests. Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000 2006 build show no regressions both with and without -mavx512f option. 4. Spec 2000 2006 run shows no stability regressions

Make simple loop peeling to happen at gimple level

2013-11-19 Thread Jan Hubicka
Hi, this is update of my 2012 patch to move rtl loop peeling (the one based on profile feedback) to tree level. Peeling expose new optimization oppurtunities and it is good idea to have gimple passes to see them. Moreover we probably want to hook in a new heuristic that use value histograms of

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Jan Hubicka
Hmm, gcc/gcc.c still reads: 690 /* Conditional to test whether the LTO plugin is used or not. 691FIXME: For slim LTO we will need to enable plugin unconditionally. This 692still cause problems with PLUGIN_LD != LD and when plugin is built but 693not useable. For GCC 4.6

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Richard Biener
On Tue, 19 Nov 2013, Jan Hubicka wrote: Hmm, gcc/gcc.c still reads: 690 /* Conditional to test whether the LTO plugin is used or not. 691FIXME: For slim LTO we will need to enable plugin unconditionally. This 692still cause problems with PLUGIN_LD != LD and when plugin

[PATCH] Fix PR58956

2013-11-19 Thread Richard Biener
This fixes PR58956 where we TER a load into a call statement lhs that is modified by the call. TER already has measures to avoid doing this for regular assignments so the following simply extends it to arbitrary stmts (including ASMs). Bootstrapped and tested on x86_64-unknown-linux-gnu,

[x86,PATCH] Additional fix for 57756.

2013-11-19 Thread Yuri Rumyantsev
Hi All, We found out that compiler configured with '-fpmath=sse' option does not generate scalar floating-point instructions present in the SSE instruction set for generic32 that leads to performance degradation for Fortran benchmarks using library functions in 32-bit mode. This simple fix was

[patch] fix graphite build

2013-11-19 Thread Andrew MacLeod
graphite-sese-to-poly.c needs expr.h to compile. Fixed thusly and checked in as revision 205027. Andrew * graphite-sese-to-poly.c: Include expr.h. Index: graphite-sese-to-poly.c === *** graphite-sese-to-poly.c (revision 205024)

Re: [x86,PATCH] Additional fix for 57756.

2013-11-19 Thread H.J. Lu
On Tue, Nov 19, 2013 at 5:31 AM, Yuri Rumyantsev ysrum...@gmail.com wrote: Hi All, We found out that compiler configured with '-fpmath=sse' option does not generate scalar floating-point instructions present in the SSE instruction set for generic32 that leads to performance degradation for

Re: [patch] fix graphite build

2013-11-19 Thread Diego Novillo
On Tue, Nov 19, 2013 at 8:36 AM, Andrew MacLeod amacl...@redhat.com wrote: graphite-sese-to-poly.c needs expr.h to compile. Fixed thusly and checked in as revision 205027. Thanks! Diego.

Re: [x86,PATCH] Additional fix for 57756.

2013-11-19 Thread Yuri Rumyantsev
Resend modified ChangeLog: 2013-11-20 Yuri Rumyantsev ysrum...@gmail.com PR target/57756 * config/i386/i386.c (ix86_option_override_internal): Add missed argument prefix for 'ix86_fpmath'. * config/i386/ssemath.h: Add missed definition of TARGET_FPMATH_DEFAULT_P macros. 2013/11/19 H.J. Lu

Re: [x86,PATCH] Additional fix for 57756.

2013-11-19 Thread Uros Bizjak
On Tue, Nov 19, 2013 at 2:45 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Resend modified ChangeLog: 2013-11-20 Yuri Rumyantsev ysrum...@gmail.com PR target/57756 * config/i386/i386.c (ix86_option_override_internal): Add missed argument prefix for 'ix86_fpmath'. * config/i386/ssemath.h:

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Jan Hubicka
On Tue, 19 Nov 2013, Jan Hubicka wrote: Hmm, gcc/gcc.c still reads: 690 /* Conditional to test whether the LTO plugin is used or not. 691FIXME: For slim LTO we will need to enable plugin unconditionally. This 692still cause problems with PLUGIN_LD != LD and when

Re: Enale -fno-fat-lto-objects by default

2013-11-19 Thread Jan Hubicka
On Tue, 19 Nov 2013, Jan Hubicka wrote: Hmm, gcc/gcc.c still reads: 690 /* Conditional to test whether the LTO plugin is used or not. 691FIXME: For slim LTO we will need to enable plugin unconditionally. This 692still cause problems with PLUGIN_LD != LD

Re: [gomp4 simd, RFC] Simple fix to override vectorization cost estimation.

2013-11-19 Thread Sergey Ostanevich
:) agree to you, but as soon as you're a user who tries to introduce vector code and face a bug in cost model you'd like to have a workaround until the bug will be fixed and compiler will come to you with new OS distribution, don't you? I propose the following, yet SLP have to use a NULL as a

Re: [patch] gcc fstack-protector-explicit

2013-11-19 Thread Marcos Díaz
My employer is working on the signature of the papers. Could someone please do the review meanwhile? On Tue, Nov 19, 2013 at 3:00 AM, Jeff Law l...@redhat.com wrote: On 11/18/13 13:05, Marcos Díaz wrote: Hi, the attached patch adds a new attribute and option flag to control when to do

[PATCH, MPX, 2/X] Pointers Checker. Add flag to varpool_node

2013-11-19 Thread Ilya Enkovich
Hi, Here is a patch to add flag for marking symbols as requiring static initialization of bounds. Used by Pointer Bounds Checker to handle statically initialized pointers and static bounds vars. Thanks, Ilya -- 2013-11-19 Ilya Enkovich ilya.enkov...@intel.com * cgraph.h

Re: [gomp4 simd, RFC] Simple fix to override vectorization cost estimation.

2013-11-19 Thread Richard Biener
On Tue, 19 Nov 2013, Sergey Ostanevich wrote: :) agree to you, but as soon as you're a user who tries to introduce vector code and face a bug in cost model you'd like to have a workaround until the bug will be fixed and compiler will come to you with new OS distribution, don't you? I

Re: [gomp4 simd, RFC] Simple fix to override vectorization cost estimation.

2013-11-19 Thread Jakub Jelinek
On Tue, Nov 19, 2013 at 03:07:52PM +0100, Richard Biener wrote: On Tue, 19 Nov 2013, Sergey Ostanevich wrote: :) agree to you, but as soon as you're a user who tries to introduce vector code and face a bug in cost model you'd like to have a workaround until the bug will be fixed and

Re: [PATCH] Fix PR58115

2013-11-19 Thread H.J. Lu
On Sun, Nov 3, 2013 at 2:25 AM, Bernd Edlinger bernd.edlin...@hotmail.de wrote: Hello, on i686-pc-linux-gnu the test case gcc.target/i386/intrinsics_4.c fails because of an internal compiler error, see PR58155. The reason for this is that the optab CODE_FOR_movv8sf is disabled when it

Re: [gomp4 simd, RFC] Simple fix to override vectorization cost estimation.

2013-11-19 Thread Sergey Ostanevich
On Tue, Nov 19, 2013 at 6:07 PM, Richard Biener rguent...@suse.de wrote: On Tue, 19 Nov 2013, Sergey Ostanevich wrote: :) agree to you, but as soon as you're a user who tries to introduce vector code and face a bug in cost model you'd like to have a workaround until the bug will be fixed and

RE: [PATCH] Fix PR58115

2013-11-19 Thread Bernd Edlinger
On Tue, 19 Nov 2013 06:21:22, H.J. Lu wrote: On Sun, Nov 3, 2013 at 2:25 AM, Bernd Edlinger bernd.edlin...@hotmail.de wrote: Hello, on i686-pc-linux-gnu the test case gcc.target/i386/intrinsics_4.c fails because of an internal compiler error, see PR58155. The reason for this is that the

Re: [PATCH i386 8/8] [AVX-512] Add SHA support.

2013-11-19 Thread Uros Bizjak
On Tue, Nov 19, 2013 at 1:41 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch introduces new SHA instructions described in [1] along with tests. Testing: 1. Bootstrap pass. 2. make check shows no regressions. 3. Spec 2000 2006 build show no regressions both with

[PATCH, PR 57748] Check for out of bounds access, Part 2

2013-11-19 Thread Bernd Edlinger
Hello, this is a minor update to my previous version of this patch, (using a boolean expand_reference, instead of adding a new expand_modifier enum value): I forgot to pass down the expand_reference value at the second expand_expr call inside the case VIEW_CONVERT_EXPR. Sorry for the

Re: [gomp4 simd, RFC] Simple fix to override vectorization cost estimation.

2013-11-19 Thread Sergey Ostanevich
I propose the following, yet SLP have to use a NULL as a loop info which looks somewhat hacky. I think this is overengineering. -fvect-cost-model will do as workaround. And -fsimd-vect-cost-model has what I consider duplicate - simd and vect. I think it is a good idea, though I agree

Re: [gomp4 simd, RFC] Simple fix to override vectorization cost estimation.

2013-11-19 Thread Jakub Jelinek
On Tue, Nov 19, 2013 at 06:39:48PM +0400, Sergey Ostanevich wrote: pedwarn isn't really desirable for this, you want just warning, but some warning you can actually also turn off. -Wopenmp-simd (and we'd use it also when we ignore #pragma omp declare simd because it wasn't

Re: [PATCH] Use libbacktrace as libsanitizer's symbolizer

2013-11-19 Thread Ian Lance Taylor
On Mon, Nov 18, 2013 at 11:44 PM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Nov 19, 2013 at 05:32:12PM +1030, Alan Modra wrote: On Tue, Nov 19, 2013 at 06:17:41AM +0100, Hans-Peter Nilsson wrote: In file included from /tmp/x/gcc/libbacktrace/atomic.c:37:

Re: [PATCH] Use libbacktrace as libsanitizer's symbolizer

2013-11-19 Thread Ian Lance Taylor
On Tue, Nov 19, 2013 at 12:04 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Nov 18, 2013 at 09:09:03AM -0800, Ian Lance Taylor wrote: 2) for tsan querying of data symbols, apparently the classes want to see not just the symbol name and start value, but also size. libbacktrace has

Re: [PATCH] Use libbacktrace as libsanitizer's symbolizer

2013-11-19 Thread Jakub Jelinek
On Tue, Nov 19, 2013 at 06:43:07AM -0800, Ian Lance Taylor wrote: 2013-11-19 Jakub Jelinek ja...@redhat.com * backtrace.h (backtrace_syminfo_callback): Add symsize argument. * elf.c (elf_syminfo): Pass 0 or sym-size to the callback as last argument. *

Please don't commit changes to gcc/go/gofrontend

2013-11-19 Thread Ian Lance Taylor
Hi, as noted in gcc/go/README.gcc, the files in gcc/go/gofrontend are actually mirrored from a different repository. Please do not directly commit changes to those files. Instead, send the changes to me. I will commit them upstream. Thanks. Ian

[PATCH i386] Enable -freorder-blocks-and-partition

2013-11-19 Thread Teresa Johnson
This patch enables -freorder-blocks-and-partition by default for x86 at -O2 and up. It is showing some modest gains in cpu2006 performance with profile feedback and -O2 on an Intel Westmere system. Specifically, I am seeing consistent improvements in 401.bzip2 (1.5-3%), 483.xalancbmk (1.5-3%), and

[PATCH] Fix PR59164

2013-11-19 Thread Richard Biener
This fixes PR59164 - a mismatch during vectorizer analysis and transform. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk sofar. Richard. 2013-11-19 Richard Biener rguent...@suse.de PR tree-optimization/59164 * tree-vect-loop-manip.c

Re: [PATCH i386 7/8] [AVX-512] Add tests.

2013-11-19 Thread Uros Bizjak
On Tue, Nov 19, 2013 at 11:35 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Here is a patch that introduces tests for AVX-512 instructions. While implementing testsuite we were strongly connected to the fact that we don't want more then 2 test files per each instruction - a scan assembler

Go patch committed: Update for mainline changes

2013-11-19 Thread Ian Lance Taylor
This patch to the Go frontend incorporates patches by Richard S and Diego for changes to the middle-end. These patches were already committed to the GCC repository. This change commits them to the master Go repository. Ian diff -r 75537ee240ab go/expressions.cc --- a/go/expressions.cc Mon Nov

[v3 patch] move library TS status to new table

2013-11-19 Thread Jonathan Wakely
2013-11-19 Jonathan Wakely jwakely@gmail.com * doc/xml/manual/status_cxx2014.xml: Create new table for TS statuses. Committed to trunk. commit f6ca0fe406b379b4fd7cd4575756fda04156cde0 Author: Jonathan Wakely jwakely@gmail.com Date: Tue Nov 19 14:56:22 2013 + *

[PATCH] C++-ify and simplify loop iterators

2013-11-19 Thread Richard Biener
$subject - the following turns loop_iterator li; FOR_EACH_LOOP (li, loop, LI_ONLY_INNERMOST) { ... if () FOR_EACH_LOOP_BREAK; } into FOR_EACH_LOOP (loop, LI_ONLY_INNERMOST) { ... if () break; } Bootstrapped on x86_64-unknown-linux-gnu, testing

libgo patch committed: Update for libbacktrace change

2013-11-19 Thread Ian Lance Taylor
This patch to libgo updates the use of the libbacktrace library for the recent addition of a size argument to the syminfo callback. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 923fd178d72b libgo/runtime/go-caller.c ---

[PATCH, rs6000] Fix ICE when loading vectors into GPRs in little-endian

2013-11-19 Thread Ulrich Weigand
Hello, running the testsuite in powerpc64le-linux with --with-cpu=power7 causes FAIL: tmpdir-g++.dg-struct-layout-1/t024 cp_compat_x_tst.o compile, (internal compiler error) due to an unrecognizable insn (insn 137 136 138 5 (set (reg:V2DI 5 5) (vec_select:V2DI (reg:V2DI 211)

[PATCH, rs6000] Make ppc64-abi-2.c test case endian safe

2013-11-19 Thread Ulrich Weigand
Hello, some routines in the ppc64-abi-2.c test case attempt to verify that the slots of the parameter save area in the caller hold correct values. However, those slots holds (parts of) vector int data, which the test case compares against immediate long values. This of course hard-codes byte

Re: [PATCH] Add reference binding instrumentation

2013-11-19 Thread Jason Merrill
On 11/18/2013 11:39 AM, Marek Polacek wrote: + init = fold_build2 (COMPOUND_EXPR, TREE_TYPE (init), + ubsan_instrument_reference (input_location, init), + init); This looks like it will evaluate init twice. Jason

Re: Please don't commit changes to gcc/go/gofrontend

2013-11-19 Thread Diego Novillo
On Tue, Nov 19, 2013 at 9:48 AM, Ian Lance Taylor i...@google.com wrote: Hi, as noted in gcc/go/README.gcc, the files in gcc/go/gofrontend are actually mirrored from a different repository. Please do not directly commit changes to those files. Instead, send the changes to me. I will commit

Re: [PATCH, rs6000] Fix ICE when loading vectors into GPRs in little-endian

2013-11-19 Thread David Edelsohn
On Tue, Nov 19, 2013 at 10:03 AM, Ulrich Weigand uweig...@de.ibm.com wrote: Hello, running the testsuite in powerpc64le-linux with --with-cpu=power7 causes FAIL: tmpdir-g++.dg-struct-layout-1/t024 cp_compat_x_tst.o compile, (internal compiler error) due to an unrecognizable insn (insn

  1   2   3   >