Re: [Patch, AArch64] Optimized implementation of vget_low_* in arm_neon.h.

2013-08-27 Thread Marcus Shawcroft
On 20 August 2013 12:21, Tejas Belagod tbela...@arm.com wrote: Hi, This patch replaces all inline asm implementations of vget_low_* in arm_neon.h with optimized implementations using other neon intrinsics. Tested with aarch64-none-elf. OK? This is OK. /Marcus

Re: [Patch,AArch64] Support SISD Shifts (SHL/USHR/SSHL/USHL/SSHR)

2013-08-27 Thread Marcus Shawcroft
On 20 August 2013 16:04, Vidya Praveen vidyaprav...@arm.com wrote: 2013-08-20 Vidya Praveen vidyaprav...@arm.com * config/aarch64/aarch64.md (unspec): Add UNSPEC_SISD_SSHL, UNSPEC_SISD_USHL, UNSPEC_USHL_2S, UNSPEC_SSHL_2S, UNSPEC_SISD_NEG. (optabmode3_insn): Remove.

Re: [RFC Patch, Aarch64] : Macros for profile code generation to enable gprof support

2013-08-27 Thread Marcus Shawcroft
Hi Venkat, On 3 August 2013 19:01, Venkataramanan Kumar venkataramanan.ku...@linaro.org wrote: This patch adds macros to support gprof in Aarch64. The difference from the previous patch is that the compiler, while generating mcount routine for an instrumented function, also passes the return

[PATCH, i386] PR 57927: -march=core-avx2 different than -march=native on INTEL Haswell (i7-4700K)

2013-08-27 Thread Christian Widmer
Hello! As reported in [1] the host processor detection has not yet been updated to recognize Intel Ivy Bridge and Haswell processors. This small patch adds the detection of these processors and assumes core-avx2 as march for unknown processors of the PENTIUMPRO family that support AVX2. Best

Re: [patch] Fix error recovery issue with alias

2013-08-27 Thread Richard Biener
On Tue, Jun 25, 2013 at 9:23 AM, Eric Botcazou ebotca...@adacore.com wrote: Hi, this fixes a segfault on a malformed alias declaration, which is correctly flagged as an error by handle_alias_pairs: error: 'Linker_Alias.Var' aliased to undefined symbol 'var2' but is nevertheless later

Re: Document __builtin_isinf_sign more precisely

2013-08-27 Thread Richard Biener
On Wed, Jun 26, 2013 at 4:16 PM, Marc Glisse marc.gli...@inria.fr wrote: Ping http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00475.html Ok. Thanks, Richard. On Sun, 9 Jun 2013, Marc Glisse wrote: Hello, this patch documents that __builtin_isinf_sign returns +-1 for +-Inf. This builtin was

Re: [PATCH] Improve jump threading using VRP information

2013-08-27 Thread Richard Biener
On Thu, Jun 27, 2013 at 6:29 AM, Jeff Law l...@redhat.com wrote: Just something else I saw while analyzing dumps from an unrelated set of changes. It's relatively common to see sequences like this: # parent_1 = PHI parent_7(3), parent_6(D)(4), parent_6(D)(5) _11 = single_tree_10(D) !=

Re: SSA identifiers

