[PING] [PATCH 1/n] Add conditional compare support

2013-11-20 Thread Zhenqiang Chen
Ping? Thanks! -Zhenqiang -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen Sent: Wednesday, November 06, 2013 3:39 PM To: 'Richard Henderson' Cc: Richard Earnshaw; 'Richard Biener'; GCC Patches Subject: RE:

Re: [PATCH] Support -fsanitize=leak

2013-11-20 Thread Jakub Jelinek
On Wed, Nov 20, 2013 at 11:40:14AM +0400, Yury Gribov wrote: Jakub already wrote the invoke.texi part: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg02061.html Ah, thanks, I somehow missed it. When will this end up in trunk? I have a depending patch. When it is reviewed. I have various

[PATCH] Improve { x, x + 3, x + 6, x + 9 } expansion

2013-11-20 Thread Jakub Jelinek
Hi! I've noticed we generate terrible code for the testcase below. E.g. with -mavx2 it is: leal6(%rdi), %edx leal12(%rdi), %ecx leal18(%rdi), %esi leal3(%rdi), %eax movl%edx, -20(%rsp) movl%ecx, -24(%rsp) leal

Re: [PATCH, i386, MPX, 2/X] Pointers Checker [21/25] Size relocation

2013-11-20 Thread Uros Bizjak
On Tue, Nov 19, 2013 at 9:43 PM, Ilya Enkovich enkovich@gmail.com wrote: Here is a patch to add size relocation and instruction to obtain object's size in i386 target. +(define_insn move_size_reloc_mode + [(set (match_operand:SWI48 0 register_operand =r) +(match_operand:MODE

Re: [AArch64] AArch64 SIMD Builtins Better Type Correctness.

2013-11-20 Thread Marcus Shawcroft
On 18 November 2013 09:10, James Greenhalgh james.greenha...@arm.com wrote: 2013-11-18 James Greenhalgh james.greenha...@arm.com * gcc/config/aarch64/aarch64-builtins.c (aarch64_simd_itype): Remove. (aarch64_simd_builtin_datum): Remove itype, add qualifiers

[buildrobot] [PATCH] mips: Really remove ENTRY_BLOCK_PTR

2013-11-20 Thread Jan-Benedict Glaw
Hi! One usage of ENTRY_BLOCK_PTR was missed: g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros -Wno-overlength-strings

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

2013-11-20 Thread Richard Biener
On Tue, 19 Nov 2013, Cong Hou wrote: On Tue, Nov 19, 2013 at 1:45 AM, Richard Biener rguent...@suse.de wrote: 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;

Re: [buildrobot] [PATCH] mips: Really remove ENTRY_BLOCK_PTR

2013-11-20 Thread Richard Sandiford
Jan-Benedict Glaw jbg...@lug-owl.de writes: 2013-11-20 Jan-Benedict Glaw jbg...@lug-owl.de * config/mips/mips.c (r10k_simplify_address): Eliminate macro usage. OK. And thanks for the catch. Richard

Re: [buildrobot] [PATCH] mips: Really remove ENTRY_BLOCK_PTR

2013-11-20 Thread Steven Bosscher
On Wed, Nov 20, 2013 at 10:04 AM, Jan-Benedict Glaw wrote: 2013-11-20 Jan-Benedict Glaw ... * config/mips/mips.c (r10k_simplify_address): Eliminate macro usage. diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c index 82ca719..d06d574 100644 --- a/gcc/config/mips/mips.c

Re: RFA: patch to fix PR58785 (an ARM LRA crash)

2013-11-20 Thread Yvan Roux
Hi, as Richard said, only a subset of rclass is allowed to be returned by preferred_reload_class. I've tested the attached patched in Thumb mode, on ARMv5, A9 and A9hf and on cross A15 without regression. Yvan 2013-11-20 Yvan Roux yvan.r...@linaro.org PR target/58785 *

Re: [PATCH] Improve { x, x + 3, x + 6, x + 9 } expansion

2013-11-20 Thread Richard Biener
On Wed, 20 Nov 2013, Jakub Jelinek wrote: Hi! I've noticed we generate terrible code for the testcase below. E.g. with -mavx2 it is: leal6(%rdi), %edx leal12(%rdi), %ecx leal18(%rdi), %esi leal3(%rdi), %eax movl%edx, -20(%rsp)

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

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 10:58 AM, Ilya Tocar tocarip.in...@gmail.com wrote: 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

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

