Re: [PATCH] MIPS16 TLS support for GCC

2012-07-06 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: (3) Also related to libraries, I edited CRT_CALL_STATIC_FUNCTION to emit a 32-bit code sequence under both MIPS/MIPS16 mode (under O32). As you can see in the original Feb. patch, I had changes to emit a MIPS16 version of these static

Re: [patch][RFC] Remove tree/gimple/rtl dependencies from core CFG code

2012-07-06 Thread Steven Bosscher
On Fri, Jul 6, 2012 at 1:06 AM, Joseph S. Myers jos...@codesourcery.com wrote: My only comment is that you add a coretypes.h include to tree-flow.h and remove one from tree-ssa-alias.h - but since the GCC convention is that all .c files start with includes of config.h, system.h and coretypes.h

Re: [PATCH] MIPS16 TLS support for GCC

2012-07-06 Thread Chung-Lin Tang
On 2012/7/6 02:23 PM, Richard Sandiford wrote: Richard Sandiford rdsandif...@googlemail.com writes: (3) Also related to libraries, I edited CRT_CALL_STATIC_FUNCTION to emit a 32-bit code sequence under both MIPS/MIPS16 mode (under O32). As you can see in the original Feb. patch, I had changes

Re: [patch] Fix debug info of nested inline functions

2012-07-06 Thread Eric Botcazou
Revised patch attached.  It still generates the same (fixed) debug info for the reduced testcase.  I'll do a full testing cycle if you're happy with it.       * dwarf2out.c (function_possibly_abstracted_p): New static function. (gen_subprogram_die): Use it

[PATCH, Android] Runtime stack protector enabling for Android target

2012-07-06 Thread Igor Zamyatin
Hi! For runtime stack protector enabling on x86 for Android we have to disable current glibc-based implementation and turn on default one since bionic doesn't use value from gs:0x14. Tested in android environment(x86_64-*-linux-android), also bootstrapped and regtested on

Re: [ARM Patch 3/3]PR53189: optimizations of 64bit logic operation with constant

2012-07-06 Thread Carrot Wei
Hi I updated this patch to use const_ok_for_dimode_op to decide if a const is legal for iordi3 insn. The special cases of all 0 and all 1 constant are also handled. Tested on arm qemu with all thumb/arm neon/non-neon combination, no regression. thanks Carrot 2012-07-06 Wei Guozhi

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-07-06 Thread Richard Guenther
On Thu, Jun 14, 2012 at 10:13 PM, Sriraman Tallam tmsri...@google.com wrote: +cc c++ front-end maintainers Hi, C++ Frontend maintainers, Could you please take a look at the front-end part when you find the time? So you have (for now?) omitted the C frontend change(s)? Honza, your

Re: [patch][RFC] Remove tree/gimple/rtl dependencies from core CFG code

2012-07-06 Thread Richard Guenther
On Thu, Jul 5, 2012 at 11:53 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, This is a first stab at making the core CFG code independent of the expression intermediate representation datatypes. This patch deals with cfg.c and cfganal.c but it shouldn't be a surprise that the patch

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-06 Thread Richard Guenther
On Thu, 5 Jul 2012, Magnus Fromreide wrote: On Thu, Jul 05, 2012 at 11:51:19AM +0200, Richard Guenther wrote: On Thu, 5 Jul 2012, Richard Guenther wrote: On Thu, 5 Jul 2012, Richard Guenther wrote: On Thu, 5 Jul 2012, Tristan Gingold wrote: On Jul 4, 2012, at 8:31

Re: RFA: MN10300: Call default_encode_section_info

2012-07-06 Thread nick clifton
Hi Jeff, Thanks - patch committed. I decided that it would be best to call default_encode_section_info first, just in case it turns out to be important one day. Cheers Nick

[PATCH, i386]: Make construct_plt_address static

2012-07-06 Thread Uros Bizjak
Hello! 2012-07-06 Uros Bizjak ubiz...@gmail.com * config/i386/i386.c (construct_plt_address): Make static. * config/i386/i386-protos.h (construct_plt_address): Remove. Tested on x86_64-pc-linux-gnu, committed to mainline SVN. Uros. Index: i386-protos.h

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-06 Thread Richard Guenther
On Thu, Jul 5, 2012 at 8:45 PM, Tom de Vries tom_devr...@mentor.com wrote: On 05/07/12 15:30, Michael Matz wrote: Hi, On Thu, 5 Jul 2012, Tom de Vries wrote: The asserts allow the return result to be optimized, but not the cfg conditions. AFAIU, we can insert the asserts earlier. F.i., we

Re: [PR49888, VTA] don't keep VALUEs bound to modified MEMs