2013-08-27 Thread Richard Biener
On Thu, Jun 27, 2013 at 8:52 PM, Andrew MacLeod amacl...@redhat.com wrote: On 06/27/2013 02:39 PM, Jakub Jelinek wrote: in tree-ssanames.c:release_ssa_names() : if (! SSA_NAME_IN_FREE_LIST (var)) { tree saved_ssa_name_var = SSA_NAME_VAR (var); int saved_ssa_name_version

Re: [AArch64] Rewrite the vdup_lane intrinsics in C

2013-08-27 Thread Marcus Shawcroft
On 9 August 2013 10:48, James Greenhalgh james.greenha...@arm.com wrote: --- gcc/ 2013-08-09 James Greenhalgh james.greenha...@arm.com * config/aarch64/aarch64-simd-builtins.def (dup_lane_scalar): Remove. * config/aarch64/aarch64-simd.md

[committed] Add testcases from PR578{60,61,75,76,77}

2013-08-27 Thread Jakub Jelinek
Hi! These issues were caused by the LRA optional reloads support that got reverted later on, but the testcases look to be useful for the testsuite, so I went ahead and checked them all in. 2013-08-27 Jakub Jelinek ja...@redhat.com PR rtl-optimization/57860 PR

Re: [Patch, Fortran, OOP] PR 57305: ICE when calling SIZEOF on an unlimited polymorphic variable

2013-08-27 Thread Mikael Morin
Le 26/08/2013 16:16, Janus Weil a écrit : I'm slightly inclined to kindly invite the user to switch to STORAGE_SIZE+SIZE instead. Any other opinion? Since the SIZEOF intrinsic has been around for some time in gfortran (before STORAGE_SIZE was available), I would say we should at least

Re: [PATCH] Fix illegal cast to rtx (*insn_gen_fn) (rtx, ...)

2013-08-27 Thread Stefan Kristiansson
On Tue, Aug 27, 2013 at 11:03:32AM +0200, Richard Biener wrote: On Wed, Jul 10, 2013 at 3:14 AM, Stefan Kristiansson stefan.kristians...@saunalahti.fi wrote: The (static arg) generator functions are casted to a var arg function pointer, making the assumption that the ABI for passing the

Re: [PATCH 11/11] Make opt_pass and gcc::pipeline be GC-managed

2013-08-27 Thread Richard Biener
On Sat, Aug 3, 2013 at 2:21 AM, Richard Henderson r...@redhat.com wrote: On 08/02/2013 11:53 AM, David Malcolm wrote: FWIW I had a go at avoiding templates by attempting to tell gengtype to write out functions for all GTY((user)) types, regardless of whether it thinks they're referenced, with

Re: [PATCH] Fix PR57521

2013-08-27 Thread Richard Biener
On Mon, 24 Jun 2013, Richard Biener wrote: The following fixes a long-standing bug in tree if-conversion. The transform phase relies on being able to extract edge predicates by simply using the predicate under which its source block is executed. That obviously isn't the correct one if the

Re: Fix PR middle-end/57370

2013-08-27 Thread Richard Biener
On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman era...@google.com wrote: I have a new patch that supersedes this. The new patch also fixes PR tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and no test regression on x86_64/linux. Ok for trunk? 2013-07-31 Easwaran Raman

Re: [PATCH][4.8 backport] Fix PR57735

2013-08-27 Thread Richard Biener
On Mon, Jul 22, 2013 at 6:27 PM, Kyrylo Tkachov kyrylo.tkac...@arm.com wrote: Hi all, The fix for PR57735 is in current trunk (for a different issue I think), just needs a backport to 4.8. It is r198462 by Richard Sandiford: 2013-04-30 Richard Sandiford rsand...@linux.vnet.ibm.com

Re: [PATCH 1/2] Convert symtab, cgraph and varpool nodes into a real class hierarchy

2013-08-27 Thread Richard Biener
On Fri, Aug 16, 2013 at 2:57 AM, David Malcolm dmalc...@redhat.com wrote: This patch is the handwritten part of the conversion of these types to C++; it requires the followup patch, which is autogenerated. It converts: struct GTY(()) symtab_node_base to: class GTY((user))

Re: [PATCH 1/n] Add conditional compare support

2013-08-27 Thread Richard Biener
On Wed, Aug 21, 2013 at 12:23 PM, Zhenqiang Chen zhenqiang.c...@arm.com wrote: Hi, The attached patch is the basic support for conditional compare (CCMP). It adds a set of keywords on TREE to represent CCMP: DEFTREECODE (TRUTH_ANDIF_LT_EXPR, truth_andif_lt_expr, tcc_ccomparison, 3)

Re: [PATCH] Add a new option -ftree-bitfield-merge (patch / doc inside)

2013-08-27 Thread Richard Biener
On Tue, Jul 30, 2013 at 4:47 PM, Zoran Jovanovic zoran.jovano...@imgtec.com wrote: Thank you for the reply. I am in the process of modifying the patch according to some comments received. Currently I am considering the usage of DECL_BIT_FIELD_REPRESENTATIVE. I see that they can be used

Re: Stream TYPE_FINAL_P and DECL_FINAL_P to LTO

2013-08-27 Thread Richard Biener
On Sun, Aug 25, 2013 at 5:23 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this patch adds code to stream DECL_FINAL_P/TYPE_FINAL_P into LTO so we can use it after the ipa-devirt code at LTO time is merged in. (http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01007.html) Bootstrapped/regtested

Re: [PATCH 1/2] Convert symtab, cgraph and varpool nodes into a real class hierarchy

2013-08-27 Thread Jan Hubicka
+ while (x != xlimit) +{ + /* Code common to all symtab nodes. */ + gt_ggc_m_9tree_node (x-decl); + gt_ggc_mx_symtab_node_base (x-next); + gt_ggc_mx_symtab_node_base (x-previous); + gt_ggc_mx_symtab_node_base (x-next_sharing_asm_name); +

Re: [ubsan] Introduce pointer_sized_int_node

2013-08-27 Thread Richard Biener
On Mon, Aug 26, 2013 at 12:15 PM, Marek Polacek pola...@redhat.com wrote: I noticed I forgot to apply this old patch, already acked by Jason. It introduces new pointer_sized_int_node, thus we can get rid of uptr_type function in ubsan, and it allows us to do some clean-up in asan.c, too.

Re: [PATCH 1/n] Add conditional compare support

2013-08-27 Thread Richard Earnshaw
On 27/08/13 12:10, Richard Biener wrote: What's this for and what's the desired semantics? I don't like having extra tree codes for this. Is this for a specific instruction set feature? The background is to support the conditional compare instructions in ARM (more effectively) and AArch64 at

Re: [PATCH] Change the badness computation to ensure no integer-underflow

2013-08-27 Thread Richard Biener
On Tue, Jun 25, 2013 at 12:45 AM, Dehao Chen de...@google.com wrote: The original patch has some flaw. The new patch is attached. Bootstrapped and passed regression tests. Ok. Thanks, Richard. Thanks, Dehao On Mon, Jun 24, 2013 at 9:44 AM, Dehao Chen de...@google.com wrote: Hi, Richard,

Re: Stream TYPE_FINAL_P and DECL_FINAL_P to LTO

2013-08-27 Thread Jan Hubicka
+ v = iterative_hash_host_wide_int (TYPE_TRANSPARENT_AGGR (t), v); + v = iterative_hash_host_wide_int (TYPE_FINAL_P (t), v); please use | (TYPE_FINAL_P (t) 1) to speed this up like in other cases. Thank you, I missed that TYPE_TRANSPARENT_AGGR is also flag. I will update

Re: [PATCH 1/2] Convert symtab, cgraph and varpool nodes into a real class hierarchy

2013-08-27 Thread Richard Biener
On Tue, Aug 27, 2013 at 1:40 PM, Jan Hubicka hubi...@ucw.cz wrote: + while (x != xlimit) +{ + /* Code common to all symtab nodes. */ + gt_ggc_m_9tree_node (x-decl); + gt_ggc_mx_symtab_node_base (x-next); + gt_ggc_mx_symtab_node_base (x-previous); +

Re: [PATCH 1/n] Add conditional compare support

2013-08-27 Thread Richard Biener
On Tue, Aug 27, 2013 at 1:56 PM, Richard Earnshaw rearn...@arm.com wrote: On 27/08/13 12:10, Richard Biener wrote: What's this for and what's the desired semantics? I don't like having extra tree codes for this. Is this for a specific instruction set feature? The background is to support

[ubsan] Use pointer map instead of hash table.

2013-08-27 Thread Marek Polacek
It turned out that for tree - tree mapping we don't need the hash table at all; pointer map is much more convenient. So this patch weeds out the hash table out of ubsan and introduces pointer map instead. Quite a lot of code could go away--no need to set the alloc pools up etc. Regtested, ran

Re: [PATCH 1/2] Convert symtab, cgraph and varpool nodes into a real class hierarchy

2013-08-27 Thread Jan Hubicka
Also all of the symbol table is reachable from the global symbol_table dynamic array which is a GC root. So instead of walking -next/previous and edges you should have a custom marker for the symbol_table global which does more efficient marking with loops. Indeed, good point! All

Re: [PATCH] Change the badness computation to ensure no integer-underflow

2013-08-27 Thread Jan Hubicka
I've updated the patch (as attached) to use sreal to compute badness. + badness = ((int)((double) edge-count / max_count + * relbenefit / RELATIVE_TIME_BENEFIT_RANGE * INT_MIN / 2)) / growth; + FP operations on the host are frowned upon if code generation depends on their

Re: ELF interposition and One Definition Rule

2013-08-27 Thread Nathan Sidwell
On 08/26/13 20:58, Jason Merrill wrote: I would be happy with an even stronger default that optimizes on the assumption that no interposition occurs; typically interposition is overriding a symbol found in a dynamic library (i.e. malloc) rather than a symbol defined in the same translation unit

Re: [ubsan] Introduce pointer_sized_int_node

2013-08-27 Thread Marek Polacek
On Tue, Aug 27, 2013 at 01:48:29PM +0200, Richard Biener wrote: + TI_POINTER_SIZED_TYPE, I'd rather see TI_UINTPTR_TYPE and TI_INTPTR_TYPE (note they might not be exactly of POINTER_SIZE but larger). We already have [u]intptr_type_node -- but only in c-family/, thus ubsan.c/asan.c cannot

[Patch, Fortran, F03] PR 55603: Memory leak with scalar allocatable function result

2013-08-27 Thread Janus Weil
Hi all, here is a patch for PR 55603, which plugs a memory leak with scalar allocatable function results. To accomplish this, several things are done: 1) Allocatable scalar function results are passed as argument now and returned by reference (just like array or character results, cf.

[C++ Patch] PR 13981

2013-08-27 Thread Paolo Carlini
Hi, in this very old issue submitter requested some sort of hint in the diagnostic that the specific problem with the testcase has to do with class type B being incomplete. I think an inform can do. I feared that we would provide too many false positives, in the sense that the inform would

Re: [PING PATCH, PR 57748] Set mode of structures with zero sized arrays to be BLK

2013-08-27 Thread Martin Jambor
Hi, On Fri, Aug 23, 2013 at 05:29:23PM +0200, Jakub Jelinek wrote: On Fri, Aug 23, 2013 at 05:11:22PM +0200, Martin Jambor wrote: Hi Jakub and/or Joseph, the reporter of this bug seems to be very anxious to have it fixed in the repository. While Richi is away, do you think you could

Re: [PING PATCH, PR 57748] Set mode of structures with zero sized arrays to be BLK

2013-08-27 Thread Jakub Jelinek
On Tue, Aug 27, 2013 at 04:03:42PM +0200, Martin Jambor wrote: On Fri, Aug 23, 2013 at 05:29:23PM +0200, Jakub Jelinek wrote: On Fri, Aug 23, 2013 at 05:11:22PM +0200, Martin Jambor wrote: Hi Jakub and/or Joseph, the reporter of this bug seems to be very anxious to have it fixed in

Re: Request to merge Undefined Behavior Sanitizer in (take 3)

2013-08-27 Thread Marek Polacek
On Thu, Aug 22, 2013 at 09:01:57PM +0200, Marek Polacek wrote: On Thu, Aug 22, 2013 at 07:51:07PM +0200, Marek Polacek wrote: Ping. I'm withdrawing the ping for now. I'll have to deal with some bootstrap comparison failures first (ugh!). Fixed with this patch:

Re: [GOOGLE] Update AutoFDO annotation

2013-08-27 Thread Dehao Chen
Patch updated. Thanks, Dehao On Mon, Aug 26, 2013 at 4:11 PM, Xinliang David Li davi...@google.com wrote: Can you add missing documentation on functions like ...:get_count_info -- documenting return value etc. Also it might be better to avoid using 'set' as the local variable name. Change it

Re: [PATCH 1/2] Convert symtab, cgraph and varpool nodes into a real class hierarchy

2013-08-27 Thread Mike Stump
On Aug 27, 2013, at 4:08 AM, Richard Biener richard.guent...@gmail.com wrote: and converts: struct GTY(()) cgraph_node to: struct GTY((user)) cgraph_node : public symtab_node_base GTY didn't like single inheritance for me in in wide-int.h. I extended GTY to support it better. See the

Re: [GOOGLE] Update AutoFDO annotation

2013-08-27 Thread Xinliang David Li
Ok. David On Tue, Aug 27, 2013 at 7:36 AM, Dehao Chen de...@google.com wrote: Patch updated. Thanks, Dehao On Mon, Aug 26, 2013 at 4:11 PM, Xinliang David Li davi...@google.com wrote: Can you add missing documentation on functions like ...:get_count_info -- documenting return value etc.

Re: v3 of GDB hooks for debugging GCC

2013-08-27 Thread Jeff Law
On 08/26/2013 12:42 PM, David Malcolm wrote: The patch also adds me a maintainer of gdbhooks.py into the MAINTAINERS file. (There doesn't seem to be any sort order to the maintainer part of that file, should there be?) Finally, I added a copyright header to the new file (part of GCC, FSF

Re: v2 of GDB hooks for debugging GCC

2013-08-27 Thread Tom Tromey
David == David Malcolm dmalc...@redhat.com writes: David Is there a precanned event provided by gdb that I can connect to for David when the underlying code has changed and my caches need to be David invalidated? Maybe not :( You could use the exited event as a decent approximation. Also, and

Re: RFA: AVR: Support building AVR Linux targets

2013-08-27 Thread nick clifton
Hi Joseph, C. Draw up another patch that restricts the AVR patterns in config.gcc to -none and -elf. A and C - I think both changes should be applied. OK - the patch for item A is already applied. Here is a proposed patch for item C. I have not applied the patch as obvious because I

Re: [PING 3] [Patch RX] Add assembler option -mcu for generating assembler

2013-08-27 Thread nick clifton
Hi Sandeep, gas/config: 2013-07-18 Sandeep Kumar Singhsandeep.sin...@kpitcummins.com * rx.h: Add option -mcpu for target variants RX100 and RX200. Approved - please apply. Cheers Nick

Re: RFA: fix some avr stdint issues

2013-08-27 Thread Georg-Johann Lay
Joern Rennecke wrote: This patch fixes the gcc.dg/c99-stdint-5.c and gcc.dg/c99-stdint-6.c excess error failures. FYI, some of the problems with the c99-stdint tests are related to the stdint.h implementation in use. For example, some types in AVR-Libc's stdint.h are not defined in the way

Re: [PATCH 3/3] Support dumping type bindings in lambda diagnostics.

2013-08-27 Thread Jason Merrill
This patch doesn't seem to depend on the others; go ahead and apply it. Jason

Re: [PATCH, i386] PR 57927: -march=core-avx2 different than -march=native on INTEL Haswell (i7-4700K)

2013-08-27 Thread Uros Bizjak
Hello! As reported in [1] the host processor detection has not yet been updated to recognize Intel Ivy Bridge and Haswell processors. This small patch adds the detection of these processors and assumes core-avx2 as march for unknown processors of the PENTIUMPRO family that support AVX2. I

Re: [PATCH, i386] PR 57927: -march=core-avx2 different than -march=native on INTEL Haswell (i7-4700K)

2013-08-27 Thread H.J. Lu
On Tue, Aug 27, 2013 at 10:28 AM, Uros Bizjak ubiz...@gmail.com wrote: Hello! As reported in [1] the host processor detection has not yet been updated to recognize Intel Ivy Bridge and Haswell processors. This small patch adds the detection of these processors and assumes core-avx2 as march

Re: [PATCH, i386] PR 57927: -march=core-avx2 different than -march=native on INTEL Haswell (i7-4700K)

2013-08-27 Thread Uros Bizjak
On Tue, Aug 27, 2013 at 7:36 PM, H.J. Lu hjl.to...@gmail.com wrote: As reported in [1] the host processor detection has not yet been updated to recognize Intel Ivy Bridge and Haswell processors. This small patch adds the detection of these processors and assumes core-avx2 as march for unknown

wide-int branch updated.

2013-08-27 Thread Kenneth Zadeck
removed all knowledge of SHIFT_COUNT_TRUNCATED from wide-int both Richard Biener and Richard Sandiford had commented negatively about this. fixed bug with wide-int::fits_uhwi_p. kenny Index: gcc/fold-const.c === ---

Re: [PATCH] Convert more passes to new dump framework

2013-08-27 Thread Teresa Johnson
Ping #3. Thanks, Teresa On Mon, Aug 19, 2013 at 11:33 AM, Teresa Johnson tejohn...@google.com wrote: Ping. Thanks, Teresa On Mon, Aug 12, 2013 at 6:54 AM, Teresa Johnson tejohn...@google.com wrote: On Tue, Aug 6, 2013 at 10:23 PM, Teresa Johnson tejohn...@google.com wrote: On Tue, Aug 6,

[PING][PATCH] Fix PR58139 by correctly initializing reg_raw_mode[]

2013-08-27 Thread Peter Bergner
I'd like to ping the following patch which fixes a wrong code bug on powerpc64-linux due to a lost dependency within the scheduler: Fix PR58139 by correctly initializing reg_raw_mode[] http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00910.html H.J., can you please try and SPEC test this

Re: [PATCH] Convert more passes to new dump framework

2013-08-27 Thread Xinliang David Li
+ Honza On Tue, Aug 27, 2013 at 10:56 AM, Teresa Johnson tejohn...@google.com wrote: Ping #3. Thanks, Teresa On Mon, Aug 19, 2013 at 11:33 AM, Teresa Johnson tejohn...@google.com wrote: Ping. Thanks, Teresa On Mon, Aug 12, 2013 at 6:54 AM, Teresa Johnson tejohn...@google.com wrote: On

Re: [PATCH] Sanitize block partitioning under -freorder-blocks-and-partition

2013-08-27 Thread Teresa Johnson
On Mon, Aug 19, 2013 at 10:47 AM, Teresa Johnson tejohn...@google.com wrote: On Mon, Aug 19, 2013 at 8:09 AM, Jan Hubicka hubi...@ucw.cz wrote: Remember it isn't using dominance anymore. The latest patch was instead ensuring the most frequent path between hot blocks and the entry/exit are

Re: [PATCH i386 2/8] [AVX512] Add mask registers.

2013-08-27 Thread Kirill Yukhin
Hello Reichard, On 26 Aug 09:37, Richard Henderson wrote: On 08/26/2013 09:13 AM, Kirill Yukhin wrote: +(define_split + [(set (match_operand:SWI12 0 mask_reg_operand) + (any_logic:SWI12 (match_operand:SWI12 1 mask_reg_operand) +(match_operand:SWI12 2

Re: [PATCH, i386] PR 57927: -march=core-avx2 different than -march=native on INTEL Haswell (i7-4700K)

2013-08-27 Thread H.J. Lu
On Tue, Aug 27, 2013 at 10:41 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Aug 27, 2013 at 7:36 PM, H.J. Lu hjl.to...@gmail.com wrote: As reported in [1] the host processor detection has not yet been updated to recognize Intel Ivy Bridge and Haswell processors. This small patch adds the

Re: opt-info message change for vectorizer

2013-08-27 Thread Xinliang David Li
Does this one look ok? thanks, David On Thu, Aug 22, 2013 at 4:20 PM, Xinliang David Li davi...@google.com wrote: Hi, In this patch, loop alignment peeling and loop versioning transformation will be reported via -fopt-info by default. This will help vectorizer size tuning. It also enhances

Re: wide-int branch updated

2013-08-27 Thread Richard Sandiford
Kenneth Zadeck zad...@naturalbridge.com writes: fixed fits_uhwi_p. tested on x86-64. kenny Index: gcc/wide-int.h === --- gcc/wide-int.h(revision 201985) +++ gcc/wide-int.h(working copy) @@ -1650,7 +1650,7 @@

Re: opt-info message change for vectorizer

2013-08-27 Thread Teresa Johnson
My only concern is whether the dump messages will get too long with the full function name on the same line. The infrastructure that emits inform() notes ensures that the function name is printed before each block of messages related to that function (via an In function foo: type message), but I

Re: wide-int branch updated

2013-08-27 Thread Kenneth Zadeck
you are about an hour behind in reading your email. I had just committed a patch that is very close to this. On 08/27/2013 02:31 PM, Richard Sandiford wrote: Kenneth Zadeck zad...@naturalbridge.com writes: fixed fits_uhwi_p. tested on x86-64. kenny Index: gcc/wide-int.h

Re: [PATCH i386 3/8] [AVX512] [1/n] Add AVX-512 patterns: VF iterator extended.

2013-08-27 Thread Kirill Yukhin
Hello, This patch is still far too large. I think you should split it up based on every single mode iterator that you need to add or change. Problem is that some iterators are depend on each other, so patches are not going to be tiny. Here is 1st one. It extends VF iterator - biggest

Re: [PATCH i386 2/8] [AVX512] Add mask registers.

2013-08-27 Thread Kirill Yukhin
On 27 Aug 22:11, Kirill Yukhin wrote: Hello, I've while pasting the patch I've accidentally put extra brace. Pls Ignore it +(define_insn kxnormode + [(set (match_operand:SWI12 0 register_operand =r,!k) + (not:SWI12 + (xor:SWI12 + (match_operand:SWI12 1 register_operand

Re: opt-info message change for vectorizer

2013-08-27 Thread Xinliang David Li
yes -- the long unmangled names can be annoying -- that is why I chose to dump the short form of the function names -- combined with line numbers, it should be enough to get the full context. David On Tue, Aug 27, 2013 at 11:36 AM, Teresa Johnson tejohn...@google.com wrote: My only concern is

Re: PING: Re: [patch] implement simd loops in trunk (OMP_SIMD)

2013-08-27 Thread Aldy Hernandez
On 08/19/13 13:18, Aldy Hernandez wrote: Well, apparently this last revision was approved and I didn't even know about it :). Tested one last time against current trunk and committed as revision 202029. My apologies to Jakub for the merge problems he will inherit on the gomp-4_0-branch.

Re: [PATCH 3/3] Support dumping type bindings in lambda diagnostics.

2013-08-27 Thread Adam Butcher
Hi Jason, Was just about to compose a mail to gcc-patches... Been busy, then ill, now just about ready to submit a new set of diffs. On 27.08.2013 17:47, Jason Merrill wrote: This patch doesn't seem to depend on the others; go ahead and apply it. Okay. As it stands, it means that you

[PATCH 1/4] Support lambda templates.

2013-08-27 Thread Adam Butcher
* parser.c (cp_parser_lambda_declarator_opt): Accept template parameter list with std=c++1y or std=gnu++1y. (cp_parser_lambda_body): Don't call 'expand_or_defer_fn' for lambda call operator template to avoid adding template result to symbol table. * lambda.c

Re: Lambda templates and implicit function templates.

2013-08-27 Thread Adam Butcher
Hi Jason, Here's an updated patch set. The fully_implicit_function_template_p field has been moved into cp_parser and the other comments addressed. I've done some testing with parameter packs also. They work okay with the explicit template parameter syntax for lambdas. Unfortunately, due to

[PATCH 2/4] Don't generate lambda conversion op if arglist has parameter pack.

2013-08-27 Thread Adam Butcher
* lambda.c (maybe_add_lambda_conv_op): Optimize argvec building and early out if CALLOP contains a function parameter pack. --- gcc/cp/lambda.c | 60 ++--- 1 file changed, 44 insertions(+), 16 deletions(-) diff --git

[PATCH 3/4] Support dumping type bindings in lambda diagnostics.

2013-08-27 Thread Adam Butcher
* error.c (dump_function_decl): Use standard diagnostic flow to dump a lambda diagnostic, albeit without stating the function name or duplicating the parameter spec (which is dumped as part of the type). --- gcc/cp/error.c | 22 +++--- 1 file changed, 11

[PATCH 4/4] Support using 'auto' in a function parameter list to introduce an implicit template parameter.

2013-08-27 Thread Adam Butcher
* cp-tree.h (type_uses_auto_or_concept): Declare. (is_auto_or_concept): Declare. * decl.c (grokdeclarator): Allow 'auto' parameters with -std=gnu++1y or -std=c++1y. * type-utils.h: New header defining ... (find_type_usage): ... this new template

Re: [PATCH i386 2/8] [AVX512] Add mask registers.

2013-08-27 Thread Richard Henderson
On 08/27/2013 11:11 AM, Kirill Yukhin wrote: What happened to the bmi andn alternative we discussed? BMI only supported for 4- and 8- byte integers, while kandw - for HI/QI We're talking about values in registers. Ignoring the high bits of the andn result still produces the correct

[wide-int] fix mips build error

2013-08-27 Thread Mike Stump
Since we use these in a cpp #if, we have to use #define for them. Index: wide-int.h === --- wide-int.h (revision 202032) +++ wide-int.h (working copy) @@ -249,15 +249,15 @@ along with GCC; see the file COPYING3. on any

Re: opt-info message change for vectorizer

2013-08-27 Thread Mike Stump
On Aug 27, 2013, at 11:22 AM, Xinliang David Li davi...@google.com wrote: Does this one look ok? We don't capitalize text after error:, warning: or note:. thanks, David On Thu, Aug 22, 2013 at 4:20 PM, Xinliang David Li davi...@google.com wrote: Hi, In this patch, loop alignment peeling

wide-int branch: cleaned up comparison functions.

2013-08-27 Thread Kenneth Zadeck
Removed the redundant implementations of several comparison function by just forwarding the oo version to the static version. Added static versions of cmp, cmpu and cmps. kenny Index: gcc/wide-int.h === --- gcc/wide-int.h

Re: opt-info message change for vectorizer

2013-08-27 Thread Xinliang David Li
If this is the convention, we should probably have another patch to fix all the existing opt-info messages. thanks, David On Tue, Aug 27, 2013 at 1:23 PM, Mike Stump mikest...@comcast.net wrote: On Aug 27, 2013, at 11:22 AM, Xinliang David Li davi...@google.com wrote: Does this one look ok?

[wwwdocs] Use https for twitter.com link

2013-08-27 Thread Gerald Pfeifer
No cookie, Gerald. I should have caught this permanent redirect (verified with wget among others). Change applied. Gerald Index: style.mhtml === RCS file: /cvs/gcc/wwwdocs/htdocs/style.mhtml,v retrieving revision 1.121 diff -u -3

Re: RFA: Consider int and same-size short as equivalent vector components

2013-08-27 Thread Joern Rennecke
Quoting Marc Glisse marc.gli...@inria.fr: On Mon, 26 Aug 2013, Joern Rennecke wrote: Quoting Marc Glisse marc.gli...@inria.fr: The issue seems larger than just short/int. On x86, (ll)l fails to compile for a vector of long, with ll that has opaque type vector of int, that seems wrong. I

Re: RFA: Consider int and same-size short as equivalent vector components

2013-08-27 Thread Paolo Carlini
Hi, On 08/28/2013 12:29 AM, Joern Rennecke wrote: * c-common.h: (same_scalar_type_ignoring_signedness): Delete prototype. (vector_types_compatible_elements_p): Prototype. Sorry for nitpicking, but since we are now using C++ I think declaration is more correct than prototype.

Re: [PATCH 3/3] Support dumping type bindings in lambda diagnostics.

2013-08-27 Thread Jason Merrill
On 08/27/2013 02:46 PM, Adam Butcher wrote: Okay. As it stands, it means that you get an additional 'const' in diagnostics for lambda's not declared 'mutable'. Hmm, I guess it would be preferable to use 'mutable' or nothing when printing the lambda just like when declaring one. Jason

Re: v3 of GDB hooks for debugging GCC

2013-08-27 Thread David Malcolm
On Tue, 2013-08-27 at 09:58 -0600, Jeff Law wrote: On 08/26/2013 12:42 PM, David Malcolm wrote: The patch also adds me a maintainer of gdbhooks.py into the MAINTAINERS file. (There doesn't seem to be any sort order to the maintainer part of that file, should there be?) Finally, I added

Re: [PATCH, i386] PR 57927: -march=core-avx2 different than -march=native on INTEL Haswell (i7-4700K)

2013-08-27 Thread Ryan Hill
On Tue, 27 Aug 2013 19:41:09 +0200 Uros Bizjak ubiz...@gmail.com wrote: On Tue, Aug 27, 2013 at 7:36 PM, H.J. Lu hjl.to...@gmail.com wrote: As reported in [1] the host processor detection has not yet been updated to recognize Intel Ivy Bridge and Haswell processors. This small patch adds

COMDAT missing profile handling (was Re: [PATCH] Sanitize block partitioning under -freorder-blocks-and-partition)

2013-08-27 Thread Teresa Johnson
On Wed, Aug 21, 2013 at 8:25 AM, Jan Hubicka hubi...@ucw.cz wrote: Because offline COMDAT functoin will be porduced for every COMDAT used, I think it is bad to porduce any COMDAT (or any reachable function via calls with non-0 count) that has empty profile (either because it got lost

Re: [PATCH][2/n] 2nd try: Re-organize -fvect-cost-model, enable basic vectorization at -O2

2013-08-27 Thread Xinliang David Li
Richard, I have some comments about the patch. -ftree-vectorizer-verbose=numberThis switch is deprecated. Use -fopt-info instead. ftree-slp-vectorize ! Common Report Var(flag_tree_slp_vectorize) Optimization Enable basic block vectorization (SLP) on trees The code dealing with