2013-11-20 Thread Jakub Jelinek
On Wed, Nov 20, 2013 at 10:34:30AM +0100, Richard Biener wrote: On Tue, Nov 19, 2013 at 10:58 AM, Ilya Tocar tocarip.in...@gmail.com wrote: On 14 Nov 11:27, Richard Biener wrote: + /* Set when symbol needs to be dumped for lto/offloading. */ + unsigned need_dump : 1; + That's

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

2013-11-20 Thread Richard Biener
On Mon, Nov 18, 2013 at 8:46 PM, Kai Tietz kti...@redhat.com wrote: Hello, this is the second approach of the patch I've sent some months ago. I think we saw that there was no other approach shown, so I will continue on that ... The idea of this pass is to introduce a statement

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

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 12:54 PM, Richard Sandiford rdsandif...@googlemail.com wrote: 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

Re: [PATCH] Improve { x, x + 3, x + 6, x + 9 } expansion

2013-11-20 Thread Jakub Jelinek
On Wed, Nov 20, 2013 at 10:31:38AM +0100, Richard Biener wrote: Aww ;) Nice improvement. Generally when I see this I always wonder whether we want to do this kind of stuff pre RTL expansion. 1st to not rely on being able to TER, 2nd to finally eventually get rid of TER. These patches are

Re: [3/4] Avoid undefined operation in overflow check

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 1:06 PM, Richard Sandiford rdsandif...@googlemail.com wrote: 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

Re: [wwwdocs] Broken links

2013-11-20 Thread Patrick Marlier
+li Secify code[docs]/code in the subject header./li Also [wwwdocs] instead of [docs]. -- Patrick Marlier

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

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/19 Jeff Law l...@redhat.com: On 11/19/13 05:20, Ilya Enkovich wrote: 2013/11/19 Richard Biener richard.guent...@gmail.com: On Mon, Nov 18, 2013 at 8:12 PM, Ilya Enkovich enkovich@gmail.com wrote:

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

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 9:29 PM, Ilya Enkovich enkovich@gmail.com wrote: On 19 Nov 12:33, Jeff Law wrote: On 11/19/13 05:13, Ilya Enkovich wrote: On 19 Nov 13:00, Richard Biener wrote: I'd say not in the gimplifier either but in varpool (symbol table) code where the symbols are

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

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 10:57 AM, Richard Biener richard.guent...@gmail.com wrote: On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/19 Jeff Law l...@redhat.com: On 11/19/13 05:20, Ilya Enkovich wrote: 2013/11/19 Richard Biener richard.guent...@gmail.com:

Re: [buildrobot] [PATCH] mips: Really remove ENTRY_BLOCK_PTR

2013-11-20 Thread Jan-Benedict Glaw
On Wed, 2013-11-20 10:08:45 +0100, Steven Bosscher stevenb@gmail.com wrote: [...] I wonder if there are any more cases like this missed... Could you please check that? Something like: egrep -w ENTRY_BLOCK_PTR|EXIT_BLOCK_PTR gcc/*.[ch] gcc/config/*.[ch] gcc/config/*/*.{c,h,md} No more

Re: [PATCH, MPX, 2/X] Pointers Checker [16/25] Tail recursion

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 8:02 PM, Jeff Law l...@redhat.com wrote: On 11/18/13 14:03, Ilya Enkovich wrote: 2013/11/19 Jeff Law l...@redhat.com: On 11/18/13 12:16, Ilya Enkovich wrote: With current recursion elimination we will have: test (int *param1) { bb1: bb2: _7 = PHIparam1(D)

Re: [PATCH, 4.8] Backport patches to fix parallel build fail PR 57683

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 2:30 AM, Zhenqiang Chen zhenqiang.c...@linaro.org wrote: Hi, The issue files in PR 57683 include lra-constraints.o, lra-eliminations.o and tree-switch-conversion.o. Jeff had fixed them in trunk (r197467 and r198999):

Re: [PATCH] Improve { x, x + 3, x + 6, x + 9 } expansion

2013-11-20 Thread Richard Biener
On Wed, 20 Nov 2013, Jakub Jelinek wrote: On Wed, Nov 20, 2013 at 10:31:38AM +0100, Richard Biener wrote: Aww ;) Nice improvement. Generally when I see this I always wonder whether we want to do this kind of stuff pre RTL expansion. 1st to not rely on being able to TER, 2nd to finally

[PATCH] Fix PR59035

2013-11-20 Thread Richard Biener
The following makes sure the languages default of -ffp-contract is properly transfered to the LTO stage (which defaults to fast). Merging different options from different TUs is done conservatively. LTO bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2013-11-20 Richard

