Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-11 Thread Martin Jambor
Hi, On Fri, Aug 10, 2012 at 04:39:44PM +0200, Martin Jambor wrote: On Fri, Aug 10, 2012 at 05:12:31AM +0200, Jan Hubicka wrote: Hi, ... 2012-07-31 Martin Jambor mjam...@suse.cz PR fortran/48636 * ipa-inline.h (condition): New fields offset, agg_contents and

[PATCH, i386]: Fix a couple of XOP issues

2012-08-11 Thread Uros Bizjak
Hello! There is no point to use nonimmediate_operand predicate, if the insn accepts only register operands. Also, the patch fixes wrong selectors for horizontal add insn. 2012-08-11 Uros Bizjak ubiz...@gmail.com * config/i386/sse.md (xop integer multiply/add insns): Use

Re: combine permutations in gimple

2012-08-11 Thread Marc Glisse
On Fri, 10 Aug 2012, Marc Glisse wrote: this patch detects permutations of permutations and merges them. It also canonicalizes permutations a bit more. There are several issues with this patch: 1) I am not sure we always want to combine permutations. Indeed, someone (user? vectorizer?) may

Re: Value type of map need not be default copyable

2012-08-11 Thread François Dumont
Here is an other attempt. I took the time to refactor the hashtable implementation. I prefer to rename _M_insert_node into _M_insert_unique_node and use it also into _M_emplace implementation. I introduce _M_insert_multi_node that is used in _M_insert and _M_emplace when keys are not

Re: Value type of map need not be default copyable

2012-08-11 Thread Marc Glisse
On Sat, 11 Aug 2012, François Dumont wrote: Your remark on using std::move rather than std::forward Marc made sens but didn't work. I don't understand why but the new test is showing that std::forward works. If anyone can explain why std::move doesn't work I am interested. What testcase

Re: Merge varpool and cgraph encoders into symtab encoders

2012-08-11 Thread H.J. Lu
On Fri, Aug 10, 2012 at 8:56 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this patch does the boring job of merging varpool and cgraph encoders into single symtab encoder. This has some snowballing effect. Because the IDs of cgraphs and varpools now use single space, the cgraph and varpool

Re: Reload/i386 patch for secondary memory vs subregs

2012-08-11 Thread John David Anglin
* reload1.c (replaced_subreg): New static function. (gen_reload): Use it when deciding whether to use secondary memory. This causes the following on hppa*-*-* (32-bit): ../../gcc/gcc/reload1.c: In function 'rtx_def* gen_reload(rtx, rtx, int, reload_ type)':

[PATCH, alpha]: Variable arguments handling broken by r190229

2012-08-11 Thread Uros Bizjak
Hello! We have to apply the same DECL_UID change as in tree-stdarg.c to alpha_stdarg_optimize_hook. 2012-08-11 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.c (alpha_stdarg_optimize_hook): Shift DECL_UID in the va_list_vars bitmap by num_ssa_names. Tested on

Unbreak lto plugin

2012-08-11 Thread Jan Hubicka
Hi, my prevoius LTO cleanup run into problem with plugin enabled builds; lto-plugin is stupid enought to confused symtab_nodes section with symtab because it ignores the suffixes. Fixed thus. Comitted as obvoius. Honza Index: ChangeLog

Re: Merge varpool and cgraph encoders into symtab encoders

2012-08-11 Thread Jan Hubicka
This caused: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54229 It was the other patch, but I just fixed it. See my other mail. Honza -- H.J.

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-11 Thread Jan Hubicka
This is the patch I have committed after resolving a conflict (and another round of bootstrapping and testing). The ChangeLog is still the same. The previous version of the patch (but with tree code already converted to an enum), the aggregate jump functions together with enlarged

Re: complex.h

2012-08-11 Thread Paolo Carlini
On 08/10/2012 09:17 PM, Marc Glisse wrote: Ping http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01440.html I would say, let's give Gaby and Benjamin another couple of days, then Marc is welcome to commit the patch. And, after all the help he gave us to sort out the Sparc issues vs c++11 we can

Re: Reload/i386 patch for secondary memory vs subregs

2012-08-11 Thread Bernd Schmidt
On 08/11/2012 04:20 PM, John David Anglin wrote: * reload1.c (replaced_subreg): New static function. (gen_reload): Use it when deciding whether to use secondary memory. This causes the following on hppa*-*-* (32-bit): ../../gcc/gcc/reload1.c: In function 'rtx_def*

Re: complex.h

2012-08-11 Thread Marc Glisse
On Sat, 11 Aug 2012, Paolo Carlini wrote: On 08/10/2012 09:17 PM, Marc Glisse wrote: Ping http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01440.html I would say, let's give Gaby and Benjamin another couple of days, then Marc is welcome to commit the patch. And, after all the help he gave us to

Re: Unbreak lto plugin

2012-08-11 Thread Richard Guenther
On Sat, Aug 11, 2012 at 5:51 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, my prevoius LTO cleanup run into problem with plugin enabled builds; lto-plugin is stupid enought to confused symtab_nodes section with symtab because it ignores the suffixes. Err, I suppose we should fix the plugin as

[PATCH] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

2012-08-11 Thread Senthil Kumar Selvaraj
This patch allows cflags set in board config files using set_board_info cflags to be used in the selectors of dg-skip-if and other dejagnu commands that use the check-flags proc. The code merely adds cflags to compiler_flags in the check-flags proc, exactly the same way as multilib_flags is

Re: complex.h

2012-08-11 Thread Paolo Carlini
Marc Glisse marc.gli...@inria.fr ha scritto: To be honest, I only checked the patch on linux/glibc, so there is a real risk on other platforms (which I don't have access to). I also did a quick sanity check on freebsd (not a true test). Ok, conditioning the small change on glibc would not be a

PR 54193: raw gimple dump of vec_perm_expr

2012-08-11 Thread Marc Glisse
Hello, I'll have to retest this patch tomorrow (although I don't expect the modified code is ever called), for some reason the testsuite took twice as long as usual to run and showed some weird stuff tonight. There doesn't seem to be any test calling -fdump-tree-*-raw, so I didn't add any.

[PATCH][Cilkplus] Fixes builtin Array functions

2012-08-11 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilk Plus branch affecting mainly the C++ compiler. This patch fixes a crash when builtin array notation functions are used as part of a modify expression. Thanks, Balaji V. Iyer. Index: gcc/cp/cp-array-notation.c

Re: [PATCH] PR 53528 c++/ C++11 Generalized Attribute support

2012-08-11 Thread Jason Merrill
On 08/10/2012 04:04 PM, Dodji Seketeli wrote: In cp_parser_decl_specifier_seq, I first tried to apply the c++11 attribute to the (already constructed) type it follows, like what you suggest. But then I am getting the warning: warning: ignoring attributes applied to 'A' after definition

Re: Unbreak lto plugin

2012-08-11 Thread Jan Hubicka
On Sat, Aug 11, 2012 at 5:51 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, my prevoius LTO cleanup run into problem with plugin enabled builds; lto-plugin is stupid enought to confused symtab_nodes section with symtab because it ignores the suffixes. Err, I suppose we should fix the plugin