[ARM] TLS Descriptor support

This patch implements TLS descriptor support in GCC. TLS descriptors are described at http://www.codesourcery.com/publications/RFC-TLSDESC-ARM.txt and blessed by ARM, who have reserved the relocation numbers. Binutils and GLIBC patches are already committed (there is an orthogonal glibc patch

Re: [PATCH] C++0x, fixes for override/final

Hi! Just small style nits: On Tue, May 10, 2011 at 09:35:42AM +0300, Ville Voutilainen wrote: > --- a/gcc/cp/class.c > +++ b/gcc/cp/class.c > @@ -2453,6 +2453,7 @@ get_basefndecls (tree name, tree t) > void > check_for_override (tree decl, tree ctype) > { > + int overrides_found = 0; s/int/b

Re: [PATCH] C++0x, fixes for override/final

At Tue, 10 May 2011 00:46:54 -0400, Jason Merrill wrote: > This is breaking an optimization: before this change we don't bother > looking for overrides if we've never seen a virtual function with the > same name. Just set overrides_found as well as DECL_VINDEX. Ok. Tests still pass. diff --git

[ping] 5 unreviewed patches

Fix annoying gcov filename handling: (2 lines) http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01380.html Introduce -Wstack-usage: http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01992.html Extend TYPE_DECL_IS_STUB trick in dwarf2out.c: (1 line) http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00683.

Re: [PATCH,c++] mark EXPR_PACK_EXPANSION as typed only, v2

OK. Jason

Re: [PATCH] C++0x, fixes for override/final

On 05/09/2011 05:55 PM, Ville Voutilainen wrote: + overrides_found = look_for_overrides (ctype, decl); if ((DECL_DESTRUCTOR_P (decl) || IDENTIFIER_VIRTUAL_P (DECL_NAME (decl)) || DECL_CONV_FN_P (decl)) -&& look_for_overrides (ctype, decl) +&& overrides_found && !

Re: [build] Define HAVE_GAS_HIDDEN on Darwin

On May 4, 2011, at 5:08 AM, Rainer Orth wrote: > The following patch is a prerequisite for making > > [lto, testsuite] Don't use visibility on targets that don't support it > (PR lto/47334) >http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00295.html > -# define USE_LINKONCE_INDIRECT (

[PATCH,c++] mark EXPR_PACK_EXPANSION as typed only, v2

Following on: http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00563.html and the review received in: http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00651.html I present the patch below. We have a new function, cp_tree_operand_length, and the code has been retrofitted to use it in various places. T

[C++, committed] Add c++/48522 testcase

Hi, I committed the testcase to mainline and 4_6-branch and closed the PR as already fixed. Tested x86_64-linux. Paolo. 2011-05-09 Paolo Carlini PR c++/48522 * g++.dg/cpp0x/pr48522.C: New. Index: g++.dg/cpp0x/pr48522.C ==

[C++, committed] Add c++/48735 testcase

Hi, I committed the testcase provided by Daniel in c++/48735 and closed the PR as already fixed. Also took the occasion to move my recent c++0x testcases to the proper cpp0x directory. Tested x86_64-linux. Paolo. 2011-05-09 Paolo Carlini PR c++/48735

Re: [PATCH] Fix dfp issue with dconst{1,2,m1,half} (PR debug/48928)

On Mon, May 09, 2011 at 09:11:59PM +0200, Jakub Jelinek wrote: > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK, thanks! Cheers, Ben -- "These man-made problems have man-made solutions. Unfortunately, the men and women needed to solve them are all politicians." -- Pet

Use Enum for rs6000 -mcpu=, -mtune= options

This patch makes the handling of the rs6000 -mcpu= and -mtune= options use Enum, completing the conversion of rs6000_handle_option - and so of all target option handlers - to avoid using modifiable global state. As with various other targets, a generated rs6000-tables.opt is used. This target had

Re: [google]Implement an optimization based on reuse distance profiling (issue4517049)

Ok. The instrumentation and optimization runtime has not been open sourced yet -- will need to be done later at some point. (For reference, see Silvius's CGO2011 paper). Thanks, David On Mon, May 9, 2011 at 2:49 PM, Easwaran Raman wrote: > This patch by Silvius Rus replaces calls to  certain

[PATCH] C++0x, fixes for override/final

Diagnose final on non-virtuals properly, diagnose override on virtuals that don't actually override properly. Amend these cases to the tests. Tested on Linux/X86-32. 2011-05-10 Ville Voutilainen Fixes for override/final. * class.c (check_for_override): Diagnose final on a

[google]Implement an optimization based on reuse distance profiling (issue4517049)

This patch by Silvius Rus replaces calls to certain functions with a specialized version that uses non-temporal stores based on memory reuse distance profiling. Bootstraps, no test regressions and the profiling works for a small test case. Ok for google/main.? -Easwaran 2011-05-09 Silvius Ru

Re: [PATCH] Fix remove_unreachable_handlers (PR tree-optimization/48794, tree-optimization/48611)

On 05/09/2011 12:19 PM, Jakub Jelinek wrote: > PR tree-optimization/48611 > PR tree-optimization/48794 > * tree-eh.c (remove_unreachable_handlers): Don't remove regions > referenced from RESX or EH_DISPATCH arguments. > > * gfortran.dg/gomp/pr48611.f90: New test. >

[patch][gimplefe] Preliminary variable declaration support

This patch is just a slight modification to an earlier patch that I submitted which was non-working. This is working. It just recognizes variable declarations and might be helpful for ketaki to start with things. It is now more or less up to date with the information on the wiki although not exactl

[PATCH, i386]: Handle flag_prefer_avx128 in ix86_autovectorize_vector_sizes

Hello! There is no point in trying to vectorize with 256bit vector sizes when we prefer 128bit vectors. Also, cleanup ix86_preferred_simd_mode and return word_mode for DFmode without SSE2. 2011-05-09 Uros Bizjak * config/i386/i386.c (ix86_autovectorize_vector_sizes): Return 0

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

On Tue, 2011-05-03 at 16:24 -0400, Jason Merrill wrote: > That makes sense to me; it seems appropriate for -g1 to have information > that makes a backtrace more informative, but not information for > interactive debugging. Jim, do you have an opinion? I'm not aware of any significant use of -g1

Re: PING: [PATCH] Fix up typed DWARF stack support for POINTERS_EXTEND_UNSIGNED targets (PR debug/48853)

On Mon, May 9, 2011 at 11:43 AM, Jakub Jelinek wrote: > On Mon, May 09, 2011 at 10:58:56AM -0700, H.J. Lu wrote: >> On Fri, May 6, 2011 at 6:11 PM, H.J. Lu wrote: >> > 2011-05-06  H.J. Lu   >> > >> >        PR debug/48853 >> >        * dwarf2out.c (mem_loc_descriptor) : If >> >        POINTERS_EX

Re: Fix thinko in var-tracking.c

On May 9, 2011, at 6:05 AM, Joseph S. Myers wrote: > The guality tests have random results. I don't know if that's the issue > here, but perhaps someone can review Jeff's patch > to make them > more predictable. Without that, I generall

Re: [PATCH, MELT] fix useless forcing of GCC garbage collector

Thanks you for the correction, I will take care next time. Pierre On 09/05/2011 19:03, Basile Starynkevitch wrote: On Mon, 09 May 2011 14:15:30 +0200 Pierre Vittet wrote: This patch is for the MELT branch. The diff file was slightly wrong (diff run under gcc/). Pierre, you should r

Re: Fix problem with guality tests

On Mar 17, 2011, at 1:33 PM, Jeff Law wrote: > The guality tests can randomly fail due to expect buffering issues. > OK for trunk? Generally, I really would prefer the people that create and maintain an area to review and approve these sorts of changes... The narrower the reviewer, the better.

Re: [Patch] testcase for c++/20039

OK. Jason

[Patch] testcase for c++/20039

Hi, PR c++/20039 was closed without providing a testcase, here's one. tested x86_64-unknown-linux-gnu. OK for trunk ? gcc/testsuite/ChangeLog: 2011-05-09 Fabien Chêne PR c++/20039 * g++.dg/init/pr20039.C: New. -- Fabien pr20039.patch Description: Binary data

Re: [C++0x] contiguous bitfields race implementation

From a quick look it seems that this patch considers bitfields following the one we're deliberately touching, but not previous bitfields in the same memory location; we need to include those as well. With your struct foo, the bits touched are the same regardless of whether we name .a or .b.

[PATCH] Fix remove_unreachable_handlers (PR tree-optimization/48794, tree-optimization/48611)

Hi! As the new testcase shows, while my PR48611 patch stopped the problem from triggering on that testcase, it wasn't a real fix, just (IMHO) right thing to do to get better code. The real problem seems to be that if for whatever reason some optimizations don't happen or don't happen fully (in th

[PATCH] Fix dfp issue with dconst{1,2,m1,half} (PR debug/48928)

Hi! The folder/middle-end/tree passes/rtl passes apparently use dconst{1,2,m1,half} and/or const_tiny_rtx[{1,2}][{S,D,T}Dmode] in various places. E.g. /* Convert x+x into x*2.0. */ if (operand_equal_p (arg0, arg1, 0) && SCALAR_FLOAT_TYPE_P (type)) re

Re: [C++0x] contiguous bitfields race implementation

On Mon, May 09, 2011 at 01:41:13PM -0500, Aldy Hernandez wrote: > Jakub also gave me a testcase which triggered a buglet in > max_field_size. I have now added a parameter INNERDECL which is the > inner reference, so we can properly determine if the inner decl is > thread visible or not. What I me

Re: [PATCH, testsuite]: Add -mprefer-avx128 to DEFAULT_VECTCFLAGS

On Mon, 9 May 2011, H.J. Lu wrote: > > Index: lib/target-supports.exp > > === > > --- lib/target-supports.exp     (revision 173569) > > +++ lib/target-supports.exp     (working copy) > > @@ -3845,6 +3845,8 @@ > >         set dg-do-wha

Re: PING: [PATCH] Fix up typed DWARF stack support for POINTERS_EXTEND_UNSIGNED targets (PR debug/48853)

On Mon, May 09, 2011 at 10:58:56AM -0700, H.J. Lu wrote: > On Fri, May 6, 2011 at 6:11 PM, H.J. Lu wrote: > > 2011-05-06  H.J. Lu   > > > >        PR debug/48853 > >        * dwarf2out.c (mem_loc_descriptor) : If > >        POINTERS_EXTEND_UNSIGNED, don't give up if mode is Pmode and > >        me

Re: [C++0x] contiguous bitfields race implementation

_TO_SINK) +#define ALLOW_STORE_DATA_RACES \ + PARAM_VALUE (PARAM_ALLOW_STORE_DATA_RACES) #endif /* ! GCC_PARAMS_H */ Index: testsuite/gcc.dg/20110509.c === --- testsuite/gcc.dg/20110509.c (revision 0) +++ testsuite/gcc.dg/20110509.c

Use expr_location_or in more cases

This patch changes fold_ternary_loc to use the new expr_location_or instead of the manual equivalent; no functional changes. Once this is done, grepping the file for UNKNOWN_LOCATION yields one occurrence (apart from expr_location_or): if ((lhs == 0 || rhs == 0 || operand_equal_p (lhs, rhs, 0

Re: [PATCH] Fix up typed DWARF stack support for POINTERS_EXTEND_UNSIGNED targets (PR debug/48853)

OK. Jason

Re: [C++0x] contiguous bitfields race implementation

On 05/09/11 11:26, Aldy Hernandez wrote: > >>> struct >>> { >>> unsigned int a : 4; >>> unsigned char b; >>> unsigned int c: 6; >>> } var; > > >> Well, the kernel guys would like to be able to be able to preserve the >> padding bits too. It's a long long sad story that I won't re

C++ PATCH for c++/48936 (failure with static constant template variable in sizeof)

This issue had to do with us bailing out too soon when trying to mark_used a constant variable in an unevaluated context. I fixed this for 4.6 as part of the constexpr work, but a subset of that patch seems suitable for backporting. Tested x86_64-pc-linux-gnu, applying to 4.4 and 4.5. commit

PING: [PATCH] Fix up typed DWARF stack support for POINTERS_EXTEND_UNSIGNED targets (PR debug/48853)

On Fri, May 6, 2011 at 6:11 PM, H.J. Lu wrote: > On Thu, May 5, 2011 at 2:20 AM, Jakub Jelinek wrote: >> Hi! >> >> My typed DWARF stack changes apparently broke ia64-hpux and H.J.'s out of >> tree x32 target.  There are several issues: >> 1) for SUBREG mem_loc_descriptor's 3rd argument was wrong,

Re: [PATCH, testsuite]: Add -mprefer-avx128 to DEFAULT_VECTCFLAGS

On Mon, May 9, 2011 at 9:44 AM, Uros Bizjak wrote: > Hello! > > Vectorizer testsuite assumes 128bit vector widths. AVX breaks this > assumption and that results in various spurious testsuite failures. > > Attached patch clears all failures on AVX target. > > 2011-05-09  Uros Bizjak   > >        *

Re: [C++ Patch] PRs 48737 & 48744

OK, thanks. Jason

[PATCH, i386]: Remove unneeded predicates

Hello! 2011-05-09 Uros Bizjak * config/i386/predicates.md (const_pow2_1_to_2_operand): Remove. (const_pow2_1_to_8_operand): Ditto. (const_pow2_1_to_128_operand): Ditto. (const_pow2_1_to_32768_operand): Ditto. * config/i386/mmx.md (*mmx_pinsrw): Use const

C++ PATCH for c++/34772 (warning about self-initialization without -Winit-self)

In this testcase -Wuninitialized was warning about 'int i = i' without -Winit-self because the C++ front end always uses separate code for non-constant initialization. But for simple initialization, it makes sense to use DECL_INITIAL. Tested x86_64-pc-linux-gnu, applying to trunk. commit b105

Re: [C++0x] contiguous bitfields race implementation

struct { unsigned int a : 4; unsigned char b; unsigned int c: 6; } var; Well, the kernel guys would like to be able to be able to preserve the padding bits too. It's a long long sad story that I won't repeat... And I don't think we should further complicate this stuff with th

Re: [C++0x] contiguous bitfields race implementation

On 05/09/11 10:24, Aldy Hernandez wrote: > Seeing that the current C++ draft has been approved, I'd like to submit > this for mainline, and get the proper review everyone's being quietly > avoiding :). > > To refresh everyone's memory, here is the problem: > > struct > { > unsigned int a : 4;

Re: [patch] make default linker --hash-style configurable option

On Mon, May 9, 2011 at 9:51 AM, Joseph S. Myers wrote: > On Mon, 9 May 2011, Paolo Bonzini wrote: > >> Uhm, so we deadlocked, I thought the other way.  I cannot really >> express any opinion about the desirability of the feature, but the >> configure syntax is certainly okay with me, and I gather

Re: [PATCH, MELT] fix useless forcing of GCC garbage collector

On Mon, 9 May 2011 19:03:54 +0200 Basile Starynkevitch wrote: > I applied the patch. The gcc/ChangeLog.MELT proposed by Pierre was > wrong, I wrote: > > 2011-05-09 Pierre Vittet > > * melt-runtime.c (melt_garbcoll): Don't force collection by > gcc_collect. > > Committed revision

Re: [PATCH, MELT] fix useless forcing of GCC garbage collector

On Mon, 09 May 2011 14:15:30 +0200 Pierre Vittet wrote: > This patch is for the MELT branch. The diff file was slightly wrong (diff run under gcc/). Pierre, you should run svn diff -x -p at the top source directory to get a patch file. I applied the patch. The gcc/ChangeLog.MELT proposed by Pie

Re: [PATCH, ARM] Unaligned accesses for packed structures [1/2]

On Fri, 06 May 2011 14:04:16 +0100 Richard Earnshaw wrote: > On Fri, 2011-05-06 at 13:34 +0100, Julian Brown wrote: > > This is the first of two patches to add unaligned-access support to > > the ARM backend. [...] > The compiler should fault -munaligned-access on cores that don't > support it.

Re: [patch] make default linker --hash-style configurable option

On Mon, 9 May 2011, Paolo Bonzini wrote: > On Mon, May 9, 2011 at 18:45, Joseph S. Myers wrote: > > On Mon, 9 May 2011, Paolo Bonzini wrote: > > > >> On 05/09/2011 05:59 PM, Paul Pluzhnikov wrote: > >> > Ping? Ping? Ping? Ping? Ping? > >> > > >> > http://gcc.gnu.org/ml/gcc-patches/2011-04/msg0024

Re: [patch] make default linker --hash-style configurable option

On Mon, May 9, 2011 at 18:45, Joseph S. Myers wrote: > On Mon, 9 May 2011, Paolo Bonzini wrote: > >> On 05/09/2011 05:59 PM, Paul Pluzhnikov wrote: >> > Ping? Ping? Ping? Ping? Ping? >> > >> > http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00246.html >> > >> > CC'ing the rest of build system maintai

Re: [patch] make default linker --hash-style configurable option

On Mon, 9 May 2011, Paolo Bonzini wrote: > On 05/09/2011 05:59 PM, Paul Pluzhnikov wrote: > > Ping? Ping? Ping? Ping? Ping? > > > > http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00246.html > > > > CC'ing the rest of build system maintainers. > > None of the build system maintainers can approve g

[PATCH, testsuite]: Add -mprefer-avx128 to DEFAULT_VECTCFLAGS

Hello! Vectorizer testsuite assumes 128bit vector widths. AVX breaks this assumption and that results in various spurious testsuite failures. Attached patch clears all failures on AVX target. 2011-05-09 Uros Bizjak * lib/target-supports.exp (check_vect_support_and_set_flags)

[C++0x] contiguous bitfields race implementation

Seeing that the current C++ draft has been approved, I'd like to submit this for mainline, and get the proper review everyone's being quietly avoiding :). To refresh everyone's memory, here is the problem: struct { unsigned int a : 4; unsigned char b; unsigned int c: 6; } var; vo

Re: [PATCH][ARM] Thumb2 replicated constants

On 06/05/11 12:18, Richard Earnshaw wrote: + RETURN_SEQUENCE must be an int[4]. It would be a more robust coding style to define a struct with an int[4] array as its only member. Then it wouldn't be possible to pass an undersized object to these routines. I've attached an updated patch with

Re: [patch] make default linker --hash-style configurable option

On 05/09/2011 05:59 PM, Paul Pluzhnikov wrote: Ping? Ping? Ping? Ping? Ping? http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00246.html CC'ing the rest of build system maintainers. None of the build system maintainers can approve gcc.c changes. And those can be approved only by either a global

Re: [patch] make default linker --hash-style configurable option

Ping? Ping? Ping? Ping? Ping? http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00246.html CC'ing the rest of build system maintainers. On Mon, May 2, 2011 at 8:56 AM, Paul Pluzhnikov wrote: > On Mon, May 2, 2011 at 7:59 AM, Joseph S. Myers > wrote: > > Thanks for your comments. > >> When pinging,

[v3] Fix libstdc++/48933

Hi, tested x86_64-linux, committed to mainline. Thanks, Paolo. /// 2011-05-09 Paolo Carlini PR libstdc++/48933 * include/c_global/cmath (acosh, asinh, atanh, cbrt, copysign, erf, erfc, exp2, expm1, fdim, fma, fmax, hypot, ilogb, lgamma, llrint

[PATCH][1/n] LTO type merging cleanup

This makes gimple_types_compatible_p private to gimple.c again, removing the need to call it from lto-symtab.c as we now merge types early (and thus we can resort to the middle-end types_compatible_p function). Bootstrapped on x86_64-unknown-linux-gnu, testing and SPEC2k6 build in progress. Rich

Ping Re: [PATCH] Canonicalize compares in combine [3/3] ARM backend part

Ping. On 04/22/2011 11:21 PM, Chung-Lin Tang wrote: > Hi Richard, this part's for you. > > The ARM backend changes needed are very little after the prior patches, > basically just a case in arm_canonicalize_comparison() to detect > (zero_extend:SI (subreg:QI (reg:SI ...) 0)), and swap it into (an

[Comitted] S/390: Fix TD/TF mem to reg splitter for -m31 -mzarch

Hi, the attached patch fixes a testsuite failure with -m31 -mzarch. The TD/TF mem to reg move splitter uses the first word mode register of the target operand as temporary register for storing the address. This generates an invalid address when having a 64 bit word mode and a 32 bit address mode

[PATCH, MELT] fix useless forcing of GCC garbage collector

This patch is for the MELT branch. My GCC contributor number is 634276. After speaking with Basile Starynkevitch, we saw that there might be a useless forcing of the garbage collector in melt-runtime.c . I tested with the patch, and I haven't seen any problem (I could compile GCC, using a MEL

Re: Fix thinko in var-tracking.c

On Mon, 9 May 2011, Eric Botcazou wrote: > doesn't affect code generation, I think it's appropriate for all branches. > It > seems to introduce a couple of XPASSes in the guality testsuite on i586: > > XPASS: gcc.dg/guality/inline-params.c -O2 -flto execution test > XPASS: gcc.dg/guality/pr4

Re: [PATCH, IRA]: Fix PR 48927 - Issues with "enable" attribute and IRA register preferences

On Sun, May 08, 2011 at 09:39:09PM +0200, Uros Bizjak wrote: > Hello! > > Attached patch fixes changed register allocation where "enabled" > attribute is used. The core of the problem was with IRA, where IRA > does not look at "enabled" attribute when scanning through > alternatives string to perf

[PATCH 11/n, i386]: Merge SSE and AVX patterns using "enable" attribute.

Hello! After PR 48972 fix, these two patterns can be merged without testsuite scan-asm failures. 2011-05-09 Uros Bizjak * config/i386/sse.md (*vec_concatv4si): Merge from *vec_concatv4si_1 and *vec_concatv4si_1_avx. Tested on x86_64-pc-linux-gnu {,-m32} AVX and non-AVX target

Re: [PATCH, IRA]: Fix PR 48927 - Issues with "enable" attribute and IRA register preferences

On 05/08/2011 09:39 PM, Uros Bizjak wrote: > Attached patch fixes changed register allocation where "enabled" > attribute is used. The core of the problem was with IRA, where IRA > does not look at "enabled" attribute when scanning through > alternatives string to perform various tasks (including

Re: [PATCH] allow setting LDFLAGS_FOR_TARGET in top-level configuration.

On 05/09/2011 11:49 AM, Joseph S. Myers wrote: > The x86 Android toolchain needs setting LDFLAGS_FOR_TARGET to > build. This patch does that. The patch was tested by bootstrapping > natively on x86_64 linux. Do I also need to submit this to binutils > as well? My view is that you don

Fix thinko in var-tracking.c

There is a known pitfall with REG_OFFSET/MEM_OFFSET: the former is an integer whereas the latter is a RTX. So a test like MEM_OFFSET (node->loc) == 0 is essentially never true for user MEMs since you have const0_rtx instead. The attached patch fixes 3 occurrences introduced in the VTA merge; si

[Patch,AVR]: Fix PR48896

Fixed the build warnings mentioned in PR48896. Johann -- 2011-05-09 Georg-Johann Lay PR target/48896 * config/avr/avr.c (avr_ret_register): Return unsigned int instead of int. (avr_function_value): Mark fn_decl_or_type as unused, don't pass it to avr_l

Re: [C++ Patch] PRs 48737 & 48744

Hi, On 05/08/2011 12:51 PM, Paolo Carlini wrote: @@ -5203,7 +5203,7 @@ reshape_init_r (tree type, reshape_iter *d, bool f { ++d->cur; gcc_assert (BRACE_ENCLOSED_INITIALIZER_P (init)); - return reshape_init (type, init); + return reshape_init (type

Re: [PATCH] allow setting LDFLAGS_FOR_TARGET in top-level configuration.

On Sun, 8 May 2011, Doug Kwan wrote: > Hi, > > The x86 Android toolchain needs setting LDFLAGS_FOR_TARGET to > build. This patch does that. The patch was tested by bootstrapping > natively on x86_64 linux. Do I also need to submit this to binutils > as well? My view is that you don't need

Re: Minor type merging optimization

> I haven't yet tested this (apart from with running the LTO testsuite), > but I'm going to give the following bootstrap & SPEC2k6 LTO build tests. > > Richard. > > 2011-05-09 Richard Guenther > > * gimple.c (gtc_visited, gtc_ob, struct type_pair_d, type_pair_t, > type_pair_hash,

Re: Minor type merging optimization

On Mon, 9 May 2011, Jan Hubicka wrote: > > > > > > I hacked quick unionfind two days ago, but the stopper is that the cache > > > is temporarily caching equilvaences in SCC regions. You mentioned it is > > > no > > > longer neccesary, so perhaps if you send me patch to remove this, I can > > >

Re: Minor type merging optimization

> > > > I hacked quick unionfind two days ago, but the stopper is that the cache > > is temporarily caching equilvaences in SCC regions. You mentioned it is no > > longer neccesary, so perhaps if you send me patch to remove this, I can give > > a try to this idea. > > I think we already do exact

Re: Minor type merging optimization

On Mon, 9 May 2011, Jan Hubicka wrote: > > On Fri, May 6, 2011 at 10:24 PM, Jan Hubicka wrote: > > > Hi, > > > while looking at type merging code I noticed that type pairs can be > > > managed > > > to be ordered by their UIDs.  This save some of hashing overhead in one of > > > most intensively

Re: Minor type merging optimization

On Mon, 9 May 2011, Jan Hubicka wrote: > > On Fri, May 6, 2011 at 10:24 PM, Jan Hubicka wrote: > > > Hi, > > > while looking at type merging code I noticed that type pairs can be > > > managed > > > to be ordered by their UIDs.  This save some of hashing overhead in one of > > > most intensively

Re: fix up hot/cold partitioning on ports that don't have long conditional branches

[Sorry for the delay] > > The patch contains long lines. > > Didn't know we had switched over to caring that much. Want me to fix all > of gcc/*.[ch]? Nope, just not introduce new long lines. > Ok? Almost. It looks like we now pass twice the same argument to follow_jumps, i.e. we have LABEL

Re: [pph] Allow streamer to define unique INTEGER_CST nodes (issue4489044)

On Fri, 6 May 2011, Diego Novillo wrote: > > I'm not altogether happy with this approach, so I'm looking for > suggestions on how to address this issue. > > The front end defines a bunch of unique constants in c_global_trees > that are not meant to be shared and are subject to pointer comparison

Re: Minor type merging optimization

> On Fri, May 6, 2011 at 10:24 PM, Jan Hubicka wrote: > > Hi, > > while looking at type merging code I noticed that type pairs can be managed > > to be ordered by their UIDs.  This save some of hashing overhead in one of > > most intensively querried hashes. > > > > Also gimple_lookup_type_leader

Re: [PATCH] allow setting LDFLAGS_FOR_TARGET in top-level configuration.

On 05/09/2011 04:38 AM, Doug Kwan (關振德) wrote: Sorry, forgot the patch and the ChangeLog 2011-05-08 Doug Kwan * configure.ac: Propagate LDFLAGS_FOR_TARGET. * configure: Regenerated. * Makefile.tpl (LDFLAGS_FOR_TARGET): Use LDFLAGS_FOR_TARGET value from configure