Re: [PATCH, MPX, 2/X] Pointers Checker [16/25] Tail recursion

2013-11-20 Thread Ilya Enkovich
2013/11/20 Richard Biener richard.guent...@gmail.com: On Tue, Nov 19, 2013 at 8:02 PM, Jeff Law l...@redhat.com wrote: On 11/18/13 14:03, Ilya Enkovich wrote: 2013/11/19 Jeff Law l...@redhat.com: On 11/18/13 12:16, Ilya Enkovich wrote: With current recursion elimination we will have:

[PATCH] Strict volatile bit-fields clean-up

2013-11-20 Thread Bernd Edlinger
Hello Richard, as a follow-up patch to the bit-fields patch(es), I wanted to remove the dependencies on the variable flag_strict_volatile_bitfields from expand_assignment and expand_expr_real_1. Additionally I want the access mode of the field to be selected in the memory context, instead of

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

2013-11-20 Thread Ilya Enkovich
2013/11/20 Richard Biener richard.guent...@gmail.com: On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/19 Jeff Law l...@redhat.com: On 11/19/13 05:20, Ilya Enkovich wrote: 2013/11/19 Richard Biener richard.guent...@gmail.com: On Mon, Nov 18, 2013 at 8:12

Re: [PATCH, MPX, 2/X] Pointers Checker [16/25] Tail recursion

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 11:41 AM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/20 Richard Biener richard.guent...@gmail.com: On Tue, Nov 19, 2013 at 8:02 PM, Jeff Law l...@redhat.com wrote: On 11/18/13 14:03, Ilya Enkovich wrote: 2013/11/19 Jeff Law l...@redhat.com: On 11/18/13

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

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 11:55 AM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/20 Richard Biener richard.guent...@gmail.com: On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/19 Jeff Law l...@redhat.com: On 11/19/13 05:20, Ilya Enkovich wrote:

Re: [AArch64] Remove mode, mode2 attributes

2013-11-20 Thread Richard Earnshaw
On 19/11/13 18:12, James Greenhalgh wrote: There are no consumers for these attributes, nor should there ever be. Remove them. Regression tested on aarch64-none-elf with no issues. OK? Thanks, James --- 2013-11-19 James Greenhalgh james.greenha...@arm.com *

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

2013-11-20 Thread Ilya Enkovich
2013/11/20 Richard Biener richard.guent...@gmail.com: On Wed, Nov 20, 2013 at 11:55 AM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/20 Richard Biener richard.guent...@gmail.com: On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/19 Jeff Law

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

2013-11-20 Thread Kirill Yukhin
On 19 Nov 15:36, Uros Bizjak wrote: Please also add new command options to g++.dg/other/sse-2.C and g++.dg/other/sse-3.C OK with the small nit below and with above testsute addition. No need to document negative option here. Thanks! I'll fix your inputs and check in after rest is

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

2013-11-20 Thread Eric Botcazou
Thanks. Committed as rev 205023. Nice work, but why did you antedate the entries in the various ChangeLog files? That's rather confusing when you use them to track things in specific directories (yes, we all know your opinion about ChangeLog files ;-) -- Eric Botcazou

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

2013-11-20 Thread Andreas Schwab
Jan Hubicka hubi...@ucw.cz writes: * config/bootstrap-lto.mk: Use -ffat-lto-objects. * common.opt (ffat-lto-objects): Disable by default. * doc/invoke.texi (fat-lto-objects): Update documentation. * opts.c: Enable fat-lto-objects on lto plugin disable setups. This is

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

2013-11-20 Thread Jan Hubicka
Jan Hubicka hubi...@ucw.cz writes: * config/bootstrap-lto.mk: Use -ffat-lto-objects. * common.opt (ffat-lto-objects): Disable by default. * doc/invoke.texi (fat-lto-objects): Update documentation. * opts.c: Enable fat-lto-objects on lto plugin disable setups. This is

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

2013-11-20 Thread Jan Hubicka
Hi, actually the flag is being passed to the frontends instead of being consumed by the driver that makes life easier. I am testing the attached patch and will commit it as obvious if it passes (to unbreak the testing). Index: opts.c

Re: [PATCH, i386, MPX, 2/X] Pointers Checker [21/25] Size relocation

2013-11-20 Thread Ilya Enkovich
On 20 Nov 09:49, Uros Bizjak wrote: On Tue, Nov 19, 2013 at 9:43 PM, Ilya Enkovich enkovich@gmail.com wrote: Here is a patch to add size relocation and instruction to obtain object's size in i386 target. +(define_insn move_size_reloc_mode + [(set (match_operand:SWI48 0

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

