Re: Reduce complette unrolling & peeling limits

2012-11-23 Thread Jan Hubicka
> On Sun, 18 Nov 2012, Jan Hubicka wrote: > > > > > this patch reduces max-peeled-insns and max-completely-peeled-insns > > > > > from 400 > > > > > to 100. The reason why I am doing this is that I want to reduce code > > > > > bloat > > > > > caused by my cunroll work that enabled a lot more un

Re: patch to fix PR55122

2012-11-23 Thread H.J. Lu
On Wed, Nov 7, 2012 at 2:11 PM, Vladimir Makarov wrote: > The following patch fixes > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55122 > > The problem was in generation of reload pseudo for matching operands with > uniq value which prevented to assign the same hard register for the reload >

[C++ Patch] PR 55446

2012-11-23 Thread Paolo Carlini
Hi, this is a wrong code regression unfortunately caused by my fix for c++/45385. The story went (*) that in order to fix the latter missing warning we decided to remove the kludge added in c++/35602 to avoid a bogus warning *and* we also decided to early return from build_vec_init when maxi

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-23 Thread Hans-Peter Nilsson
On Thu, 22 Nov 2012, Kirill Yukhin wrote: > Hi folks, > > This patch adds an utility for dumping MD-files after iterators and > define_substs expanding. The new utility is named genmddump and is > built along with other gen* programs. > > I also added new target to Makefile to invoke this utility.

[patch] fix two multiarch issues

2012-11-23 Thread Matthias Klose
Fixing two multiarch issues: - the kfreebsd configuration gets the MULTILIB_OSDIRNAMES from the linux definition, which now has x32 defined. Just filter out the x32 definition, kfreebsd is plain 32/64 biarch. - the configure.ac used $withval instead of $enableval. Explicit --{en,dis}ab

Re: [Patch, libquadmath, committed] Updated the I/O related part of libquadmath

2012-11-23 Thread H.J. Lu
On Fri, Nov 23, 2012 at 4:59 PM, H.J. Lu wrote: > On Fri, Nov 23, 2012 at 3:38 PM, H.J. Lu wrote: >> On Fri, Nov 23, 2012 at 3:21 PM, H.J. Lu wrote: >>> On Fri, Nov 23, 2012 at 2:12 PM, Tobias Burnus wrote: As suggested by Joseph, it uses fegetround instead of trying to get the inform

Re: Fix twolf -funroll-loops -O3 miscompilation (a semi-latent web.c bug)

2012-11-23 Thread Steven Bosscher
On Fri, Nov 23, 2012 at 11:45 PM, Steven Bosscher wrote: > Removing the note is easier but it may hurt optimization later on: > CSE2 puts the note back in but this introduces a pass ordering > dependency. Perhaps that's not a big problem, but I'm not sure... Hmm, actually CSE2 fails to put the not

Re: [Patch, libquadmath, committed] Updated the I/O related part of libquadmath