2012-07-06 Thread Alexandre Oliva
On Jun 21, 2012, Alexandre Oliva aol...@redhat.com wrote: Here's one more patch that addresses a problem I found out while investigating the PR53671 regressions: rather than recording incoming stack args as MEMs with non-VALUE expressions, it's more consistent (and less surprising) if we emit

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-06 Thread Richard Guenther
On Fri, 6 Jul 2012, Richard Guenther wrote: Yes, I think that would be the most clean solution - unfortunately the current setup of the toplevel hierarchy does not support this (easily, at least). A way out would be to always claim stuff is installed - thus, use the =system variants and add

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-06 Thread Richard Guenther
On Fri, 6 Jul 2012, Richard Guenther wrote: On Fri, 6 Jul 2012, Richard Guenther wrote: Yes, I think that would be the most clean solution - unfortunately the current setup of the toplevel hierarchy does not support this (easily, at least). A way out would be to always claim stuff is

Re: [PATCH, Android] Runtime stack protector enabling for Android target

2012-07-06 Thread Igor Zamyatin
Right, flag_android looks better, thanks. Probably it's even better to check also bionic (OPTION_BIONIC) On Fri, Jul 6, 2012 at 12:13 PM, Andrew Pinski pins...@gmail.com wrote: On Fri, Jul 6, 2012 at 12:49 AM, Igor Zamyatin izamya...@gmail.com wrote: Hi! For runtime stack protector enabling

refine libstdc++ configure check on mutex_timedlock

2012-07-06 Thread Olivier Hainque
Hello, On a gcc 4.7 tree, we are witnessing build failures on Aix 5.2 with libstdc++ failing on ...pthread/libstdc++-v3/include/powerpc-ibm-aix5.2.0.0/bits/gthr-posix.h:789:69: error: 'pthread_mutex_timedlock' was not declared in this scope The anomaly is visible from the libstdc++

Re: refine libstdc++ configure check on mutex_timedlock

2012-07-06 Thread Olivier Hainque
On Jul 6, 2012, at 15:04 , David Edelsohn wrote: libstdc++-v3/ * acinclude.m4 (check for mutex_timelock): Check straight on _POSIX_TIMEOUTS, regardless of defined (_PTHREADS). * configure: Regenerate. It's okay with me, but I do not know who can approve it. Yes,

[PATCH] Fix undefined behaviour in combine.c:force_to_mode