2013-11-20 Thread Sergey Ostanevich
Thanks for comments, hope I got all of em. Note: I used a LOOP_VINFO_LOC (loop_vinfo) to print the loop location but it appears to be 0, so the output is somewhat lousy. The global vect_location points somewhere inside the loop, which is not that better. Shall we address this separately? Sergos

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

2013-11-20 Thread Jan Hubicka
Hi, this is version I commited - we need to intrdocue var for -fuse-linker-plugin to be able to check it. I apologize for the breakage. Honza * opts.c (finish_options): Imply -ffat-lto-objects with -fno-use-linker-plugin. * common.opt (fuse-linker-plugin): Add var. Index: opts.c

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

2013-11-20 Thread H.J. Lu
On Thu, Nov 14, 2013 at 12:28 PM, Diego Novillo dnovi...@google.com wrote: This patch applies the rule that functions defined in FOO.c must be declared in FOO.h. One of the worst offenders in the code base is tree.h, unsurprisingly. The first patch contains the actual moves from tree.h into

[patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Andrew MacLeod
This has been bugging me since I moved it out of gimple.h to gimplify.h. The gimplify context structure was exposed in the header file to allow a few other files to push and pop contexts off the gimplification stack. Unfortunately, the struct contains a hash-table template, which means it

[wide-int, committed] Remove remaining wide_int (x)s

2013-11-20 Thread Richard Sandiford
One simple switch to wi::add and one simplification that became possible after Kenny's patch to remove the other real_from_integer. Tested on x86_64-linux-gnu and applied as obvious (I hope). Thanks, Richard Index: gcc/objc/objc-act.c

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Andrew MacLeod
On 11/20/2013 08:35 AM, Andrew MacLeod wrote: Bootstraps on x86_64-unknown-linux-gnu with no new regressions. OK for mainline? Andrew And I've already fixed the typo in the changelog :-) I had it open in a window but hadn't saved it when I created the patch. Andrew *

Re: Add slim-lto support to gcc's build machinery

2013-11-20 Thread Paolo Bonzini
Il 20/11/2013 08:23, Markus Trippelsdorf ha scritto: Hi, now that slim-lto objects are enabled by default, it would be nice to use them when building gcc with bootstrap-lto. The following patch implements the handling of these objects in gcc's build machinery. (Once -fuse-linker-plugin is

[v3 patch] add missing 'template' keyword

2013-11-20 Thread Jonathan Wakely
2013-11-20 Jonathan Wakely jwakely@gmail.com PR c++/59173 * include/ext/pointer.h (pointer_traits::rebind): Add template keyword in nested name. Tested x86_64-linux, committed to trunk. commit 7359ca380f9e4ec5d1193e1efcc3aec5af95ec93 Author: Jonathan Wakely

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Jakub Jelinek
On Wed, Nov 20, 2013 at 08:35:28AM -0500, Andrew MacLeod wrote: I also hacked up the compiler to report what the 'top' of the stack was for a compilation unit. I then ran it through a bootstrap and full testsuite run of all languages, and looked for the maximum number of context structs in use

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

2013-11-20 Thread Diego Novillo
On Wed, Nov 20, 2013 at 6:44 AM, Eric Botcazou ebotca...@adacore.com wrote: Thanks. Committed as rev 205023. Nice work, but why did you antedate the entries in the various ChangeLog Oh, that's because of local commits and holding on to the patch for a few days. That date is the date of the

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

2013-11-20 Thread Ilya Enkovich
On 20 Nov 10:59, Richard Biener wrote: On Tue, Nov 19, 2013 at 9:29 PM, Ilya Enkovich enkovich@gmail.com wrote: On 19 Nov 12:33, Jeff Law wrote: On 11/19/13 05:13, Ilya Enkovich wrote: On 19 Nov 13:00, Richard Biener wrote: I'd say not in the gimplifier either but in varpool (symbol

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

2013-11-20 Thread Diego Novillo
On Wed, Nov 20, 2013 at 8:32 AM, H.J. Lu hjl.to...@gmail.com wrote: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59212 Thanks. Fixed. PR 59212 * g++.dg/plugin/selfassign.c: Include stringpool.h diff --git a/gcc/testsuite/g++.dg/plugin/selfassign.c

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Andrew MacLeod
On 11/20/2013 08:44 AM, Jakub Jelinek wrote: On Wed, Nov 20, 2013 at 08:35:28AM -0500, Andrew MacLeod wrote: I also hacked up the compiler to report what the 'top' of the stack was for a compilation unit. I then ran it through a bootstrap and full testsuite run of all languages, and looked for

Re: [patch] regcprop fix for PR rtl-optimization/54300

2013-11-20 Thread Richard Earnshaw
On 19/11/13 17:48, Jeff Law wrote: On 11/19/13 10:32, Steven Bosscher wrote: Yes. In the GCC3 days it was important for sincos on i386, and on mk68 it used to be important for some of the funnier patterns. Not sure if it's still useful today, though. Might be worth looking into, just to

[wide-int] Undo some differences with trunk

2013-11-20 Thread Richard Sandiford
I've committed a patch upstream to convert TREE_INT_CST_LOW to tree_to_[su]hwi if there is an obvious tree_fits_[su]hwi_p guard. There were some other changes from TREE_INT_CST_LOW to tree_to_[su]hwi that weren't as obvious and I think we should deal with them separately. As before, these

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

2013-11-20 Thread Richard Biener
On Wed, 20 Nov 2013, Sergey Ostanevich wrote: Thanks for comments, hope I got all of em. Note: I used a LOOP_VINFO_LOC (loop_vinfo) to print the loop location but it appears to be 0, so the output is somewhat lousy. The global vect_location points somewhere inside the loop, which is not that

[wide-int] Tweak an rs6000 test

2013-11-20 Thread Richard Sandiford
This test changes from TREE_INT_CST_LOW to TREE_INT_CST_ELT. I was going to change it back as part of the previous patch, but using wi:: seemed more robust. Only compile-tested so far because of problems with gcc110. OK for wide-int? Richard Index: gcc/config/rs6000/rs6000.c

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

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 2:12 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is version I commited - we need to intrdocue var for -fuse-linker-plugin to be able to check it. I apologize for the breakage. Honza * opts.c (finish_options): Imply -ffat-lto-objects with

Re: [wide-int] Undo some differences with trunk

2013-11-20 Thread Kenneth Zadeck
I would leave out the last frag of tree.c. It actually gets rid of what i consider, a latent bug. it does not show up with the current implementation of wide-int, but if that implementation changed, it would. The problem is that you really should not expect that you can set the min or max

Re: [wide-int] Tweak an rs6000 test

2013-11-20 Thread Kenneth Zadeck
looks fine to me. kenny On 11/20/2013 09:00 AM, Richard Sandiford wrote: This test changes from TREE_INT_CST_LOW to TREE_INT_CST_ELT. I was going to change it back as part of the previous patch, but using wi:: seemed more robust. Only compile-tested so far because of problems with gcc110.

Re: [PATCH, PR 57363] IBM long double: adding qNaN and number raises inexact exception

2013-11-20 Thread Adhemerval Zanella
Hi, This is a reformatted patch after Ulrich review: --- libgcc/ChangeLog: 2013-11-20 Adhemerval Zanella azane...@linux.vnet.ibm.com * config/rs6000/ibm-ldouble.c (__gcc_qadd): Fix add of normal number and qNaN to not raise an inexact exception. gcc/testsuite/ChangeLog:

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 2:35 PM, Andrew MacLeod amacl...@redhat.com wrote: This has been bugging me since I moved it out of gimple.h to gimplify.h. The gimplify context structure was exposed in the header file to allow a few other files to push and pop contexts off the gimplification stack.

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

2013-11-20 Thread Jakub Jelinek
On Wed, Nov 20, 2013 at 02:59:21PM +0100, Richard Biener wrote: --- a/gcc/c-family/c.opt +++ b/gcc/c-family/c.opt @@ -592,6 +592,10 @@ Wold-style-definition C ObjC Var(warn_old_style_definition) Warning Warn if an old-style parameter definition is used +Wopenmp-simd +C C++

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Jakub Jelinek
On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote: The limit looks reasonable, but you could have used a simple linked list (and never free). Also being able to pop a random context looks fragile ... that is, pop_gimplify_context shouldn't have an argument. Can't we use

Re: patch PLUGIN_HEADER_FILE event for tracing of header inclusions.

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 10:57 PM, Basile Starynkevitch bas...@starynkevitch.net wrote: On Tue, 2013-11-19 at 20:33 +0100, Dominique Dhumieres wrote: Thanks: Committed revision 205038. This seems to break several g++ tests: see http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg01482.html On

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote: The limit looks reasonable, but you could have used a simple linked list (and never free). Also being able to pop a random context looks fragile ... that is,

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

2013-11-20 Thread Jan Hubicka
On Wed, Nov 20, 2013 at 2:12 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is version I commited - we need to intrdocue var for -fuse-linker-plugin to be able to check it. I apologize for the breakage. Honza * opts.c (finish_options): Imply -ffat-lto-objects with

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Andrew MacLeod
On 11/20/2013 09:12 AM, Richard Biener wrote: On Wed, Nov 20, 2013 at 2:35 PM, Andrew MacLeod amacl...@redhat.com wrote: This has been bugging me since I moved it out of gimple.h to gimplify.h. The gimplify context structure was exposed in the header file to allow a few other files to push and

Re: [Patch, ARM] New feature to minimize the literal load for armv7-m target

2013-11-20 Thread Richard Earnshaw
On 06/11/13 06:10, Terry Guo wrote: Hi, This patch intends to minimize the use of literal pool for some armv7-m targets that have slower speed to load data from flash than to fetch instruction from flash. The normal literal load instruction is now replaced by MOVW/MOVT instructions. A new

[Patchv2] Add slim-lto support to gcc's build machinery

2013-11-20 Thread Markus Trippelsdorf
On 2013.11.20 at 14:41 +0100, Paolo Bonzini wrote: Il 20/11/2013 08:23, Markus Trippelsdorf ha scritto: Hi, now that slim-lto objects are enabled by default, it would be nice to use them when building gcc with bootstrap-lto. The following patch implements the handling of these objects

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

2013-11-20 Thread Sergey Ostanevich
Updated as per Richard and Jakub feedback - assuming the default for simd-cost-model is unlmited by default. Richard - was you Ok with it? Sergos * common.opt: Added new option -fsimd-cost-model. * tree-vectorizer.h (unlimited_cost_model): Interface update to rely on

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Trevor Saunders
On Wed, Nov 20, 2013 at 03:18:07PM +0100, Richard Biener wrote: On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote: The limit looks reasonable, but you could have used a simple linked list (and never free).

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

2013-11-20 Thread Trevor Saunders
On Wed, Nov 20, 2013 at 08:43:57AM -0500, Diego Novillo wrote: On Wed, Nov 20, 2013 at 6:44 AM, Eric Botcazou ebotca...@adacore.com wrote: Thanks. Committed as rev 205023. Nice work, but why did you antedate the entries in the various ChangeLog Oh, that's because of local commits and

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

2013-11-20 Thread Venkataramanan Kumar
Hi Joseph, On 19 November 2013 21:53, Joseph S. Myers jos...@codesourcery.com wrote: On Tue, 19 Nov 2013, Jakub Jelinek wrote: 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

Re: Committed: gdbhooks.py: reorganization to support regex-matching and typedefs

2013-11-20 Thread Tom Tromey
David == David Malcolm dmalc...@redhat.com writes: David I've committed the following to trunk as r205085, as a mixture of David enabling work towards gdb being able to print vec, and to fix issues David where the string-matching logic got confused by typedefs (done in a David rather crude way,

Re: [wide-int] Undo some differences with trunk

2013-11-20 Thread Richard Sandiford
Kenneth Zadeck zad...@naturalbridge.com writes: I would leave out the last frag of tree.c. It actually gets rid of what i consider, a latent bug. it does not show up with the current implementation of wide-int, but if that implementation changed, it would. The problem is that you really

[wide-int] Remove cst_fits_*

2013-11-20 Thread Richard Sandiford
FWIW this is the last change I had planned. I thought it would be easier to do once the host_integerp replacement was in mainline, but that means it's a bit closer to the deadline than it should have been. The branch adds two new functions, cst_fits_uhwi_p and cst_fits_shwi_p, alongside the

[PATCH, ia64] [PR target/52731] internal compiler error: in ia64_st_address_bypass_p, at config/ia64/ia64.c:9357

2013-11-20 Thread Kirill Yukhin
Hello, Patch in the bottom fixes PR52731. The essense of the problem is that `ia64_single_set' allows double set as exception for `prologue_allocate_stack' and `epilogue_deallocate_stack'. Although it does not apply this exception for predicated version of the patterns. I introduce explicit

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

2013-11-20 Thread Diego Novillo
On Wed, Nov 20, 2013 at 10:10 AM, Trevor Saunders tsaund...@mozilla.com wrote: On Wed, Nov 20, 2013 at 08:43:57AM -0500, Diego Novillo wrote: On Wed, Nov 20, 2013 at 6:44 AM, Eric Botcazou ebotca...@adacore.com wrote: Thanks. Committed as rev 205023. Nice work, but why did you antedate the

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

2013-11-20 Thread Richard Biener
On Tue, Nov 19, 2013 at 8:09 PM, Jeff Law l...@redhat.com wrote: On 11/19/13 07:06, Ilya Enkovich wrote: 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

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 3:34 PM, Andrew MacLeod amacl...@redhat.com wrote: On 11/20/2013 09:12 AM, Richard Biener wrote: On Wed, Nov 20, 2013 at 2:35 PM, Andrew MacLeod amacl...@redhat.com wrote: This has been bugging me since I moved it out of gimple.h to gimplify.h. The gimplify context

Re: [wwwdocs] Broken links

2013-11-20 Thread Philippe Baril Lecavalier
Embarrassing typos, my apologies. I was told to specify that I don't have commit access, but since this mention is irrelevant, I modified my suggested notice in about.html to reflect that. See attached.Index: projects/beginner.html

Re: [PATCH] [AArch64] Fix over length lines around aarch64_save_or_restore_fprs

2013-11-20 Thread Marcus Shawcroft
On 19/11/13 16:50, Kyrill Tkachov wrote: On 19/11/13 16:39, Marcus Shawcroft wrote: Committed. /Marcus 2013-11-19 Marcus Shawcroft marcus.shawcr...@arm.com * config/aarch64/aarch64.c (aarch64_save_or_restore_fprs): Fix over length lines. Minor nit... but ENOPATCH ;)

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Richard Biener
On Wed, Nov 20, 2013 at 4:06 PM, Trevor Saunders tsaund...@mozilla.com wrote: On Wed, Nov 20, 2013 at 03:18:07PM +0100, Richard Biener wrote: On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote: The limit looks