2012-11-23 Thread H.J. Lu
On Fri, Nov 23, 2012 at 3:38 PM, H.J. Lu wrote: > On Fri, Nov 23, 2012 at 3:21 PM, H.J. Lu wrote: >> On Fri, Nov 23, 2012 at 2:12 PM, Tobias Burnus wrote: >>> As suggested by Joseph, it uses fegetround instead of trying to get the >>> information elsewhere (which glibc does to avoid mixing libm

Re: [ARM, PATCH] TARGET_LDRD reject Thumb1 targets

2012-11-23 Thread Ramana Radhakrishnan
On Wed, Nov 21, 2012 at 6:40 PM, Greta Yorsh wrote: > This patch adjusts the definition of TARGET_LDRD to false on Thumb1 targets, > as suggested here: > http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02048.html > > No regression on qemu for arm none-eabi with arch=armv5t/armv7-a > mode=thumb/arm. >

Re: [PATCH][ARM][2/2] ARMv8 aarch32 round to integral instructions

2012-11-23 Thread Ramana Radhakrishnan
On Wed, Nov 14, 2012 at 1:52 PM, Kyrylo Tkachov wrote: > Hi all, > > This patch adds the new tests for the vrint instructions in aarch32. > It also adds an effective target check to see if we support an ARMv8 VFP and > an add_options > procedure for adding the required options to a testcase if we

Re: [PATCH][ARM][1/2] ARMv8 aarch32 round to integral instructions

2012-11-23 Thread Ramana Radhakrishnan
> gcc/ChangeLog > > 2012-11-14 Kyrylo Tkachov > > * config/arm/arm.h (TARGET_FPU_ARMV8): New macro. > * config/arm/arm.md (UNSPEC_VRINTZ, UNSPEC_VRINTP, UNSPEC_VRINTM, > UNSPEC_VRINTR, UNSPEC_VRINTX, UNSPEC_VRINTA): New unspecs. > (f_rints, f_rintd): New types.

Re: [RFA/ARM] Fix PR54974: Thumb literal pools don't handle PC rounding

2012-11-23 Thread Ramana Radhakrishnan
On Wed, Nov 21, 2012 at 7:59 PM, Matthew Gretton-Dann wrote: > All, > > The attached patch fixes PR54974. > > In Thumb when calculating the PC value for a literal load the value used is > the current PC rounded down to the nearest multiple of 4. The ARM backend > currently does not take this into

Re: [PATCH, AArch64 4.7] Backport of __builtin_bswap16 optimisation

2012-11-23 Thread Marcus Shawcroft
On 23 Nov 2012, at 18:09, "Ian Bolton" wrote: I had already committed my testcase for this for aarch64, but it depends on this patch that doesn't yet exist in 4.7, so I backported to our ARM/aarch64-4.7-branch. Cheers, Ian This commit breaks the build, reverted. /Marcus

Re: [Patch, libquadmath, committed] Updated the I/O related part of libquadmath

2012-11-23 Thread H.J. Lu
On Fri, Nov 23, 2012 at 3:21 PM, H.J. Lu wrote: > On Fri, Nov 23, 2012 at 2:12 PM, Tobias Burnus wrote: >> As suggested by Joseph, it uses fegetround instead of trying to get the >> information elsewhere (which glibc does to avoid mixing libm with libc). >> >> Build and tested on x86-64-gnu-linux

Re: [Patch, libquadmath, committed] Updated the I/O related part of libquadmath

2012-11-23 Thread H.J. Lu
On Fri, Nov 23, 2012 at 2:12 PM, Tobias Burnus wrote: > As suggested by Joseph, it uses fegetround instead of trying to get the > information elsewhere (which glibc does to avoid mixing libm with libc). > > Build and tested on x86-64-gnu-linux. Committed as Rev. 193770. > > Tobias This caused: h

Re: [PING Updated]: [PATCH GCC/ARM] Fix problem that hardreg_cprop opportunities are missed on thumb1

2012-11-23 Thread Ramana Radhakrishnan
Ok if no regressions. Ramana On Wed, Oct 10, 2012 at 9:57 AM, Bin Cheng wrote: > Ping^2 > >> -Original Message- >> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] > On >> Behalf Of Bin Cheng >> Sent: Monday, October 08, 2012 2:36 PM >> To: gcc-patches@gcc.gnu.o

[patch] PR libstdc++/52680 define std::this_thread functions without --enable-libstdcxx-time

2012-11-23 Thread Jonathan Wakely
With this patch we define this_thread::yield() and this_thread::sleep_for and this_thread::sleep_until unconditionally, albeit with less functionality than when --enable-libstdcxx-time is used. This contains a #error so will fail to build on platforms that support std::thread but don't provide ::s

