Re: [cxx-conversion] Convert tree-sra.c'candidates to hash_table

2012-12-12 Thread Lawrence Crowl
On 12/12/12, Diego Novillo wrote: > On Dec 11, 2012 Lawrence Crowl wrote: > > Convert tree-sra.c'candidates from htab_t to hash_table. > > > > Fold uid_decl_map_hash and uid_decl_map_eq into new struct > > uid_decl_hasher. This change moves the definitions from tree-ssa.c > > into tree-sra.c and

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread H.J. Lu
On Wed, Dec 12, 2012 at 2:13 PM, Paolo Bonzini wrote: > Il 12/12/2012 19:11, H.J. Lu ha scritto: >> >> in RAW_CXX_TARGET_EXPORTS. There is no need to do anything. >>> > >>> > Nope, if you remove this you get the wrong definition of CC and CXX from >>> > EXTRA_TARGET_FLAGS. Instead, you

Re: [patch] fix libstdc++/55631

2012-12-12 Thread Jonathan Wakely
On 11 December 2012 22:31, Jonathan Wakely wrote: > PR libstdc++/55631 > * include/ext/alloc_traits.h: Include missing header. > * include/ext/pointer.h: Likewise. > * include/ext/string_conversions.h: Require C++11. > * libsupc++/initializer_list: Reindent.

libgo patch committed: Update to current library

2012-12-12 Thread Ian Lance Taylor
I've updated libgo to the current master Go library sources. As usual this e-mail only includes patches to files that are substantially specific to gccgo. The bulk of the patch is available from the repository or from the master Go repository. Bootstrapped and ran Go testsuite on x86_64-unknown-

Re: [PATCH] Fix nonlocalized vars regression caused by vec changes (PR debug/55665)

2012-12-12 Thread Diego Novillo
On Wed, Dec 12, 2012 at 2:58 PM, Jakub Jelinek wrote: > 2012-12-12 Jakub Jelinek > > PR debug/55665 > * tree-inline.c (remap_decls): Change nonlocalized_list > to pointer to pointer to vector from pointer to vector. > (remap_block): Pass address of BLOCK_NONLOCA

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread H.J. Lu
On Wed, Dec 12, 2012 at 3:01 PM, H.J. Lu wrote: > On Wed, Dec 12, 2012 at 2:13 PM, Paolo Bonzini wrote: >> Il 12/12/2012 19:11, H.J. Lu ha scritto: > >> > >> in RAW_CXX_TARGET_EXPORTS. There is no need to do anything. > > Nope, if you remove this you get the wrong definition of

libgo patch committed: Delete from a nil map is now a no-op

2012-12-12 Thread Ian Lance Taylor
The Go spec was changed slightly: calling delete on a nil map is now a no-op, rather than causing a runtime panic. This patch implements that in gccgo. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: libgo/runtime/go-map-delete.c ===

Re: [v3] docbook vs. texlive > 2007

2012-12-12 Thread Benjamin De Kosnik
Fixup for a typo, on trunk and 4.7 -benjamin2012-12-12 Benjamin Kosnik * doc/xml/manual/documentation_hacking.xml: Fix validation issue. diff --git a/libstdc++-v3/doc/xml/manual/documentation_hacking.xml b/libstdc++-v3/doc/xml/manual/documentation_hacking.xml index 91d16dd..05c05a6 100644

Re: [cxx-conversion] Convert tree-sra.c'candidates to hash_table