Re: [PATCH, i386, MPX, 2/X] Pointers Checker [21/25] Size relocation

2013-11-20 Thread Uros Bizjak
On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich enkovich@gmail.com wrote: Here is a patch to add size relocation and instruction to obtain object's size in i386 target. +(define_insn move_size_reloc_mode + [(set (match_operand:SWI48 0 register_operand =r) +

Re: [PATCH][AArch64] vneg testcase made big-endian safe

2013-11-20 Thread Marcus Shawcroft
On 19/11/13 17:29, Alex Velenko wrote: gcc/testsuite/ 2013-11-19 Alex Velenko alex.vele...@arm.com * gcc.target/aarch64/vneg_s.c (test_vneg_s8): fixed to not use vector indexing. (test_vneg_s16): Likewise. (test_vneg_s32): Likewise.

Re: [PATCH, i386, MPX, 2/X] Pointers Checker [21/25] Size relocation

2013-11-20 Thread Ilya Enkovich
2013/11/20 Uros Bizjak ubiz...@gmail.com: On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich enkovich@gmail.com wrote: Here is a patch to add size relocation and instruction to obtain object's size in i386 target. +(define_insn move_size_reloc_mode + [(set (match_operand:SWI48 0

Re: [PATCH, rs6000] Fix order of TDmode in FPRs in little-endian mode

