Re: [PATCH 3/3] Add a fragmentation fallback in ggc-page

2011-10-21 Thread Jakub Jelinek
On Fri, Oct 21, 2011 at 07:52:50AM +0200, Andi Kleen wrote: @@ -776,13 +778,18 @@ alloc_page (unsigned order) extras on the freelist. (Can only do this optimization with mmap for backing store.) */ struct page_entry *e, *f = G.free_pages; - int i; + int i,

[PATCH, i386]: Improve recip sequences a bit

2011-10-21 Thread Uros Bizjak
Hello! While eyeballing following testcase: float a[256], b[256], c[256]; void foo(void) { int i; for (i=0; i256; ++i) c[i] = a[i] / b[i]; } -O2 -ftree-vectorize -ffast-math I noticed that for some reason CSE doesn't eliminate memory read, resulting in: .L2: vrcpps b(%rax),

[C++ Patch] PR 30066

2011-10-21 Thread Roberto Agostino Vitillo
With this patch fvisibility-inlines-hidden affects also inline functions, e.g.: inline void foo() {} int main(){ foo(); return 0; } when compiled with -fvisibility-inlines-hidden, foo has hidden visibility: ... 10: 6 FUNCWEAK HIDDEN 6 _Z3foov ... Tested

Re: [v3] libstdc++/50196 - enable std::thread, std::mutex etc. on darwin

2011-10-21 Thread Jonathan Wakely
On 21 October 2011 00:43, Jonathan Wakely wrote: This patch should enable macosx support for thread and partial support for mutex, by defining _GLIBCXX_HAS_GTHREADS on POSIX systems without the _POSIX_TIMEOUTS option, and only disabling the types which rely on the Timeouts option,

Re: new patches using -fopt-info (issue5294043)

2011-10-21 Thread Richard Guenther
On Thu, Oct 20, 2011 at 7:11 PM, Xinliang David Li davi...@google.com wrote: On Thu, Oct 20, 2011 at 1:21 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Oct 20, 2011 at 1:33 AM, Andi Kleen a...@firstfloor.org wrote: x...@google.com (Rong Xu) writes: After some off-line

Fwd: [patch] Fix PR tree-optimization/49960 ,Fix self data dependence

2011-10-21 Thread Richard Guenther
Forwarded to the list, gcc.gnu.org doesn't like gmail anymore. -- Forwarded message -- From: Richard Guenther richard.guent...@gmail.com Date: Fri, Oct 21, 2011 at 11:03 AM Subject: Re: [patch] Fix PR tree-optimization/49960 ,Fix self data dependence To: Razya Ladelsky

Re: [PATCH, PR50763] Fix for ICE in verify_gimple

2011-10-21 Thread Richard Guenther
On Thu, Oct 20, 2011 at 3:48 PM, Tom de Vries tom_devr...@mentor.com wrote: Richard, I have a fix for PR50763. The second example from the PR looks like this: ... int bar (int i); void foo (int c, int d) {  if (bar (c))    bar (c);  d = 33;  while (c == d); } ... When compiled

Re: [PATCH] Fix PR46556 (poor address generation)

2011-10-21 Thread Richard Guenther
On Tue, Oct 18, 2011 at 4:14 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: Greetings, Here is a new revision of the tree portions of this patch.  I moved the pattern recognizer to expand, and added additional logic to look for the same pattern in gimple form.  I added two more

Re: [PATCH] Extend vect_recog_bool_pattern also to stores into bool memory (PR tree-optimization/50596)

2011-10-21 Thread Jakub Jelinek
On Fri, Oct 21, 2011 at 11:19:32AM +0200, Richard Guenther wrote: I'll try to poke at that a bit, thus support general bit-precision types for loads and stores and the few operations that are safe on them. If you have a store to a bool like int *a, *b; _Bool *c; for (;;) c[i] = a[i]

Re: [RFA:] fix breakage with Update testsuite to run with slim LTO

2011-10-21 Thread Jan Hubicka
Date: Fri, 21 Oct 2011 00:19:32 +0200 From: Jan Hubicka hubi...@ucw.cz Yes, if we scan assembler, we likely want -fno-fat-lto-objects. then IIUC you need to patch *all* torture tests that use scan-assembler and scan-assembler-not. Alternatively, patch somewhere else, like not

Re: [PATCH 1/3] Add missing page rounding of a page_entry

2011-10-21 Thread Richard Guenther
On Fri, Oct 21, 2011 at 7:52 AM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com This one place in ggc forgot to round page_entry-bytes to the next page boundary, which lead to all the heuristics in freeing to check for continuous memory failing. Round here too,

Re: [C++ Patch] PR 30066

2011-10-21 Thread Roberto Agostino Vitillo
I am resubmitting the changelog since it was not respecting the conventions. gcc/Changelog: 2011-10-21 Roberto Agostino Vitillo raviti...@lbl.gov PR c++/30066 * doc/invoke.texi (fvisibility-inlines-hidden): Documentation change. gcc/c-family/Changelog: 2011-10-21 Roberto

Re: [PATCH 1/3] Add missing page rounding of a page_entry