2012-12-12 Thread Jakub Jelinek
On Tue, Dec 11, 2012 at 02:44:41PM -0800, Lawrence Crowl wrote: > +/* Hash a tree in a uid_decl_map. */ > + > +inline hashval_t > +uid_decl_hasher::hash (const value_type *item) > +{ > + return item->decl_minimal.uid; Ugh, why aren't you using DECL_UID here? The fact that DECL_UID is right now

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-12 Thread Xinliang David Li
On Wed, Dec 12, 2012 at 10:30 AM, Jan Hubicka wrote: > Concerning 1push per cycle, I think it is same as K7 hardware did, so move > prologue should be a win. >> > Index: config/i386/i386.c >> > === >> > --- config/i386/i386.c (revisi

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-12 Thread Xinliang David Li
On Wed, Dec 12, 2012 at 4:16 PM, Xinliang David Li wrote: > On Wed, Dec 12, 2012 at 10:30 AM, Jan Hubicka wrote: >> Concerning 1push per cycle, I think it is same as K7 hardware did, so move >> prologue should be a win. >>> > Index: config/i386/i386.c >>> > ===

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-12 Thread Jan Hubicka
> > libcall is not faster up to 8KB to rep sequence that is better for > > regalloc/code > > cache than fully blowin function call. > > Be careful with this. My recollection is that REP sequence is good for > any size -- for smaller size, the REP initial set up cost is too high > (10s of cycles),

[PATCH][Cilkplus] Check invalid gotos, increments and report errors

2012-12-12 Thread Iyer, Balaji V
Hello Everyone, This patch is for Cilk Plus branch affecting mainly the C compiler. It checks to see if the gotos and increment expressions inside cilk_for are valid and report errors otherwise. Thanks, Balaji V. Iyer. Index: gcc/c-family/c-cilk.c ===

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-12 Thread Xinliang David Li
On Wed, Dec 12, 2012 at 5:19 PM, Jan Hubicka wrote: >> > libcall is not faster up to 8KB to rep sequence that is better for >> > regalloc/code >> > cache than fully blowin function call. >> >> Be careful with this. My recollection is that REP sequence is good for >> any size -- for smaller size,

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-12 Thread Jakub Jelinek
On Wed, Dec 12, 2012 at 10:09:14PM -0800, Xinliang David Li wrote: > On Wed, Dec 12, 2012 at 5:19 PM, Jan Hubicka wrote: > >> > libcall is not faster up to 8KB to rep sequence that is better for > >> > regalloc/code > >> > cache than fully blowin function call. > >> > >> Be careful with this. My

Re: [PATCH] Make some asan builtins tm_pure (PR sanitizer/55508)

2012-12-12 Thread Dmitry Vyukov
On Wed, Dec 12, 2012 at 11:50 PM, Jakub Jelinek wrote: > Hi! > > Various TM tests ICE when built with -fgnu-tm -fsanitizer=address. > The problem is that asan.c pass adds calls to builtins that weren't there > before and TM is upset about it. The __asan_report* are all like > abort, in correctly

Re: [PATCH] Make some asan builtins tm_pure (PR sanitizer/55508)

2012-12-12 Thread Dmitry Vyukov
On Thu, Dec 13, 2012 at 1:58 AM, Richard Henderson wrote: > On 12/12/2012 11:50 AM, Jakub Jelinek wrote: >> 2012-12-12 Jakub Jelinek >> >> PR sanitizer/55508 >> * builtin-attrs.def (ATTR_TMPURE_NOTHROW_LEAF_LIST, >> ATTR_TMPURE_NORETURN_NOTHROW_LEAF_LIST): New. >> * asan

Re: [PATCH i386]: Enable push/pop in pro/epilogue for modern CPUs

2012-12-12 Thread Xinliang David Li
Try the following one. 1) -minline-all-stringops -mstringop-strategy=rep_8byte -O2 vs 1) -mstringop_strategy=libcall -O2. David #include #include #include #ifndef LEN #define LEN 16 #endif void copy(char* s1, char* s2,int len) __attribute__((noinline)); void copy(char* s1, char* s2,int len)

Re: PATCH: Remove AM_MAKEFLAGS from libsanitizer

2012-12-12 Thread Paolo Bonzini
Il 13/12/2012 00:23, H.J. Lu ha scritto: > On Wed, Dec 12, 2012 at 3:01 PM, H.J. Lu wrote: >> On Wed, Dec 12, 2012 at 2:13 PM, Paolo Bonzini wrote: >>> Il 12/12/2012 19:11, H.J. Lu ha scritto: in RAW_CXX_TARGET_EXPORTS. There is no need to do anything. >> >> Nope, if yo

Re: [PATCH] asan unit tests from llvm lit-test incremental changes

2012-12-12 Thread Konstantin Serebryany
On Thu, Dec 13, 2012 at 1:30 AM, Jakub Jelinek wrote: > On Wed, Dec 12, 2012 at 10:16:49PM +0100, Dodji Seketeli wrote: >> Independently of this review, I think it's be interesting to hear >> Kostya's voice on: >> >> Jakub Jelinek writes: >> >> > 2) In large-func-test-1.C, I had to stop matching

<    1   2