2013-11-20 Thread David Edelsohn
On Mon, Nov 18, 2013 at 3:07 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Also note that this patch does not change how TDmode values are loaded into GPRs: on little-endian, this means we do get the usual LE subreg order there (least significant word in lowest-numbered register). This does

Re: [PATCH, PR 57363] IBM long double: adding qNaN and number raises inexact exception

2013-11-20 Thread David Edelsohn
On Wed, Nov 20, 2013 at 9:11 AM, Adhemerval Zanella azane...@linux.vnet.ibm.com wrote: libgcc/ChangeLog: 2013-11-20 Adhemerval Zanella azane...@linux.vnet.ibm.com * config/rs6000/ibm-ldouble.c (__gcc_qadd): Fix add of normal number and qNaN to not raise an inexact

[PATCH][ARM]Use of vcvt for float to fixed point conversions.

2013-11-20 Thread Renlin Li
Hi all, This patch will make the arm back-end use vcvt for float to fixed point conversions when applicable. Test on arm-none-linux-gnueabi has been done on the model. Okay for trunk? Kind regards, Renlin Li gcc/ChangeLog: 2013-11-20 Renlin Li renlin...@arm.com *

Re: [PATCH, i386, MPX, 2/X] Pointers Checker [21/25] Size relocation

2013-11-20 Thread Uros Bizjak
On Wed, Nov 20, 2013 at 5:11 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/20 Uros Bizjak ubiz...@gmail.com: On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich enkovich@gmail.com wrote: Here is a patch to add size relocation and instruction to obtain object's size in i386