2011-10-21 Thread Jakub Jelinek
On Fri, Oct 21, 2011 at 11:42:26AM +0200, Richard Guenther wrote: On Fri, Oct 21, 2011 at 7:52 AM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com This one place in ggc forgot to round page_entry-bytes to the next page boundary, which lead to all the heuristics

Re: [PATCH 2/3] Free large chunks in ggc

2011-10-21 Thread Richard Guenther
On Fri, Oct 21, 2011 at 7:52 AM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com This implements the freeing back of large chunks in the ggc madvise path Richard Guenther asked for.  This way on systems with limited address space malloc() and other allocators still

[C++ Patch] __builtin_choose_expr *bump*

2011-10-21 Thread Andy Gibbs
Hi, Please can I bump this patch and ask for it to be approved and committed: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01711.html The patch is to implement the C built-in function __builtin_choose_expr(...) in C++. I'm afraid I am new to contributing to GCC, so I hope I am going about this

Re: [C++ Patch] __builtin_choose_expr *bump*

2011-10-21 Thread Richard Guenther
On Fri, Oct 21, 2011 at 11:57 AM, Andy Gibbs andyg1...@hotmail.co.uk wrote: Hi, Please can I bump this patch and ask for it to be approved and committed: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01711.html The patch is to implement the C built-in function __builtin_choose_expr(...) in

[Patch,AVR]: Use EIND consistently

2011-10-21 Thread Georg-Johann Lay
This patch adds support to consistently use EIND. The compiler never sets this SFR but uses it in table jumps and EIJMP/EICALL instructions. Custom startup code could set EIND to an other value than 0 and the compiler should use EIND consistently given that EIND might not be zero. EIND != 0

[C++ Patch / RFC] PR 45385

2011-10-21 Thread Paolo Carlini
Hi, this one is a bit subtler. It's actually a regression due to the fix for PR35602, which was about a bogus warning for: struct c { ~c(); c(); }; int main() { c x[0UL][0UL] = // { dg-bogus warning: conversion to .long unsigned int. from .long int. may change the sign of the result

[commit][arm] Fix pr50809: build failure with --enable-build-with-cxx

2011-10-21 Thread Andrew Stubbs
I've just committed this patch to fix PR50809, in which driver-arm.c failed to build with a C++ compiler and -Werror. The patch is pre-approved by Ramana, and anyway probably qualifies as obvious. Andrew 2011-10-21 Andrew Stubbs a...@codesourcery.com PR target/50809 gcc/ *

[Patch] Add support of AIX response files in collect2

2011-10-21 Thread Tristan Gingold
Hi, the AIX linker supports response files with the '-fFILE' command line option. With this patch, collect2 reads the content of the response files and listed object files are handled. Therefore these files are not forgotten by the collect2 machinery. Although AIX ld supports glob(3)

[i386, PR50740] CPUID leaf 7 for BMI/BMI2/AVX2 feature detection not qualified with max_level and doesn't use subleaf

2011-10-21 Thread Kirill Yukhin
Hello, Here is the patch which checks CPUID correctly to get BMI/BMI2/AVX2 feature. ChangeLog entry is: 2011-10-21 H.J. Lu hongjiu...@intel.com Kirill Yukhin kirill.yuk...@intel.com * config/i386/driver-i386.c (host_detect_local_cpu): Do cpuid 7 only if max_level

Re: [Patch,AVR]: Use EIND consistently

2011-10-21 Thread Denis Chertykov
2011/10/21 Georg-Johann Lay a...@gjlay.de: This patch adds support to consistently use EIND. The compiler never sets this SFR but uses it in table jumps and EIJMP/EICALL instructions. Custom startup code could set EIND to an other value than 0 and the compiler should use EIND consistently

Handle weakrefs from callgraph

2011-10-21 Thread Jan Hubicka
Hi, this patch makes weakref that has no declaration they alias to to go the callgraph way instead of alias pair way. This should make practically all sane aliases go the callgraph way, but we still do not handle aliases from functions to variables and vice versa. They are not too hard to

Re: [RFA:] fix breakage with Update testsuite to run with slim LTO

2011-10-21 Thread Iain Sandoe
On 21 Oct 2011, at 10:31, Jan Hubicka wrote: Date: Fri, 21 Oct 2011 00:19:32 +0200 From: Jan Hubicka hubi...@ucw.cz Yes, if we scan assembler, we likely want -fno-fat-lto-objects. then IIUC you need to patch *all* torture tests that use scan-assembler and scan-assembler-not. Alternatively,

Re: [PATCH][PING] Vectorize conversions directly

2011-10-21 Thread Ramana Radhakrishnan
Otherwise the generic parts of the patch look good. Please get separate approval for the arm portions of the patch. Is it just me or has no one else seen this patch on the archives at gcc-patches@. ? Ramana

Re: [RFA:] fix breakage with Update testsuite to run with slim LTO

2011-10-21 Thread Rainer Orth
Iain Sandoe develo...@sandoe-acoustics.co.uk writes: It looks like the gnat testsuite is also broken - but HP's fix doesn't recover that. .. will try and take a look - but short on time today, I think I see what's going on: in gnat.log, I find Running

Re: [cxx-mem-model] Handle x86-64 with -m32