[committed] PATCH: PR sanitizer/55450: libtool: install: error: cannot install `libtsan.la' to a directory not ending in

2012-11-23 Thread H.J. Lu
/usr/lib/gcc/x86_64-pc-linux-gnu/ Reply-To: "H.J. Lu" Hi, I checked in this patch to fix PR sanitizer/55450. H.J. --- Index: ChangeLog === --- ChangeLog (revision 193766) +++ ChangeLog (working copy) @@ -1,12 +1,18 @@ +2012-11

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread Jakub Jelinek
On Fri, Nov 23, 2012 at 11:33:37AM -0800, H.J. Lu wrote: > 2012-11-21 H.J. Lu > > PR bootstrap/55380 > * charset.c (_cpp_convert_input): Clear CPP_PAD_BUFFER_SIZE > bytes if CLEAR_CPP_PAD_BUFFER isn't 0. Allocate extra > CPP_PAD_BUFFER_SIZE bytes and clear it if CLEAR_C

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread H.J. Lu
On Fri, Nov 23, 2012 at 11:23 AM, Uros Bizjak wrote: > On Fri, Nov 23, 2012 at 8:16 PM, H.J. Lu wrote: >> On Fri, Nov 23, 2012 at 10:59 AM, Uros Bizjak wrote: >>> Hello! >>> This patch allocates extra 16 bytes for -fsanitize=address so that asan won't report read beyond memory buffer.

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread Uros Bizjak
On Fri, Nov 23, 2012 at 8:16 PM, H.J. Lu wrote: > On Fri, Nov 23, 2012 at 10:59 AM, Uros Bizjak wrote: >> Hello! >> >>> This patch allocates extra 16 bytes for -fsanitize=address so that >>> asan won't report read beyond memory buffer. It is used by >>> bootstrap-asan. OK to install? >> >>/*

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread H.J. Lu
On Fri, Nov 23, 2012 at 10:59 AM, Uros Bizjak wrote: > Hello! > >> This patch allocates extra 16 bytes for -fsanitize=address so that >> asan won't report read beyond memory buffer. It is used by >> bootstrap-asan. OK to install? > >/* Resize buffer if we allocated substantially too much, or

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread Uros Bizjak
Hello! > This patch allocates extra 16 bytes for -fsanitize=address so that > asan won't report read beyond memory buffer. It is used by > bootstrap-asan. OK to install? /* Resize buffer if we allocated substantially too much, or if we - haven't enough space for the \n-terminator. */ +

Re: Reduce complette unrolling & peeling limits

2012-11-23 Thread Hans-Peter Nilsson
On Sun, 18 Nov 2012, Jan Hubicka wrote: > > > > this patch reduces max-peeled-insns and max-completely-peeled-insns > > > > from 400 > > > > to 100. The reason why I am doing this is that I want to reduce code > > > > bloat > > > > caused by my cunroll work that enabled a lot more unrolling then

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread H.J. Lu
On Fri, Nov 23, 2012 at 10:12 AM, Jakub Jelinek wrote: > On Fri, Nov 23, 2012 at 10:08:11AM -0800, H.J. Lu wrote: >> > to also change the caller, read_file_guts, where it does >> > buf = XNEWVEC (uchar, size + 1); >> > and >> > buf = XRESIZEVEC (uchar, buf, size + 1); >> >> I don't thi

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread Jakub Jelinek
On Fri, Nov 23, 2012 at 10:08:11AM -0800, H.J. Lu wrote: > > to also change the caller, read_file_guts, where it does > > buf = XNEWVEC (uchar, size + 1); > > and > > buf = XRESIZEVEC (uchar, buf, size + 1); > > I don't think it is necessary since there is no real data in > those extra

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread Konstantin Serebryany
On Fri, Nov 23, 2012 at 10:08 PM, H.J. Lu wrote: > On Fri, Nov 23, 2012 at 9:38 AM, Jakub Jelinek wrote: >> On Fri, Nov 23, 2012 at 09:23:37AM -0800, H.J. Lu wrote: >>> This patch allocates extra 16 bytes for -fsanitize=address so that >>> asan won't report read beyond memory buffer. It is used b

[PATCH, AArch64 4.7] Backport of __builtin_bswap16 optimisation

2012-11-23 Thread Ian Bolton
I had already committed my testcase for this for aarch64, but it depends on this patch that doesn't yet exist in 4.7, so I backported to our ARM/aarch64-4.7-branch. Cheers, Ian From: http://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=f811051bf87b1de7804c19c8192 d0d099d157145 diff --git a/gcc/Cha

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread H.J. Lu
On Fri, Nov 23, 2012 at 9:38 AM, Jakub Jelinek wrote: > On Fri, Nov 23, 2012 at 09:23:37AM -0800, H.J. Lu wrote: >> This patch allocates extra 16 bytes for -fsanitize=address so that >> asan won't report read beyond memory buffer. It is used by >> bootstrap-asan. OK to install? > > As valgrind wa

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread Jakub Jelinek
On Fri, Nov 23, 2012 at 09:23:37AM -0800, H.J. Lu wrote: > This patch allocates extra 16 bytes for -fsanitize=address so that > asan won't report read beyond memory buffer. It is used by > bootstrap-asan. OK to install? As valgrind warns about that too, I'd say we should do that unconditionally,

[PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-11-23 Thread H.J. Lu
Hi, This patch allocates extra 16 bytes for -fsanitize=address so that asan won't report read beyond memory buffer. It is used by bootstrap-asan. OK to install? Thanks. H.J. --- 2012-11-21 H.J. Lu PR bootstrap/55380 * charset.c (_cpp_convert_input): Allocate extra 16 bytes

Re: [PATCH][Revisedx5] Enable libsanitizer on darwin

2012-11-23 Thread Konstantin Serebryany
Looks great. (I am not an expert in the build system either, but the changes look trivial). Thanks! --kcc On Fri, Nov 23, 2012 at 8:29 PM, Alexander Potapenko wrote: > The mach_override path looks good to me. I don't have enough knowledge > of GCC buildsystem yet to review the rest. > > On Fri,

[Patch, committed] Remove obsolete asan remark from invoke.texi

2012-11-23 Thread Tobias Burnus
As suggested by Kostya. Committed as Rev. 193764. Tobias Index: gcc/ChangeLog === --- gcc/ChangeLog (Revision 193763) +++ gcc/ChangeLog (Arbeitskopie) @@ -1,3 +1,8 @@ +2012-11-23 Tobias Burnus + + * doc/invoke.texi (-fsanitize=ad

Re: [tsan] Instrument atomics

2012-11-23 Thread Xinliang David Li
On Fri, Nov 23, 2012 at 8:39 AM, Jakub Jelinek wrote: > On Fri, Nov 23, 2012 at 08:10:39PM +0400, Dmitry Vyukov wrote: >> > This patch attempts to instrument __atomic_* and __sync_* builtins. >> > Unfortunately for none of the builtins there is 1:1 mapping to the tsan >> > replacements, tsan uses

Re: [Patch, fortran] PR46897 - [OOP] type-bound defined ASSIGNMENT(=) not used for derived type component in intrinsic assign

2012-11-23 Thread Tobias Burnus
Dear Paul, thanks for the updated patch. While reading your patch, I was wondering whether the attached test case works or not. Result: It does *not* print "Hello World" with neither gfortran nor crayftn. If one changes in "m3" the declared type of "x" from "t" to "t2", it shows "Hello World

Re: [tsan] Instrument atomics

2012-11-23 Thread Jakub Jelinek
On Fri, Nov 23, 2012 at 08:10:39PM +0400, Dmitry Vyukov wrote: > > This patch attempts to instrument __atomic_* and __sync_* builtins. > > Unfortunately for none of the builtins there is 1:1 mapping to the tsan > > replacements, tsan uses weirdo memory model encoding (instead of values > > from 0 t

Re: [PATCH][Revisedx5] Enable libsanitizer on darwin

2012-11-23 Thread Alexander Potapenko
The mach_override path looks good to me. I don't have enough knowledge of GCC buildsystem yet to review the rest. On Fri, Nov 23, 2012 at 8:17 PM, Jack Howarth wrote: >The attached patch imports the missing mach_override/mach_override.h and > mach_override/mach_override.c files from llvm.org'

Re: [libsanitizer] merge fresh sources from upstream

2012-11-23 Thread Konstantin Serebryany
On Fri, Nov 23, 2012 at 8:07 PM, Jakub Jelinek wrote: > On Fri, Nov 23, 2012 at 06:47:06PM +0400, Konstantin Serebryany wrote: >> >> > Executing on host: addr2line -f -e >> >> > /usr/local/google/kcc/gcc-build/gcc/testsuite/gcc/memcmp-1.exe >> >> > 0x80488d1 0x8048560 (timeout = 300) >> >> > spa

[PATCH][Revisedx5] Enable libsanitizer on darwin

2012-11-23 Thread Jack Howarth
The attached patch imports the missing mach_override/mach_override.h and mach_override/mach_override.c files from llvm.org's compiler-rt at r168032 | glider | 2012-11-15 03:32:16 -0500 (Thu, 15 Nov 2012) | 3 lines [ASan] Add the "lea $imm(%rip),%rax" instruction to mach_override.c The need fo

Re: [C++ PATCH] Fix PR 55418

2012-11-23 Thread Paolo Carlini
... thus we could also do the below, for example. Actually I probably like it a tad better ;) Paolo. Index: method.c === --- method.c(revision 193758) +++ method.c(working copy) @@ -1175,6 +1175,7 @@ synthe

Re: [libsanitizer] merge fresh sources from upstream

2012-11-23 Thread Jakub Jelinek
On Fri, Nov 23, 2012 at 06:47:06PM +0400, Konstantin Serebryany wrote: > >> > Executing on host: addr2line -f -e > >> > /usr/local/google/kcc/gcc-build/gcc/testsuite/gcc/memcmp-1.exe > >> > 0x80488d1 0x8048560 (timeout = 300) > >> > spawn addr2line -f -e > >> > /usr/local/google/kcc/gcc-build/gcc

[patch] Add workaround for LEON 3 FP errata

2012-11-23 Thread Eric Botcazou
Hi, this adds a workaround (-mfix-ut699 switch) for the floating-point errata of the LEON 3 processor UT699. It's almost entirely implemented in the SPARC back-end, but there is a small change in expand_builtin_mathfn to make use of sqrt_optab with a larger mode if it isn't available in the or

Fix core's cost of stringops

2012-11-23 Thread Jan Hubicka
Hi, when SSE memcpy/memset code was reverted last stage3 (because of alignment handling bug), it went also with some unrelated changes, in particular retunning of stringop descriptors for the new glibc implementation that finally is sane for modern architectures. This patch partly restores the sec

Fix PR rtl-optimization/55388

2012-11-23 Thread Eric Botcazou
This is the recent bootstrap failure on SPARC/Linux --with-cpu=ultrasparc in release mode: cp/decl.c:copy_type_enum is miscompiled at -O2 after http://gcc.gnu.org/ml/gcc-cvs/2012-11/msg00546.html The patch has enabled the use of MEM_EXPR for bitfields. This was possible thanks to earlier chan

[PING, PATCH, ARM] Improved core -> NEON extend

2012-11-23 Thread Ulrich Weigand
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg00984.html Ping. Thanks, Ulrich -- Dr. Ulrich Weigand GNU Toolchain for Linux on System z and Cell BE ulrich.weig...@de.ibm.com

Re: patch to fix PR55430

2012-11-23 Thread Vladimir Makarov
On 12-11-23 7:42 AM, Uros Bizjak wrote: Hello! The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55430 The patch was successfully bootstrapped and tested on x86/x86-64. Committed as rev. 193742. 2012-11-22 Vladimir Makarov PR middle-end/55430

Re: [C++ PATCH] Fix PR 55418

2012-11-23 Thread Paolo Carlini
Hi, On 11/23/2012 04:47 PM, Jason Merrill wrote: On 11/23/2012 10:46 AM, Jason Merrill wrote: On 11/22/2012 11:03 AM, Paolo Carlini wrote: initialized to true too), but Markus (and me ;) proposes anyway to initialize trivial_p to false, thus we should be fine (way below trivial_p is used again

Re: [C++ PATCH] Fix PR 55418

2012-11-23 Thread Jason Merrill
On 11/23/2012 10:46 AM, Jason Merrill wrote: On 11/22/2012 11:03 AM, Paolo Carlini wrote: initialized to true too), but Markus (and me ;) proposes anyway to initialize trivial_p to false, thus we should be fine (way below trivial_p is used again but only when deleted_p is false). It certainly

Re: [C++ PATCH] Fix PR 55418

2012-11-23 Thread Jason Merrill
On 11/22/2012 11:03 AM, Paolo Carlini wrote: initialized to true too), but Markus (and me ;) proposes anyway to initialize trivial_p to false, thus we should be fine (way below trivial_p is used again but only when deleted_p is false). It certainly can't hurt. Jason

Re: [PATCH] Avoid some further -Wreturn-type false positives (PR c++/54046)

2012-11-23 Thread Jason Merrill
On 11/20/2012 03:31 PM, Jakub Jelinek wrote: +case THROW_EXPR: + return block_may_fallthru (TREE_OPERAND (stmt, 0)); Shouldn't this just be false? OK with that change. Jason

Re: patch to fix PR55430

2012-11-23 Thread Vladimir Makarov
On 12-11-23 7:42 AM, Uros Bizjak wrote: Hello! The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55430 The patch was successfully bootstrapped and tested on x86/x86-64. Committed as rev. 193742. 2012-11-22 Vladimir Makarov PR middle-end/55430

Re: [PATCH] Add PR55430 testcase (PR middle-end/55430)

2012-11-23 Thread Vladimir Makarov
On 12-11-23 9:44 AM, Jakub Jelinek wrote: Hi! On Thu, Nov 22, 2012 at 08:29:36PM -0500, Vladimir Makarov wrote: The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55430 The patch was successfully bootstrapped and tested on x86/x86-64. Committed as rev. 193742. 20

[PATCH] Add PR55430 testcase (PR middle-end/55430)

2012-11-23 Thread Jakub Jelinek
Hi! On Thu, Nov 22, 2012 at 08:29:36PM -0500, Vladimir Makarov wrote: > The following patch fixes > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55430 > > The patch was successfully bootstrapped and tested on x86/x86-64. > > Committed as rev. 193742. > > 2012-11-22 Vladimir Makarov >

Re: RFC: PATCH to add abi_tag attribute

2012-11-23 Thread Jason Merrill
On 11/23/2012 04:58 AM, Florian Weimer wrote: Okay, this might work in the sense that it flags the relevant cases. I'm still not convinced that this actually helps programmers that much because it pretty much separates the two worlds. If this is the intend, surely there are simpler approaches (s

Re: [libsanitizer] merge fresh sources from upstream

2012-11-23 Thread Jakub Jelinek
On Fri, Nov 23, 2012 at 06:12:24PM +0400, Konstantin Serebryany wrote: > Ok, the tests behave better on ubuntu 12.04. > tested: > rm -rf rm -rf */{*/,}libsanitizer && make -j20 && make -C gcc > check-g{cc,++} RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} > asan.exp' > > The fresh asan library pro

Re: [PATCH] Use working set profile info to determine hotness (issue6852069)

2012-11-23 Thread Jan Hubicka
> Sounds good. I am travelling the rest of the week so I'll get the revised > patch ready by Mon. Thanks, Teresa Hi, I updated the patch, so we make progress on the heuristic retunning. There was a segfault during profiledbootstrap trying to fetch DECL_STRUCT_FUNCTION of calle of indirect call and

[tsan] Instrument atomics

2012-11-23 Thread Jakub Jelinek
Hi! This patch attempts to instrument __atomic_* and __sync_* builtins. Unfortunately for none of the builtins there is 1:1 mapping to the tsan replacements, tsan uses weirdo memory model encoding (instead of values from 0 to 5 apparently 1 << 0 to 1 << 5, as if one could have more than one memory

Re: [PATCH] Use working set profile info to determine hotness (issue6852069)

2012-11-23 Thread Jan Hubicka
Hi, I went ahead, updated the patch, tested wth profiledbootstrap on x86_64-linux and commited. I really need to progress with heuristic re-tunning before we get too far in stage3. In addition to caching result of find_working_set I had to avoid ICE when we try to determine DECL_STRUCT_FUNCTION o

[tsan] Don't instrument clobber stmts, minor cleanups

2012-11-23 Thread Jakub Jelinek
Hi! var ={v} {CLOBBER}; stmts shouldn't be tsan instrumented, those aren't stores, just markups that var's scope ends. Additionally this patch removes the IMHO unneeded TODO_update_address_taken (discussed earlier already) and fixes formatting of a few comments. Ok for trunk? 2012-11-23 Jakub

[Patch,testsuite] ad PR52641: More fixes for not-so-common targets

2012-11-23 Thread Georg-Johann Lay
Here are some more fixes for 16-bit int and similar. Ok for trunk? Johann * gcc.dg/c1x-align-4.c: Skip avr. * gcc.dg/54455.c: Require scheduling. * gcc.dg/pr44024.c: Skip avr in final scan. PR testsuite/52641 * gcc.c-torture/execute/20120919-1.x: New file

Re: [wwwdocs] Mention -faddress-sanitizer in gcc-4.8/changes.html

2012-11-23 Thread Konstantin Serebryany
On Fri, Nov 23, 2012 at 5:30 PM, Tobias Burnus wrote: > Konstantin Serebryany wrote: >>> >>> >I think the man page should be then updated. >> >> man page? > > > I mean gcc/doc/invoke.texi, which is available as "man gcc" and also part of > the GCC Manual (http://gcc.gnu.org/onlinedocs/). It curren

Re: [wwwdocs] Mention -faddress-sanitizer in gcc-4.8/changes.html

2012-11-23 Thread Tobias Burnus
Konstantin Serebryany wrote: >I think the man page should be then updated. man page? I mean gcc/doc/invoke.texi, which is available as "man gcc" and also part of the GCC Manual (http://gcc.gnu.org/onlinedocs/). It currently contains: @item -fsanitize=address Enable AddressSanitizer, a fast

Re: [wwwdocs] Mention -faddress-sanitizer in gcc-4.8/changes.html

2012-11-23 Thread Konstantin Serebryany
On Fri, Nov 23, 2012 at 5:22 PM, Tobias Burnus wrote: > Konstantin Serebryany wrote: >> >> Looks good. > > > And now available at http://gcc.gnu.org/gcc-4.8/changes.html Cool! > > >>> Notes: I didn't mention Sparc, PowerPC, and Darwin >> >> Darwin works fine with clang, but not yet in gcc. > > >

Re: [wwwdocs] Mention -faddress-sanitizer in gcc-4.8/changes.html

2012-11-23 Thread Tobias Burnus
Konstantin Serebryany wrote: Looks good. And now available at http://gcc.gnu.org/gcc-4.8/changes.html Notes: I didn't mention Sparc, PowerPC, and Darwin Darwin works fine with clang, but not yet in gcc. I know – and actually it is a bit unclear to me what's the review status of Jack Howar

Re: [libsanitizer] merge fresh sources from upstream

2012-11-23 Thread Konstantin Serebryany
On Fri, Nov 23, 2012 at 4:47 PM, Jakub Jelinek wrote: > On Fri, Nov 23, 2012 at 04:35:29PM +0400, Konstantin Serebryany wrote: >> > Ok, provided it has been properly tested >> >> The upstream library is continuously tested on Linux, Mac, Windows and >> Android >> using the existing test suite (un

Re: [libsanitizer] merge fresh sources from upstream

2012-11-23 Thread Jakub Jelinek
On Fri, Nov 23, 2012 at 04:35:29PM +0400, Konstantin Serebryany wrote: > > Ok, provided it has been properly tested > > The upstream library is continuously tested on Linux, Mac, Windows and Android > using the existing test suite (unfortunately, the build bots are > private so far). Yeah, but on

Re: patch to fix PR55430

2012-11-23 Thread Uros Bizjak
Hello! >> The following patch fixes >> >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55430 >> >> The patch was successfully bootstrapped and tested on x86/x86-64. >> >> Committed as rev. 193742. >> >> 2012-11-22 Vladimir Makarov >> >> PR middle-end/55430 >> * lra.c: Move

Re: [libsanitizer] merge fresh sources from upstream

2012-11-23 Thread Konstantin Serebryany
On Fri, Nov 23, 2012 at 3:15 PM, Jakub Jelinek wrote: > Hi! > > Ok, provided it has been properly tested The upstream library is continuously tested on Linux, Mac, Windows and Android using the existing test suite (unfortunately, the build bots are private so far). > (given that we don't have >

Re: [wwwdocs] Mention -faddress-sanitizer in gcc-4.8/changes.html

2012-11-23 Thread Konstantin Serebryany
Looks good. On Fri, Nov 23, 2012 at 3:27 PM, Tobias Burnus wrote: > Konstantin Serebryany wrote: >> >> On Mon, Nov 19, 2012 at 10:44 PM, Tobias Burnus wrote: >>> >>> attached is a first draft for -faddress-sanitizer in the release notes. >> >> stack overflow is something different, I guess we wa

Re: [wwwdocs] Mention -faddress-sanitizer in gcc-4.8/changes.html

2012-11-23 Thread Tobias Burnus
Konstantin Serebryany wrote: On Mon, Nov 19, 2012 at 10:44 PM, Tobias Burnus wrote: attached is a first draft for -faddress-sanitizer in the release notes. stack overflow is something different, I guess we want to say "stack buffer overflow". I typically write something like "heap-, stack-, an

Re: [libsanitizer] merge fresh sources from upstream

2012-11-23 Thread Jakub Jelinek
Hi! Ok, provided it has been properly tested (given that we don't have almost any testsuite so far, that is at least rm -rf */{*/,}libsanitizer; make # -jN in the build tree and make -C gcc check-g{cc,++} RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} asan.exp' ). On Fri, Nov 23, 2012 at 03:03:2

[Ada] Fix ICE on 'Old attribute and discriminated record type

2012-11-23 Thread Eric Botcazou
This is an internal error on a postcondition applying the 'Old attribute to a parameter with discriminated record type. The problem is that S'Old ends up being rewritten into a local constrained variable very late in the game by the front-end and this yields a slightly skewed tree. Fixed thusl

[Ada] Fix layout of variant record types with aliased components

2012-11-23 Thread Eric Botcazou
The layout of variant record types doesn't always guarantee the proper alignment of aliased components if the record types are also packed. Fixed thusly, tested on x86_64-suse-linux, applied on the mainline. 2012-11-23 Eric Botcazou * gcc-interface/decl.c (components_need_strict_ali

Re: [Patch,build,ada]: Fix PR55243: Set STAMP

2012-11-23 Thread Georg-Johann Lay
Better attach the patch... Georg-Johann Lay wrote: > This patchlet fixes missing STAMP command line tool that is needed if gnat is > build for target avr. > > The patch is untested and as proposed in > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55243#c5 > > Ok for trunk and 4.7? > > Johann

[Patch,build,ada]: Fix PR55243: Set STAMP

2012-11-23 Thread Georg-Johann Lay
This patchlet fixes missing STAMP command line tool that is needed if gnat is build for target avr. The patch is untested and as proposed in http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55243#c5 Ok for trunk and 4.7? Johann PR ada/55243 * Makefile.in (TOOLS_FLAGS_TO_PASS_CROSS)

Re: RFC: PATCH to add abi_tag attribute

2012-11-23 Thread Florian Weimer
On 11/15/2012 02:51 AM, Jason Merrill wrote: On 11/11/2012 11:58 PM, Jason Merrill wrote: On 11/11/2012 08:01 AM, Florian Weimer wrote: I'm not sure if this sufficiently far-reaching. It seems that this doesn't allow me to implement a virtual function which takes a std::string parameter in new

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-23 Thread Mike Stump
On Nov 22, 2012, at 9:48 AM, Kirill Yukhin wrote: > This patch adds an utility for dumping MD-files after iterators and > define_substs expanding. > What do you guys think? Cool. (That's not an official review. :-))

Re: [Patch,testsuite] ad PR52641: Fix some more tests that fail for small targets

2012-11-23 Thread Mike Stump
On Nov 22, 2012, at 8:54 AM, Georg-Johann Lay wrote: > Some more skips for tests that will fail with 16-bit int etc. > > Ok for trunk? Ok.

Ping: RE: [RFC] New feature to reuse one multilib among different targets

2012-11-23 Thread Terry Guo
Hi Joseph, Can you please help to review this patch and share your thoughts on this feature? Thanks. BR, Terry > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Terry Guo > Sent: Tuesday, November 13, 2012 12:47 PM > To: jos