Re: [PATCH, i386, MPX, 2/X] Pointers Checker [22/25] Target builtins

2013-11-20 Thread Uros Bizjak
Hello! Here is a patch introducing i386 target versions of Pointer Bounds Checker builtins. 2013-11-15 Ilya Enkovich ilya.enkov...@intel.com * config/i386/i386-builtin-types.def (BND): New. (ULONG): New. (BND_FTYPE_PCVOID_ULONG): New. (VOID_FTYPE_BND_PCVOID): New.

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Andrew MacLeod
On 11/20/2013 10:51 AM, Richard Biener wrote: On Wed, Nov 20, 2013 at 4:06 PM, Trevor Saunders tsaund...@mozilla.com wrote: On Wed, Nov 20, 2013 at 03:18:07PM +0100, Richard Biener wrote: On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek ja...@redhat.com wrote: On Wed, Nov 20, 2013 at 03:12:34PM

Re: [PATCH, i386, MPX, 2/X] Pointers Checker [21/25] Size relocation

2013-11-20 Thread Ilya Enkovich
2013/11/20 Uros Bizjak ubiz...@gmail.com: On Wed, Nov 20, 2013 at 5:11 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/20 Uros Bizjak ubiz...@gmail.com: On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich enkovich@gmail.com wrote: Here is a patch to add size relocation and