2011-10-21 Thread Andrew MacLeod
On 10/20/2011 06:50 PM, H.J. Lu wrote: On Thu, Oct 20, 2011 at 3:38 PM, Joseph S. Myers jos...@codesourcery.com wrote: Do these operations exist for x32 as well as for -m64? If they do, then lp64 isn't the right test either; if not, then it is. X32 has native int64 and int128. I presume

Re: [patch tree-optimization]: allow branch-cost optimization for truth-and/or on mode-expanded simple boolean-operands

2011-10-21 Thread Kai Tietz
2011/10/21 Richard Guenther richard.guent...@gmail.com: On Thu, Oct 20, 2011 at 3:08 PM, Kai Tietz kti...@redhat.com wrote: Hello, this patch re-enables the branch-cost optimization on simple boolean-typed operands, which are casted to a wider integral type.  This happens due casts from

Re: [PATCH] Fix PR46556 (poor address generation)

2011-10-21 Thread William J. Schmidt
On Fri, 2011-10-21 at 11:26 +0200, Richard Guenther wrote: On Tue, Oct 18, 2011 at 4:14 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: snip + + /* We don't use get_def_for_expr for S1 because TER doesn't forward + S1 in some situations where this transform is useful,

Re: [Patch,AVR]: Use EIND consistently

2011-10-21 Thread Georg-Johann Lay
This patch adds support to consistently use EIND. The compiler never sets this SFR but uses it in table jumps and EIJMP/EICALL instructions. Custom startup code could set EIND to an other value than 0 and the compiler should use EIND consistently given that EIND might not be zero. EIND !=

Re: [PATCHv2, RFA] Pass address space to REGNO_MODE_CODE_OK_FOR_BASE_P

2011-10-21 Thread Georg-Johann Lay
Ulrich Weigand schrieb: Georg-Johann Lay wrote: Ulrich Weigand schrieb: Hello, Georg-Johann Lay has proposed a patch to add named address space support to the AVR target here: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00471.html Since the target needs to make register allocation

Re: [RFC PATCH] SLP vectorize calls

2011-10-21 Thread Ira Rosen
On 20 October 2011 23:50, Jakub Jelinek ja...@redhat.com wrote: Hi! Hi, While looking at *.vect dumps from Polyhedron, I've noticed the lack of SLP vectorization of builtin calls. This patch is an attempt to handle at least 1 and 2 operand builtin calls (SLP doesn't handle ternary stmts

[RFC ARM] Use vcvt.f32/64.s32 with immediate bits to do fixed to floating point conversions better.

2011-10-21 Thread Ramana Radhakrishnan
Hi, Some time back Michael pointed out that the ARM backend doesn't generate vcvt.f32.stype where you have a conversion from fixed to floating point as in the example below. It should also be possible to generate the vector forms of this which will be the subject of a follow-up patch . I've

Re: [RFA/ARM][Patch 01/02]: Thumb2 epilogue in RTL

2011-10-21 Thread Ramana Radhakrishnan
Hi Sameera, The comment about REG_FRAME_RELATED_EXPR vs REG_CFA_RESTORE from one of your later patches applies here as well. diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c index 3162b30..f86a3e6 100644 --- a/gcc/config/arm/arm.c +++ b/gcc/config/arm/arm.c @@ -8754,6 +8754,140 @@

Re: [RFA/ARM][Patch 01/05]: Create tune for Cortex-A15.

2011-10-21 Thread Ramana Radhakrishnan
2011-10-11  Sameera Deshpande sameera.deshpa...@arm.com        * config/arm/arm-cores.def (cortex_a15): Update.        * config/arm/arm-protos.h (struct tune_params): Add new field...          (arm_gen_ldrd_strd): ... this.        * config/arm/arm.c (arm_slowmul_tune): Add          

Re: [RFA/ARM][Patch 03/05]: STRD generation instead of PUSH in A15 Thumb2 prologue.

2011-10-21 Thread Ramana Radhakrishnan
On 11 October 2011 10:27, Sameera Deshpande sameera.deshpa...@arm.com wrote: Hi! This patch generates STRD instruction instead of PUSH in thumb2 mode for A15. For optimize_size, original prologue is generated for A15. The work involves defining new functions, predicates and patterns.

Re: [RFA/ARM][Patch 04/05]: STRD generation instead of PUSH in A15 ARM prologue.

2011-10-21 Thread Ramana Radhakrishnan
+/* STRD in ARM mode needs consecutive registers to be stored. This function + keeps accumulating non-consecutive registers until first consecutive register numchar 80. + pair is found. It then generates multi-reg PUSH for all accumulated + registers, and then generates STRD with

Re: [RFA/ARM][Patch 05/05]: LDRD generation instead of POP in A15 ARM epilogue.

2011-10-21 Thread Ramana Radhakrishnan
2011-10-11  Sameera Deshpande sameera.deshpa...@arm.com        * config/arm/arm.c (arm_emit_ldrd_pop): New static function.          (arm_expand_epilogue): Update.        * config/arm/ldmstm.md (arm_ldrd_base): New pattern.          (arm_ldr_with_update): Likewise. rth's comment about

Re: [RFC PATCH] SLP vectorize calls

2011-10-21 Thread Jakub Jelinek
On Fri, Oct 21, 2011 at 02:37:06PM +0200, Ira Rosen wrote: @@ -1620,7 +1615,13 @@ vectorizable_call (gimple stmt, gimple_s   gcc_assert (!gimple_vuse (stmt)); -  if (modifier == NARROW) +  if (slp_node || PURE_SLP_STMT (stmt_info)) +    { +      if (modifier != NONE) +      

Re: [i386, PR50740] CPUID leaf 7 for BMI/BMI2/AVX2 feature detection not qualified with max_level and doesn't use subleaf

2011-10-21 Thread Uros Bizjak
On Fri, Oct 21, 2011 at 12:46 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Here is the patch which checks CPUID correctly to get BMI/BMI2/AVX2 feature. ChangeLog entry is: 2011-10-21  H.J. Lu  hongjiu...@intel.com            Kirill Yukhin  kirill.yuk...@intel.com        *

Re: [PATCHv2, RFA] Pass address space to REGNO_MODE_CODE_OK_FOR_BASE_P

2011-10-21 Thread Ulrich Weigand
Georg-Johann Lay wrote: Does it make sense to extend LEGITIMIZE_RELOAD_ADDRESS, too? For the target that needs your extension (AVR) there are different addressing capabilities depending on AS and there is an implementation of L_R_A. I'd say that it is an independent issue.

strlen optimization of decompose strcat?!

2011-10-21 Thread Andreas Krebbel
Hi, on s390 a strcat is already decomposed by fold_builtin_strcat into a strlen and a strcpy. Due to that 3 strlenopt testcases currently fail: strlenopt-17g.c, strlenopt-4.c, strlenopt-4g.c. For strlenopt-4.c no optimization is expected anyway (stpcpy disabled). So this can easily be fixed by

Re: [C++ Patch] PR 50811 (rejects class-virt-specifier if class-head-name includes nested-name-specifier)

2011-10-21 Thread Jason Merrill
Applied, thanks. Jason

Re: [RFC PATCH] SLP vectorize calls

2011-10-21 Thread Ira Rosen
On 21 October 2011 14:52, Jakub Jelinek ja...@redhat.com wrote: On Fri, Oct 21, 2011 at 02:37:06PM +0200, Ira Rosen wrote: @@ -1620,7 +1615,13 @@ vectorizable_call (gimple stmt, gimple_s   gcc_assert (!gimple_vuse (stmt)); -  if (modifier == NARROW) +  if (slp_node || PURE_SLP_STMT

Re: [i386, PR50740] CPUID leaf 7 for BMI/BMI2/AVX2 feature detection not qualified with max_level and doesn't use subleaf

2011-10-21 Thread Kirill Yukhin
Thanks, Updated testsuite/ChangeLog: 2011-10-21 H.J. Lu hongjiu...@intel.com Kirill Yukhin kirill.yuk...@intel.com * gcc.target/i386/avx2-check.h (main): Check CPUID level correctly. * gcc.target/i386/bmi2-check.h: Ditto. Could anybody please commit that?

Re: [C++ Patch] PR 30066

2011-10-21 Thread Jason Merrill
Need to make sure that this comment is still accurate: /* Local statics and classes get the visibility of their containing function by default, except that -fvisibility-inlines-hidden doesn't affect them. */ i.e. given inline int * f() { static int i;

Re: strlen optimization of decompose strcat?!

2011-10-21 Thread Jakub Jelinek
On Fri, Oct 21, 2011 at 03:20:54PM +0200, Andreas Krebbel wrote: on s390 a strcat is already decomposed by fold_builtin_strcat into a strlen and a strcpy. Due to that 3 strlenopt testcases currently fail: strlenopt-17g.c, strlenopt-4.c, strlenopt-4g.c. Well, fold_builtin_strcat does such kind

Re: [Patch,AVR]: Use EIND consistently

2011-10-21 Thread Georg-Johann Lay
This patch adds support to consistently use EIND. The compiler never sets this SFR but uses it in table jumps and EIJMP/EICALL instructions. Custom startup code could set EIND to an other value than 0 and the compiler should use EIND consistently given that EIND might not be zero.

Re: [i386, PR50740] CPUID leaf 7 for BMI/BMI2/AVX2 feature detection not qualified with max_level and doesn't use subleaf

2011-10-21 Thread Uros Bizjak
On Fri, Oct 21, 2011 at 3:58 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Updated testsuite/ChangeLog: 2011-10-21  H.J. Lu  hongjiu...@intel.com            Kirill Yukhin  kirill.yuk...@intel.com        * gcc.target/i386/avx2-check.h (main): Check CPUID level        correctly.        *

Re: [i386, PR50740] CPUID leaf 7 for BMI/BMI2/AVX2 feature detection not qualified with max_level and doesn't use subleaf

2011-10-21 Thread Kirill Yukhin
Thanks! K On Fri, Oct 21, 2011 at 6:34 PM, Uros Bizjak ubiz...@gmail.com wrote: On Fri, Oct 21, 2011 at 3:58 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Updated testsuite/ChangeLog: 2011-10-21  H.J. Lu  hongjiu...@intel.com            Kirill Yukhin  kirill.yuk...@intel.com        *

Re: [C++ Patch] __builtin_choose_expr *bump*

2011-10-21 Thread Joseph S. Myers
On Fri, 21 Oct 2011, Andy Gibbs wrote: Hi, Please can I bump this patch and ask for it to be approved and committed: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01711.html Have you sent in your copyright assignment papers to the FSF? The patch is large enough to need them. -- Joseph S.

Re: [C++ Patch / RFC] PR 45385

2011-10-21 Thread Jason Merrill
I think the fix for 35602 was wrong; instead of trying to suppress the warning, we should avoid building expressions that trip it. In this case, the problem is a type mismatch in build_vec_init between maxindex/iterator (ptrdiff_type_node) and array_type_nelts_total (sizetype). And indeed,

Re: [C++ Patch] __builtin_choose_expr *bump*

2011-10-21 Thread Andy Gibbs
On Friday, October 21, 2011 4:42 PM, Joseph S. Myers wrote: On Fri, 21 Oct 2011, Andy Gibbs wrote: Hi, Please can I bump this patch and ask for it to be approved and committed: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01711.html Have you sent in your copyright assignment papers to

[PR translation/47064] Fix committed as obvious

2011-10-21 Thread Paolo Carlini
Hi, committed. Thanks, Paolo. // 2011-10-21 Roland Stigge sti...@antcom.de PR translation/47064 * params.def: Fix typo compilatoin - compilation. Index: params.def === --- params.def

Re: [RFC PATCH] SLP vectorize calls

2011-10-21 Thread Ira Rosen
On 21 October 2011 16:25, Jakub Jelinek ja...@redhat.com wrote: On Fri, Oct 21, 2011 at 03:44:11PM +0200, Ira Rosen wrote: But it's OK to allow modifier != NONE if it's not SLP, so we need , no? Well, in my patch that check was guarded by the if (slp_node ...), so presumably it would allow

Re: [cxx-mem-model] Handle x86-64 with -m32

2011-10-21 Thread H.J. Lu
On Fri, Oct 21, 2011 at 5:11 AM, Andrew MacLeod amacl...@redhat.com wrote: On 10/20/2011 06:50 PM, H.J. Lu wrote: On Thu, Oct 20, 2011 at 3:38 PM, Joseph S. Myers jos...@codesourcery.com  wrote: Do these operations exist for x32 as well as for -m64?  If they do, then lp64 isn't the right

[Patch,AVR]: Fix thinko in LEGITIMIZE_RELOAD_ADDRESS

2011-10-21 Thread Georg-Johann Lay
This fixes avr_legitimize_reload_address: Since breaking out the code from LEGITIMIZE_RELOAD_ADDRESS, protiype of above is avr_legitimize_reload_address (rtx x, ... but must be avr_legitimize_reload_address (rtx *px, ... because at one place x is used as input to push_reload which is now

Re: [Patch,AVR]: Fix thinko in LEGITIMIZE_RELOAD_ADDRESS

2011-10-21 Thread Denis Chertykov
2011/10/21 Georg-Johann Lay a...@gjlay.de: This fixes avr_legitimize_reload_address: Since breaking out the code from LEGITIMIZE_RELOAD_ADDRESS, protiype of above is   avr_legitimize_reload_address (rtx x, ... but must be   avr_legitimize_reload_address (rtx *px, ... because at one place

Re: [RFA:] fix breakage with Update testsuite to run with slim LTO

2011-10-21 Thread Jan Hubicka
If running the gnat.dg testsuite, lib/gcc-dg.exp is now calling check_linker_plugin_available early, which ultimately calls ${tool}_target_compile. For all languages but Ada, ${tool}_target_compile can compile .c files just fine, but gnat_target_compile (which uses gnatmake) cannot, so it

Re: [Patch] Add support of AIX response files in collect2

2011-10-21 Thread David Edelsohn
On Fri, Oct 21, 2011 at 6:39 AM, Tristan Gingold ging...@adacore.com wrote: Hi, the AIX linker supports response files with the '-fFILE' command line option. With this patch, collect2 reads the content of the response files and listed object files are handled.  Therefore these files are not

Re: [cxx-mem-model] Handle x86-64 with -m32

2011-10-21 Thread Andrew MacLeod
On 10/21/2011 11:28 AM, H.J. Lu wrote: On Fri, Oct 21, 2011 at 5:11 AM, Andrew MacLeodamacl...@redhat.com wrote: X32 has native int64 and int128. I presume there is no atomic support for int128 though, and thats what 'condition check_effective_target_sync_int_128' is testing for. X32 uses

Re: [C++-11] User defined literals

2011-10-21 Thread 3dw4rd
Jason, I split the changelog into four parts and tweaked some of the content and formatting. Ed CL_udlit_gcc_c-family Description: Binary data CL_udlit_gcc_cp Description: Binary data CL_udlit_gcc_testsuite Description: Binary data CL_udlit_libcpp Description: Binary data

Re: [cxx-mem-model] Handle x86-64 with -m32

2011-10-21 Thread H.J. Lu
On Fri, Oct 21, 2011 at 9:08 AM, Andrew MacLeod amacl...@redhat.com wrote: On 10/21/2011 11:28 AM, H.J. Lu wrote: On Fri, Oct 21, 2011 at 5:11 AM, Andrew MacLeodamacl...@redhat.com  wrote: X32 has native int64 and int128. I presume there is no atomic support for int128 though, and thats

Re: [RFA:] fix breakage with Update testsuite to run with slim LTO

2011-10-21 Thread Hans-Peter Nilsson
Date: Fri, 21 Oct 2011 17:44:15 +0200 From: Jan Hubicka hubi...@ucw.cz I also noticed that tests scanning output of late optimization passes are now getting UNRESOLVED state with slim LTO. We don't really lose coverage here because we test fat LTO with the other compilation, but probably

Re: [C++ Patch / RFC] PR 45385

2011-10-21 Thread Paolo Carlini
On 10/21/2011 04:56 PM, Jason Merrill wrote: I think the fix for 35602 was wrong; instead of trying to suppress the warning, we should avoid building expressions that trip it. In this case, the problem is a type mismatch in build_vec_init between maxindex/iterator (ptrdiff_type_node) and

Re: new patches using -fopt-info (issue5294043)

2011-10-21 Thread Xinliang David Li
There are two proposals here. One is -fopt-info which prints out informational notes to stderr, and the other is -fopt-report which is more elaborate form of dump files. Are you object to both or just the opt-report one? The former is no different from any other informational notes we already

Re: [patch] dwarf2out crash: missing GTY? (PR 50806)

2011-10-21 Thread Steve Ellcey
FYI: I am seeing this same ICE on the hppa64-hp-hpux11.11 bootstrap. (debug_expr:DI D#49) /ctires/gcc/nightly/src/trunk/gcc/cselib.c: In function 'void cselib_record_sets(rtx)': /ctires/gcc/nightly/src/trunk/gcc/cselib.c:2424:1: internal compiler error: in mem_loc_descriptor, at

Re: [PATCH 1/3] Add missing page rounding of a page_entry

2011-10-21 Thread Andi Kleen
On Fri, Oct 21, 2011 at 11:52:44AM +0200, Jakub Jelinek wrote: On Fri, Oct 21, 2011 at 11:42:26AM +0200, Richard Guenther wrote: On Fri, Oct 21, 2011 at 7:52 AM, Andi Kleen a...@firstfloor.org wrote: From: Andi Kleen a...@linux.intel.com This one place in ggc forgot to round

Re: [patch, Fortran] Fix PR 50690

2011-10-21 Thread Thomas Koenig
Jakub Jelinek wrote: Though, what could be done is just special case OpenMP workshare regions, insert everything into BLOCK local vars unless in OpenMP workshare, in that case put the BLOCK with the temporary around the workshare rather than inside of it. In the case of omp parallel workshare

Re: [cxx-mem-model] Handle x86-64 with -m32

2011-10-21 Thread Aldy Hernandez
On 10/21/11 11:08, Andrew MacLeod wrote: On 10/21/2011 11:28 AM, H.J. Lu wrote: On Fri, Oct 21, 2011 at 5:11 AM, Andrew MacLeodamacl...@redhat.com wrote: X32 has native int64 and int128. I presume there is no atomic support for int128 though, and thats what 'condition

Re: [C++ Patch] PR 48630 (PR 31423)

2011-10-21 Thread Paolo Carlini
Hi again, Another acceptable fix is to -- leave the current diagnostic as is if -fms-extensions -- suggest '()' is member function -- otherwise suggest ''. Thanks for your help Gaby, in particular about the MS extension which I had overlooked completely (as any hard-code Linux guy

[patch] dwarf2out: Use DW_FORM_ref_udata (.debug -= 7.53%)

2011-10-21 Thread Jan Kratochvil
Hi, this is a standalone patch to replace DW_FORM_ref4 by DW_FORM_ref_udata (uleb128). It saves for libstdc++.so.debug: 5254792 - 4859136 = 7.53% Tested with: -O2 -gdwarf-4 -fdebug-types-section The references are already intra-CU ones, they are never relocated, they are pre-computed by

Re: [patch] dwarf2out: Use DW_FORM_ref_udata (.debug -= 7.53%)

2011-10-21 Thread Jan Kratochvil
On Fri, 21 Oct 2011 20:01:29 +0200, Jan Kratochvil wrote: No regressions on {x86_64,x86_64-m32,i686}-fedora16pre-linux-gnu (4.7.0 20111002). A typo, only tested for x86_64-fedora16pre-linux-gnu, sorry. Jan

Re: [C++ Patch] PR 48630 (PR 31423)

2011-10-21 Thread Jason Merrill
OK. Jason

Re: [C++ Patch / RFC] PR 45385

2011-10-21 Thread Jason Merrill
On 10/21/2011 12:20 PM, Paolo Carlini wrote: + || tree_int_cst_equal (maxindex, integer_minus_one_node)) Use integer_all_onesp instead. OK with that change. Jason

Re: [PATCH 2/3] Free large chunks in ggc

2011-10-21 Thread Andi Kleen
diff --git a/gcc/ggc-page.c b/gcc/ggc-page.c index ba88e3f..eb0eeef 100644 --- a/gcc/ggc-page.c +++ b/gcc/ggc-page.c @@ -972,6 +972,54 @@ release_pages (void)   page_entry *p, *start_p;   char *start;   size_t len; +  size_t mapped_len; +  page_entry *next, *prev, *newprev; +

Re: [RFA:] fix breakage with Update testsuite to run with slim LTO

2011-10-21 Thread Jan Hubicka
Meh... Please no, this was the kind of scatter-patches my patch aimed to avoid... for example, easy to miss some tests. Instead, on top of my patch, just copy the scan-assembler_required_options proc to a scan-tree-dump_required_options. ...no wait, should forcing fat-lto be done for

Re: Predication during scheduling

2011-10-21 Thread Bernd Schmidt
On 10/21/11 15:42, Bernd Schmidt wrote: On 10/14/11 17:35, Vladimir Makarov wrote: The scheduler part of the patch is ok for me (other part changes are obvious). Could you only commit it at the beginning of the next week. I've committed this variant. It's updated for some recent changes in

[PATCH] Fix V4DImode/V8SImode extract even/odd permutation with -mavx (PR target/50813)

2011-10-21 Thread Jakub Jelinek
Hi! No idea how I've missed this. With -mavx V8SImode and V4DImode are valid, but the choice of shuffle insns for them is limited. This patch will just pay the reinterpretation penalty and reshuffle them as corresponding V4DFmode resp. V8SFmode instead of ICE. Additionally, I've added two new

Re: [pph] Make libcpp symbol validation a warning (issue5235061)

2011-10-21 Thread Lawrence Crowl
On 10/20/11, Gabriel Charette gcharet...@gmail.com wrote: I just thought about something.. Earlier I said that ALL line_table issues were resolved after this patch (as it ignores the re-included headers that were guarded, as the non-pph compiler does naturally). One problem remains however,

Ping: [PATCH] non-GNU C++ compilers

2011-10-21 Thread Marc Glisse
Hello, anyone willing to commit this? On Sat, 24 Sep 2011, Marc Glisse wrote: On Sat, 17 Sep 2011, Joseph S. Myers wrote: These are OK (with ChangeLog entries properly omitting the include/, since they go in include/ChangeLog) in the absence of libiberty maintainer objections within 72

[Patch,AVR] Clean-up avr.c: Break long lines, move targetm to end of file.

2011-10-21 Thread Georg-Johann Lay
No functional change, just a bit of clean-up. Definition of targetm is moved towards end of the file. Ok for trunk? Johann * config/avr/avr.c: Break long lines. Define target hooks on the fly if applicable. (TARGET_ASM_FUNCTION_RODATA_SECTION): Remove first definition

[PATCH, libcpp] Fix cpp_peek_token behaviour (PR bootstrap/50778)

2011-10-21 Thread Dodji Seketeli
Hello, cpp_peek_token can fail to peek tokens sometimes because _cpp_remaining_tokens_num_in_context counts tokens only from the current context; worse, _cpp_token_from_context_at also get tokens only from the context context. They should both operate on the context given by cpp_peek_token. I

Ping: demangle C++ extern C functions

2011-10-21 Thread Marc Glisse
Ping (changing the Cc to bother a different person...). On Sat, 3 Sep 2011, Marc Glisse wrote: Hello, this patch is obviously related to PR c++/2316 (g++ fails to overload on language linkage) but seems fairly independent. Currently, the demangler recognizes 'Y' for extern C functions and

RE: [Patch,AVR] Clean-up avr.c: Break long lines, move targetm to end of file.

2011-10-21 Thread Weddington, Eric
-Original Message- From: Georg-Johann Lay [mailto:a...@gjlay.de] Sent: Friday, October 21, 2011 12:52 PM To: gcc-patches@gcc.gnu.org Cc: Weddington, Eric; Denis Chertykov Subject: [Patch,AVR] Clean-up avr.c: Break long lines, move targetm to end of file. No functional change,

Re: [patch] dwarf2out: Use DW_FORM_ref_udata (.debug -= 7.53%)

2011-10-21 Thread Jakub Jelinek
On Fri, Oct 21, 2011 at 08:01:29PM +0200, Jan Kratochvil wrote: I had a draft patch to use DW_FORM_ref1, DW_FORM_ref2, DW_FORM_ref_udata and DW_FORM_ref4 which should be slightly smaller than just DW_FORM_ref_udata. But it produced larger files than just DW_FORM_ref_udata. Assuming it was due

Re: [PATCH] Fix V4DImode/V8SImode extract even/odd permutation with -mavx (PR target/50813)

2011-10-21 Thread Richard Henderson
On 10/21/2011 11:42 AM, Jakub Jelinek wrote: PR target/50813 * config/i386/i386.c (expand_vec_perm_even_odd_1): Handle V4DImode and V8SImode for !TARGET_AVX2. * gcc.dg/torture/vshuf-32.inc: Add broadcast permutation from element other than first and reverse

Re: [RFA:] fix breakage with Update testsuite to run with slim LTO

2011-10-21 Thread Hans-Peter Nilsson
Date: Fri, 21 Oct 2011 20:34:05 +0200 From: Jan Hubicka hubi...@ucw.cz I guess we could make ipa-dump/rtl-dump/tree-dump scanning to disable fat lto and introduce variants intended to scan late tree dumps and ipa execution dumps... Ok, sounds like a plan. Are there any such

Re: [v3] libstdc++/50196 - enable std::thread, std::mutex etc. on darwin

2011-10-21 Thread Jonathan Wakely
On 21 October 2011 09:15, Jonathan Wakely wrote: On 21 October 2011 00:43, Jonathan Wakely wrote: This patch should enable macosx support for thread and partial support for mutex, by defining _GLIBCXX_HAS_GTHREADS on POSIX systems without the _POSIX_TIMEOUTS option, and only disabling the

Re: [patch] dwarf2out: Use DW_FORM_ref_udata (.debug -= 7.53%)

2011-10-21 Thread Jan Kratochvil
On Fri, 21 Oct 2011 21:11:16 +0200, Jakub Jelinek wrote: Well, you calculate the sizes multiple times anyway, so I don't see why you during the size calculations you couldn't start with DW_FORM_ref_udata as first guess and compute on the side also total sizes of those DW_FORM_ref_udata bytes

Re: [PATCH, m68k] Fix floating-point comparisons with zero

2011-10-21 Thread Andreas Schwab
Julian Brown jul...@codesourcery.com writes: gcc/ * config/m68k/m68k.c (notice_update_cc): Tighten condition for setting CC_REVERSED for FP comparisons. Ok. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276

Re: [C++-11] User defined literals

2011-10-21 Thread Tom Tromey
Ed == Ed Smith-Rowland 3dw...@verizon.net writes: Ed + /* Nonzero for the 2011 C++ Standard. */ Ed + unsigned char cxx11; I think it would be better if the new field name reflected its purpose, so something like user_literals. Ed + if (ISIDST(*cur)) Ed + { Ed + type =

Re: [cxx-mem-model] Handle x86-64 with -m32

2011-10-21 Thread Joseph S. Myers
On Fri, 21 Oct 2011, Aldy Hernandez wrote: X32 uses x86-64 instruction set with 32bit pointers. It has the same atomic support as x86-64 and has atomic support for int128. Oh, you aren't talking about 32 bit, but a 32 bit abi on a 64 bit machine. Thanks for pointing this out Joseph.

Re: [patch] dwarf2out crash: missing GTY? (PR 50806)

2011-10-21 Thread Steve Ellcey
On Fri, 2011-10-21 at 21:25 +0200, Jakub Jelinek wrote: My guess would be http://gcc.gnu.org/viewcvs?root=gccview=revrev=180194 DEBUG_EXPR certainly shouldn't make it through into mem_loc_descriptor, var-tracking is supposed to resolve that. Jakub You are right, the bug started at

Re: Fix C6x 24-bit unwinding opcodes

2011-10-21 Thread Bernd Schmidt
On 10/03/11 17:52, Paul Brook wrote: The C6XABI defined persoality routines ID 3 and 4 use a single 24-bit block word of unwinding data. Patch below makes sure this is preserved, rather than treating it as a set of unwinding opcode bytes. I seem to have lost this bit of code when I

Re: [PATCH] Fix c6x unwinding info

2011-10-21 Thread Bernd Schmidt
On 10/03/11 17:57, Paul Brook wrote: 2011-10-03 Paul Brook p...@codesourcery.com * config/c6x/c6x.c (c6x_asm_emit_except_personality, c6x_asm_init_sections): New functions. (TARGET_ASM_EMIT_EXCEPT_PERSONALITY, TARGET_ASM_INIT_SECTIONS): Define. This was also never

Re: [C++-11] User defined literals

2011-10-21 Thread Jason Merrill
I think we're down to minor cosmetic issues: On 10/21/2011 03:55 PM, Tom Tromey wrote: There are a few spots like this that are missing a space before an open paren. + if (DECL_LANGUAGE(decl) == lang_c) Another one. - if (warn_cxx0x_compat - C_RID_CODE

[trans-mem] wrong calling convention _ITM_free with i*86

2011-10-21 Thread Patrick Marlier
The calling convention with i?86 (32bits) is wrong for _ITM_free (and _ITM_malloc/_ITM_calloc). Here an example: 0x08049fd2 +251: mov-0x14(%ebp),%eax 0x08049fd5 +254: call 0x804b150 _ITM_free(void*) Dump of assembler code for function _ITM_free(void*): 0x0804b150 +0: sub

Re: [libcpp] Correctly define __cplusplus (PR libstdc++-v3/1773)

2011-10-21 Thread Mike Stump
On Oct 21, 2011, at 12:52 PM, Jason Merrill wrote: On 10/21/2011 03:11 PM, Marc Glisse wrote: Note that at least clang now defines __cplusplus to its new C++11 value (in experimental C++0X mode only). Apparently they switched around last June and say they are not the only ones. So if you want

fix typo in reload

2011-10-21 Thread Mike Stump
In: r37370 | bernds | 2000-11-10 09:10:29 -0800 (Fri, 10 Nov 2000) | 2 lines Several fixes to make reload handle POST_MODIFY correctly. We see: -regno_clobbered_p (regno, insn, mode) +regno_clobbered_p (regno, insn, mode, sets) unsigned int regno; rtx insn; enum machine_mode

  1   2   >