2012-07-06 Thread Ulrich Weigand
Hello, in testing a patch on arm-linux-gnueabihf, I ran into a bootstrap comparison failure that turned out to be caused by a pre-existing bug in common code, where combine.c code exposed undefined behaviour. The problem is this code in force_to_mode when simplifying a LSHIFTRT: /* Here

C++ PATCH for c++/53858 (parse error with alias template)

2012-07-06 Thread Jason Merrill
Since my fix for 53563, we were ignoring alias templates when looking for a type template name. Fixed thus. Tested x86_64-pc-linux-gnu, applying to trunk. commit d46136fac2f8b4b126d498031b4bb02a23d5738a Author: Jason Merrill ja...@redhat.com Date: Fri Jul 6 09:21:51 2012 -0400 PR

Re: [gimplefe] Fixing the bug for gimple_assign statement with ternary operands

2012-07-06 Thread Diego Novillo
On 12-07-06 00:38 , Sandeep Soni wrote: I am halfway through the patch for building gimple_cond statements. I will be able to complete the patch over the weekend. I am also working towards a patch that generalizes the assignment statements considering all possible types of assignments. Great!

[v3] libstdc++/53872

2012-07-06 Thread Paolo Carlini
Hi, tested x86_64-linux, committed mainline and 4_7-branch. Thanks, Paolo. 2012-07-06 Paolo Carlini paolo.carl...@oracle.com PR libstdc++/53872 * include/std/thread (thread::_M_make_routine): Qualify make_shared to prevent ADL. *

Re: [PATCH]: Fix IPA with profiling

2012-07-06 Thread H.J. Lu
On Wed, Jul 4, 2012 at 6:22 AM, Uros Bizjak ubiz...@gmail.com wrote: Hello! Attached patch partially reverts r187375 [1]. Apparently, the patch added some new functionality to symtab_remove_unreachable_nodes, although the ChangeLog said: Log: ...

Re: [PATCH, i386]: Convert some more simple LEAs to ADD.

2012-07-06 Thread Andi Kleen
Uros Bizjak ubiz...@gmail.com writes: Hello! Sometimes, gcc generates: leaq(%rbx,%rax), %rax that is in fact equivalent (modulo flags reg clobber) to: addq%rbx, %rax Attached patch adds additional peephole2 patterns that convert LEA to ADD when second operand of PLUS RTX

[PATCH, AArch64] Allow symbol+offset as symbolic constant expression

2012-07-06 Thread Ian Bolton
Hi, This patch reduces codesize for cases such as this one: int arr[100]; int foo () { return arr[10]; } Before the patch, the code looked like this: adrp x0, arr add x0, x0, :lo12:arr ldr w0, [x0,40] Now, it looks like this: adrp x0, arr+40 ldr w0, [x0,#:lo12:arr+40] Some workloads

C++ PATCH for c++/53862 (rejects-valid variadic templates)

2012-07-06 Thread Jason Merrill
tsubst doesn't handle pack expansions directly; any place where a pack expansion is valid needs to test for them and call tsubst_pack_expansion. Here, the code for testing whether a non-deduced template argument is relevant to the argument types was failing to do so. Conveniently, we already

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-06 Thread Tom de Vries
On 06/07/12 13:01, Richard Guenther wrote: On Thu, Jul 5, 2012 at 8:45 PM, Tom de Vries tom_devr...@mentor.com wrote: On 05/07/12 15:30, Michael Matz wrote: Hi, On Thu, 5 Jul 2012, Tom de Vries wrote: The asserts allow the return result to be optimized, but not the cfg conditions. AFAIU,

Re: [PATCH] Fix undefined behaviour in combine.c:force_to_mode

2012-07-06 Thread Eric Botcazou
Note that most other places in force_to_mode optimizing shifts already check for non-negative shift amounts; but in this place the check is missing. The following patch adds the check here as well, fixing the undefined behaviour (and subsequent bootstrap comparison failure) in my test.

Re: User directed Function Multiversioning via Function Overloading (issue5752064)

2012-07-06 Thread Sriraman Tallam
On Fri, Jul 6, 2012 at 2:14 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Jun 14, 2012 at 10:13 PM, Sriraman Tallam tmsri...@google.com wrote: +cc c++ front-end maintainers Hi, C++ Frontend maintainers, Could you please take a look at the front-end part when you

[testsuite] backport cleanups to 4.7

2012-07-06 Thread Janis Johnson
Between June 12 and June 28 I checked in lots of GCC testsuite patches to individual tests and to test infrastrure files to eliminate non-unique lines in test summaries, as well as a few related cleanups. May I backport those to GCC 4.7? Janis

Re: [Patch ARM] Improve vdup_n intrinsics.

2012-07-06 Thread Ramana Radhakrishnan
On 27 June 2012 21:27, Richard Henderson r...@redhat.com wrote: On 06/20/2012 05:44 AM, Ramana Radhakrishnan wrote: + case NEON_DUP: + if (TREE_CODE (argp[0]) == INTEGER_CST + || TREE_CODE (argp[0]) == REAL_CST) + return build_vector_from_val

[lra] patch for lo_sum support

2012-07-06 Thread Vladimir Makarov
Working on LRA regression on ppc after the merge, I found that lo_sum support improves code size significantly. Here the patch to support lo_sum. The patch was successfully bootstrapped on x86/x86-64 and ppc. Committed as rev. 189333. 2012-07-05 Vladimir Makarov vmaka...@redhat.com

Re: [Patch, Fortran] Add parsing support for assumed-rank array

2012-07-06 Thread Tobias Burnus
Hi Mikael, hi all, Mikael Morin wrote: index f135af1..6c58a8e 100644 --- a/gcc/fortran/trans-array.c +++ b/gcc/fortran/trans-array.c @@ -8319,12 +8323,15 @@ gfc_walk_array_ref (gfc_ss * ss, gfc_expr * expr, gfc_ref * ref) break; case AR_FULL: - newss = gfc_get_array_ss (ss,

Re: [PATCH, i386]: Convert some more simple LEAs to ADD.

2012-07-06 Thread Uros Bizjak
On Fri, Jul 6, 2012 at 5:24 PM, Andi Kleen a...@firstfloor.org wrote: Uros Bizjak ubiz...@gmail.com writes: Sometimes, gcc generates: leaq(%rbx,%rax), %rax that is in fact equivalent (modulo flags reg clobber) to: addq%rbx, %rax Attached patch adds additional peephole2 patterns

Re: [PATCH] Clarify cloog install docs

2012-07-06 Thread Matthias Klose
On 03.07.2012 15:37, Richard Guenther wrote: People are running into issues when mixing cloog built against isl 0.08 and gcc building against isl 0.10 (which is expected I guess). CLooG needs to be configured to use GMP +internally, use @option{--with-bits=gmp} to direct it to do that. why

PR middle-end/53321: [4.8 Regression] LTO bootstrap failed with bootstrap-profiled

2012-07-06 Thread H.J. Lu
Hi, After inlining, IPA calls tree_profiling which rebuilds edges: #2 0x00683ccd in rebuild_cgraph_edges () at /export/gnu/import/git/gcc/gcc/cgraphbuild.c:433 #3 0x00c02de6 in tree_profiling () at /export/gnu/import/git/gcc/gcc/tree-profile.c:564 #4 0x00a1be4f