Re: [PATCH, rs6000] Fix order of TDmode in FPRs in little-endian mode

2013-11-20 Thread Ulrich Weigand
David Edelsohn wrote: On Mon, Nov 18, 2013 at 3:07 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Also note that this patch does not change how TDmode values are loaded into GPRs: on little-endian, this means we do get the usual LE subreg order there (least significant word in

Re: [PATCH, i386, MPX, 2/X] Pointers Checker [21/25] Size relocation

2013-11-20 Thread Ilya Enkovich
2013/11/20 Ilya Enkovich enkovich@gmail.com: 2013/11/20 Uros Bizjak ubiz...@gmail.com: On Wed, Nov 20, 2013 at 5:11 PM, Ilya Enkovich enkovich@gmail.com wrote: 2013/11/20 Uros Bizjak ubiz...@gmail.com: On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich enkovich@gmail.com wrote:

Re: [PATCH][1/3] Re-submission of Altera Nios II port, gcc parts

2013-11-20 Thread Chung-Lin Tang
On 13/11/20 1:34 AM, Richard Sandiford wrote: Chung-Lin Tang clt...@codesourcery.com writes: +;; Integer logical Operations + +(define_code_iterator LOGICAL [and ior xor]) +(define_code_attr logical_asm [(and and) (ior or) (xor xor)]) + +(define_insn codesi3 + [(set (match_operand:SI 0

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Andrew MacLeod
On 11/20/2013 11:30 AM, Andrew MacLeod wrote: On 11/20/2013 10:51 AM, Richard Biener wrote: On Wed, Nov 20, 2013 at 4:06 PM, Trevor Saunders tsaund...@mozilla.com wrote: On Wed, Nov 20, 2013 at 03:18:07PM +0100, Richard Biener wrote: On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Jeff Law
On 11/20/13 06:44, Jakub Jelinek wrote: On Wed, Nov 20, 2013 at 08:35:28AM -0500, Andrew MacLeod wrote: I also hacked up the compiler to report what the 'top' of the stack was for a compilation unit. I then ran it through a bootstrap and full testsuite run of all languages, and looked for the

Re: [patch] Privatize gimplify_ctx structure.

2013-11-20 Thread Jeff Law
On 11/20/13 06:56, Andrew MacLeod wrote: On 11/20/2013 08:44 AM, Jakub Jelinek wrote: On Wed, Nov 20, 2013 at 08:35:28AM -0500, Andrew MacLeod wrote: I also hacked up the compiler to report what the 'top' of the stack was for a compilation unit. I then ran it through a bootstrap and full

Fix PR target/59207

2013-11-20 Thread Eric Botcazou
This fixes an uninitialized variable introduced by the double_int rewrite. Tested on SPARC/Solaris, applied on the mainline and 4.8 branch. 2013-11-20 Eric Botcazou ebotca...@adacore.com PR target/59207 * config/sparc/sparc.c (sparc_fold_builtin) case CODE_FOR_pdist_vis:

  1   2   >