Re: CRIS atomics revisited 4/4: give up on alignment of atomic data

2012-07-16 Thread Hans-Peter Nilsson
From: Hans-Peter Nilsson h...@axis.com Date: Mon, 16 Jul 2012 05:49:00 +0200 gcc: * config/cris/sync.md (atomic_fetch_atomic_op_namemode) (cris_atomic_fetch_atomic_op_namemode_1) (atomic_compare_and_swapmode) (cris_atomic_compare_and_swapmode_1): Make

Re: [PATCH][MIPS] NetLogic XLP scheduling

2012-07-16 Thread Chung-Lin Tang
On 2012/7/16 12:28 AM, Richard Sandiford wrote: Chung-Lin Tang clt...@codesourcery.com writes: This patch adds scheduling support for the NetLogic XLP, including a new pipeline description, and associated changes. Asides from the new xlp.md description file, there are also some sync

Re: [PATCH][MIPS] NetLogic XLP scheduling

2012-07-16 Thread Maxim Kuvyrkov
On 16/07/2012, at 6:37 PM, Chung-Lin Tang wrote: On 2012/7/16 12:28 AM, Richard Sandiford wrote: Chung-Lin Tang clt...@codesourcery.com writes: This patch adds scheduling support for the NetLogic XLP, including a new pipeline description, and associated changes. Asides from the new xlp.md

[Committed] Add new testcase

2012-07-16 Thread Andrew Pinski
Hi, When I was debugging a modified GCC 4.3 I ran into an ICE which was fixed by http://gcc.gnu.org/ml/gcc-patches/2008-10/msg00217.html . The patch did not mention it was fixing an ICE, so I decided to add a testcase. Committed the testcase as obvious. Thanks, Andrew Pinski *

Re: [patch] PR web/53919 - Add note to install.texi

2012-07-16 Thread Jonathan Wakely
On 16 July 2012 01:14, Gerald Pfeifer wrote: Hi Jonathan, On Sat, 14 Jul 2012, Jonathan Wakely wrote: Attached this time, here's the original mail again: PR c++/53919 * doc/install.texi (Installing GCC): Refer to instructions for released versions. Fix hyphenation.

Re: PATCH: PR bootstrap/53963: [4.8 Regression] LTO bootstrap failed with bootstrap-profiled

2012-07-16 Thread Richard Guenther
On Sat, Jul 14, 2012 at 8:49 PM, H.J. Lu hongjiu...@intel.com wrote: Hi, Revision 189453: http://gcc.gnu.org/ml/gcc-cvs/2012-07/msg00357.html removes type attribute comparison in gimple_types_compatible_p_1 and caused PR middle-end/53959 and PR bootstrap/53963. This restores type

Re: [PATCH] Enable vectorizer cost model by default at -O3

2012-07-16 Thread Richard Guenther
On Sun, 15 Jul 2012, William J. Schmidt wrote: The auto-vectorizer is overly aggressive when not constrained by the vectorizer cost model. Although the cost model is by no means perfect, it does a reasonable job of avoiding many poor vectorization decisions. Since the auto-vectorizer is

Re: [patch] PR web/53919 - Add note to install.texi

2012-07-16 Thread Gerald Pfeifer
On Mon, 16 Jul 2012, Jonathan Wakely wrote: Thanks, revised patch attached. OK for trunk like this? Oh, yes! I meant to imply this in my previous message. Thanks, Gerald

Re: [patch] Remove useless sets/clears of EDGE_CAN_FALLTHRU

2012-07-16 Thread Richard Guenther
On Sat, Jul 14, 2012 at 11:40 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, EDGE_CAN_FALLTHRU is internal to bb-reorder.c, set_edge_can_fallthru_flag() clears it on all edges and computes it fresh for bb-reorder. The test in rtl_verify_flow_info_1 has to stay because bb-reorder

Re: [patch] Introduce dumpfile.h, clean up tree-pass.h dependencies

2012-07-16 Thread Richard Guenther
On Fri, Jul 13, 2012 at 3:38 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, The attached monster patch re-organizes a lot of includes to avoid dependencies on tree-pass.h just for having dump_file available. I've used the following rules to decide what needs to go where: *

Re: [patch] Remove useless sets/clears of EDGE_CAN_FALLTHRU

2012-07-16 Thread Steven Bosscher
On Mon, Jul 16, 2012 at 10:53 AM, Richard Guenther richard.guent...@gmail.com wrote: Maybe we should rename EDGE_CAN_FALLTHRU to EDGE_PLF_1 and have a #define in bb-reorder.c then? Other passes might use it for their pass-local handling then and would not be content to set/unset it. I intend

Re: [PATCH][MIPS] NetLogic XLP scheduling

2012-07-16 Thread Richard Sandiford
Chung-Lin Tang clt...@codesourcery.com writes: Other generic changes include a new hilo insn attribute, to mark which of HI/LO does a m[ft]hilo insn access. The way other schedulers handle this is with things like: (define_insn_reservation ir_sb1_mfhi 1 (and (eq_attr cpu sb1,sb1a)

Re: [patch] Call free_after_parsing earlier

2012-07-16 Thread Richard Guenther
On Fri, Jul 13, 2012 at 4:33 PM, Steven Bosscher stevenb@gmail.com wrote: On Thu, Jul 12, 2012 at 9:21 AM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Jul 11, 2012 at 9:39 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, GCC calls free_after_parsing in

Re: [PATCH][MIPS] NetLogic XLP scheduling

2012-07-16 Thread Richard Sandiford
Maxim Kuvyrkov ma...@codesourcery.com writes: On 16/07/2012, at 6:37 PM, Chung-Lin Tang wrote: On 2012/7/16 12:28 AM, Richard Sandiford wrote: Chung-Lin Tang clt...@codesourcery.com writes: Tested on mips64-elf, and by making sure that there were no changes in -O2 output for a recent set of

Re: Fix PR53908

2012-07-16 Thread Richard Guenther
On Fri, Jul 13, 2012 at 11:49 PM, Steven Bosscher stevenb@gmail.com wrote: On Fri, Jul 13, 2012 at 10:56 AM, Hans-Peter Nilsson hans-peter.nils...@axis.com wrote: From: Richard Guenther richard.guent...@gmail.com Date: Fri, 13 Jul 2012 10:08:05 +0200 On Fri, Jul 13, 2012 at 9:59 AM,

[Patch, Fortran, F03] PROCEDURE w/ interface: Bogus EXTERNAL attribute conflicts with FUNCTION attribute

2012-07-16 Thread Janus Weil
Hi all, the attached patch is the solution that I came up with for this PR (after two simpler but wrong attempts, see PR). The issue addressed here is the setting of the if_source attribute for dummy procedures. When declaring a procedure (pointer) with the PROCEDURE statement, the attributes of

Re: [patch] Introduce dumpfile.h, clean up tree-pass.h dependencies

2012-07-16 Thread Steven Bosscher
On Mon, Jul 16, 2012 at 11:00 AM, Richard Guenther richard.guent...@gmail.com wrote: On Fri, Jul 13, 2012 at 3:38 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, The attached monster patch re-organizes a lot of includes to avoid dependencies on tree-pass.h just for having dump_file

Re: [Patch, Fortran, F03] PROCEDURE w/ interface: Bogus EXTERNAL attribute conflicts with FUNCTION attribute

2012-07-16 Thread Tobias Burnus
On 07/16/2012 11:09 AM, Janus Weil wrote: the attached patch is the solution that I came up with for this PR (after two simpler but wrong attempts, see PR). The issue addressed here is the setting of the if_source attribute for dummy procedures. When declaring a procedure (pointer) with the

Re: [patch] Remove expand_expr_stmt

2012-07-16 Thread Richard Guenther
On Sat, Jul 14, 2012 at 7:34 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, This patch removes expand_expr_stmt and inlines+folds it into its only user. This relies on TARGET_STACK_PROTECT_FAIL being a CALL_EXPR. The documentation only says a tree expression that (...) should

Re: [patch] Bail out if the same label is emitted twice

2012-07-16 Thread Richard Guenther
On Sat, Jul 14, 2012 at 9:25 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, Just another small cleanup. This probably made sense before function-at-a-time became the norm, 10 years ago. Bootstrappedtested on powerpc64-unknown-linux-gnu and x86_64-unknown-linux-gnu. OK for trunk?

Re: [patch] Remove useless sets/clears of EDGE_CAN_FALLTHRU

2012-07-16 Thread Richard Guenther
On Mon, Jul 16, 2012 at 11:03 AM, Steven Bosscher stevenb@gmail.com wrote: On Mon, Jul 16, 2012 at 10:53 AM, Richard Guenther richard.guent...@gmail.com wrote: Maybe we should rename EDGE_CAN_FALLTHRU to EDGE_PLF_1 and have a #define in bb-reorder.c then? Other passes might use it for

Re: [PATCH 1/6] Thread pointer built-in functions, core parts

2012-07-16 Thread Chung-Lin Tang
On 12/7/14 9:58 AM, Mike Stump wrote: On Jul 12, 2012, at 11:47 PM, Chung-Lin Tang clt...@codesourcery.com wrote: and then for the return value, maybe a const0_rtx for Pmode. A little unsure what you mean. Are you referring to return const0_rtx for default_expand_builtin_thread_pointer()

Re: [PATCH 1/2] gcc symbol database

2012-07-16 Thread Yunfeng ZHANG
But the meaning of the macro_end_arg event is really not clear to ... and so on. Let's see a sample: #define Z(a) a #define Y Z #define X(p) p + Y X(1)(2); With my solution, user get 1) `X' -- leader macro token by macro_start_expand. 2) `(', `1', `)', `(', `2',

Re: [Patch, Fortran, F03] PROCEDURE w/ interface: Bogus EXTERNAL attribute conflicts with FUNCTION attribute

2012-07-16 Thread Janus Weil
the attached patch is the solution that I came up with for this PR (after two simpler but wrong attempts, see PR). The issue addressed here is the setting of the if_source attribute for dummy procedures. When declaring a procedure (pointer) with the PROCEDURE statement, the attributes of the

[Ada] Suppress false elaboration warnings -- indirect Elaborate_All

2012-07-16 Thread Arnaud Charlet
This patch suppresses certain false-alarm warnings about elaboration in cases where a pragma Elaborate_All is not present directly, but is found in some indirectly-with'ed unit. The following test should compile silently: gnatmake -q -f -g -gnatwl -gnatE -gnat05 r.adb package P is function F

Re: [patch] Introduce dumpfile.h, clean up tree-pass.h dependencies

2012-07-16 Thread Richard Guenther
On Mon, Jul 16, 2012 at 11:12 AM, Steven Bosscher stevenb@gmail.com wrote: On Mon, Jul 16, 2012 at 11:00 AM, Richard Guenther richard.guent...@gmail.com wrote: On Fri, Jul 13, 2012 at 3:38 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, The attached monster patch re-organizes a

Re: [patch] Introduce dumpfile.h, clean up tree-pass.h dependencies

2012-07-16 Thread Steven Bosscher
On Mon, Jul 16, 2012 at 1:49 PM, Richard Guenther richard.guent...@gmail.com wrote: On Mon, Jul 16, 2012 at 11:12 AM, Steven Bosscher stevenb@gmail.com wrote: On Mon, Jul 16, 2012 at 11:00 AM, Richard Guenther richard.guent...@gmail.com wrote: On Fri, Jul 13, 2012 at 3:38 PM, Steven

[Ada] Add support for encoding in Create_Directory and Create_Path.

2012-07-16 Thread Arnaud Charlet
Tested on x86_64-pc-linux-gnu, committed on trunk 2012-07-16 Pascal Obry o...@adacore.com * s-crtl.ads (mkdir): New routine, support encoding. * adaint.h (__gnat_mkdir): Update spec to pass encoding. * mkdir.c (__gnat_mkdir): Add encoding parameter. *

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

2012-07-16 Thread Rainer Emrich
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I'm not sure if that's related, but for x86_64-w64-mingw32 bootstrap fails in stage 1: gcc -c -g -DIN_GCC -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual - -Wstrict-prototypes -Wmissing-prototypes -Wold-style-definition -Wc++-compat -

[Ada] Introduce the notion of machine occurrence

2012-07-16 Thread Arnaud Charlet
This is preliminary work to have a better support of foreign exceptions and of Windows 64 SEH. This patch introduce the notion of machine occurrence, which is the system exception propagated. This occurrence is allocated before setting up the Ada occurrence, which slightly speed-up occurrence

[Ada] Improve support of Storage_Error for Windows 64 SEH

2012-07-16 Thread Arnaud Charlet
Work to support Windows 64 SEH: allow to convert an exception code to an Ada exception ID and message; Optimize the propagation of Storage_Error on Windows 64 SEH to avoid requiring a too large stack area. 2012-07-16 Tristan Gingold ging...@adacore.com * seh_init.c (__gnat_map_SEH):

[Ada] Explicitly pass exception occurrence to notifiers

2012-07-16 Thread Arnaud Charlet
This is an internal cleanup and preliminary to removal of useless ada occurrence copy. No functional change. Tested on x86_64-pc-linux-gnu, committed on trunk 2012-07-16 Tristan Gingold ging...@adacore.com * a-exexpr.adb (Propagate_Exception): Adjust call to Exception_Traces

[Ada] Cleanup of selective accept expansion

2012-07-16 Thread Arnaud Charlet
This change reorganizes code generation for selective accept statements so that the expanded tree is well formed. This cleanup is desirable in preparation for further changes related to coverage analysis. No observable change in generated code's behaviour. Tested on x86_64-pc-linux-gnu,

[Ada] Coverage analysis of select statements

2012-07-16 Thread Arnaud Charlet
This change introduces generation of a null statement in alternatives appearing in various forms of select statement, when they do not have trailing statements. The purpose of the additional null statement is to allow a nop to be generated with an appropriate source location indication in debug

[Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Arnaud Charlet
The natural way to import a builtin that returns void * is to use System.Address in Ada, which is in fact an integral type. Addressed by this patch, which makes it possible to e.g. compile: with System; procedure Btins1 is function Frame_Address (Level : Integer) return System.Address;

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

2012-07-16 Thread H.J. Lu
On Fri, Jul 6, 2012 at 9:08 PM, H.J. Lu hongjiu...@intel.com wrote: Hi, After inlining, IPA calls tree_profiling which rebuilds edges: #2 0x00683ccd in rebuild_cgraph_edges () at /export/gnu/import/git/gcc/gcc/cgraphbuild.c:433 #3 0x00c02de6 in tree_profiling ()

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Duncan Sands
Hi Arnaud, The natural way to import a builtin that returns void * is to use System.Address in Ada, which is in fact an integral type. how about doing this for formal arguments too and not just the return type? This would improve optimization by LLVM of calls to standard library functions

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 3:16 PM, Duncan Sands wrote: Hi Arnaud, The natural way to import a builtin that returns void * is to use System.Address in Ada, which is in fact an integral type. how about doing this for formal arguments too and not just the return type? Formal arguments were

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Duncan Sands
Hi Tristan, On 16/07/12 15:17, Tristan Gingold wrote: On Jul 16, 2012, at 3:16 PM, Duncan Sands wrote: Hi Arnaud, The natural way to import a builtin that returns void * is to use System.Address in Ada, which is in fact an integral type. how about doing this for formal arguments too and

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

2012-07-16 Thread Kai Tietz
Hi, I would kindly ask to revert this patch soonish. The define OPTION_BIONIC is defined within linux.h header, which isn't used by cygwin and mingw targets. so this symbol is undefined for them and this is causing bootstrap issue for cygwin and mingw targets. Regards, Kai

[PATCH, ARM] Fix length attributes for sync.md patterns

2012-07-16 Thread Ulrich Weigand
Hello, when testing an out-of-tree patch I ran into a latent bug. The symptom is error messages along the lines of /tmp/cc6q0E3x.s:38: Error: co-processor offset out of range caused by an out-of-range reference to a literal pool constant. This happens only with -O0. This turns out to caused by

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 3:24 PM, Duncan Sands wrote: Hi Tristan, On 16/07/12 15:17, Tristan Gingold wrote: On Jul 16, 2012, at 3:16 PM, Duncan Sands wrote: Hi Arnaud, The natural way to import a builtin that returns void * is to use System.Address in Ada, which is in fact an integral

[RFC] 4.7 backport crashes (was: Re: Tree tail merging breaks __builtin_unreachable optimization)

2012-07-16 Thread Ulrich Weigand
Richard Guenther wrote: On Fri, Jul 6, 2012 at 6:36 PM, Tom de Vries tom_devr...@mentor.com wrote: Bootstrapped and reg-tested (ada inclusive) on x86_64. OK for trunk? Ok. Thanks, Richard. 2012-07-06 Tom de Vries t...@codesourcery.com Richard Guenther

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Duncan Sands
Hi Tristan, indeed, for two years already. Is there any reason not to do this for all functions, rather than just limiting it to builtins? I don't understand what do you mean. We need to do this implicit conversion for builtins because they are known by the compiler. Which other functions

Re: Allow use of ranges in copyright notices

2012-07-16 Thread Joseph S. Myers
I have now committed this patch to allow the use of ranges. -- Joseph S. Myers jos...@codesourcery.com

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

2012-07-16 Thread Kirill Yukhin
Reverted. http://gcc.gnu.org/ml/gcc-cvs/2012-07/msg00442.html Thanks, K On Mon, Jul 16, 2012 at 5:36 PM, Kai Tietz ktiet...@googlemail.com wrote: Hi, I would kindly ask to revert this patch soonish. The define OPTION_BIONIC is defined within linux.h header, which isn't used by cygwin and

Re: [RFC] 4.7 backport crashes (was: Re: Tree tail merging breaks __builtin_unreachable optimization)

2012-07-16 Thread Richard Guenther
On Mon, Jul 16, 2012 at 3:55 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Richard Guenther wrote: On Fri, Jul 6, 2012 at 6:36 PM, Tom de Vries tom_devr...@mentor.com wrote: Bootstrapped and reg-tested (ada inclusive) on x86_64. OK for trunk? Ok. Thanks, Richard. 2012-07-06 Tom de

[PATCH, i386, PR53877] New intrinsics for LZCNT

2012-07-16 Thread Kirill Yukhin
Hello guys, Here is a tiny patch which adds two new intrinsics which were introduced in recent spec [1]. They're aliased to the existing __lzcnt_* and live under same CPUID. ChangeLog entry is: 2012-07-16 Kirill Yukhin kirill.yuk...@intel.com PR target/53877 *

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 3:58 PM, Duncan Sands wrote: Hi Tristan, indeed, for two years already. Is there any reason not to do this for all functions, rather than just limiting it to builtins? I don't understand what do you mean. We need to do this implicit conversion for builtins because

Re: [PATCH, i386, PR53877] New intrinsics for LZCNT

2012-07-16 Thread H.J. Lu
On Mon, Jul 16, 2012 at 7:14 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello guys, Here is a tiny patch which adds two new intrinsics which were introduced in recent spec [1]. They're aliased to the existing __lzcnt_* and live under same CPUID. ChangeLog entry is: 2012-07-16

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Duncan Sands
Hi Tristan, Ah, what you want is the use of 'void *' for System.Address. We didn't choose that because the semantic of System.Address (which includes arithmetic on the whole address space) doesn't match the void * one. void* arithmetic of this kind exists, it's a gcc extension to C :) But,

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 4:36 PM, Duncan Sands wrote: Hi Tristan, Ah, what you want is the use of 'void *' for System.Address. We didn't choose that because the semantic of System.Address (which includes arithmetic on the whole address space) doesn't match the void * one. void* arithmetic

[PATCH] Fix up _xabort for -O0

2012-07-16 Thread Jakub Jelinek
Hi! While backporting, I've noticed there is an extra semicolon that shouldn't be there in -O0 version of _xabort. Ok for trunk? 2012-07-16 Jakub Jelinek ja...@redhat.com * config/i386/rtmintrin.h (_xabort): Remove trailing semicolon. --- gcc/config/i386/rtmintrin.h 2012-03-14

Re: [PATCH] Fix up _xabort for -O0

2012-07-16 Thread Uros Bizjak
On Mon, Jul 16, 2012 at 4:45 PM, Jakub Jelinek ja...@redhat.com wrote: While backporting, I've noticed there is an extra semicolon that shouldn't be there in -O0 version of _xabort. Ok for trunk? 2012-07-16 Jakub Jelinek ja...@redhat.com * config/i386/rtmintrin.h (_xabort):

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Duncan Sands
Hi Tristan, Ah, what you want is the use of 'void *' for System.Address. We didn't choose that because the semantic of System.Address (which includes arithmetic on the whole address space) doesn't match the void * one. void* arithmetic of this kind exists, it's a gcc extension to C :) The

Re: [PATCH, i386, PR53877] New intrinsics for LZCNT

2012-07-16 Thread Uros Bizjak
On Mon, Jul 16, 2012 at 4:32 PM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Jul 16, 2012 at 7:14 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello guys, Here is a tiny patch which adds two new intrinsics which were introduced in recent spec [1]. They're aliased to the existing

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Duncan Sands
PS: That said, I have to admit that using void* for builtins does cover the most important cases.

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Tristan Gingold
On Jul 16, 2012, at 5:16 PM, Duncan Sands wrote: PS: That said, I have to admit that using void* for builtins does cover the most important cases. Agreed. But I still think that these are two very different issues: * Doing implicit conversion to void * for builtins is just a convenient way

Re: [PATCH 1/2] gcc symbol database

2012-07-16 Thread Dodji Seketeli
Yunfeng ZHANG zyf.zer...@gmail.com writes: But the meaning of the macro_end_arg event is really not clear to ... and so on. Let's see a sample: #define Z(a) a #define Y Z #define X(p) p + Y X(1)(2); With my solution, user get 1) `X' -- leader macro token by

Re: G++ namespace association extension

2012-07-16 Thread Jason Merrill
On 07/15/2012 07:26 AM, Gerald Pfeifer wrote: I think this also should go into the GCC 4.8 release notes (gcc-4.8/changes.html)? I doubt that it has gotten wide enough usage to need that, but I suppose it doesn't hurt. Jason

Re: [Ada] Ease interface with builtins that returns void *

2012-07-16 Thread Eric Botcazou
Agreed. But I still think that these are two very different issues: * Doing implicit conversion to void * for builtins is just a convenient way for interfacing with builtins. * Using void * for System.Address shouldn't have any immediate advantage for users, but would be a better match

Re: [patch][rft] Don't mess with debug_hooks in objc

2012-07-16 Thread Mike Stump
On Jul 14, 2012, at 12:21 PM, Steven Bosscher wrote: There is code in objc-act.c to save and restore debug hooks to work around a problem with the C front end. OK for trunk? Ok. [ Thanks for waiting, I wanted to ponder this for a day or two... ]

Re: [Ada] Optimization of the Lock-free implementation of protected object

2012-07-16 Thread Iain Sandoe
Hello Arnaud, On 12 Jul 2012, at 11:43, Arnaud Charlet wrote: The optimization of the expansion of protected procedure for the lock-free implementation brings the following changes: - Several renamings in order to match GCC built-in function wordings. - Expected_Comp declaration moved to the

[Committed] Add some new testcases

2012-07-16 Thread Andrew Pinski
Hi, This is just a small collection of testcases which we (Cavium) had locally. gcc.c-torture/execute/bswap-1.c tests __builtin_bswap64 as there was no testcases before gcc.c-torture/compile/unalign-1.c ICEd when movmisalign* patterns were added to mips. gcc.c-torture/compile/20120524-1.c ICEd

[spu, commit] Fix build failure - move targetm initializer to the bottom

2012-07-16 Thread Ulrich Weigand
Hello, since Bill Schmidt's recent vector cost function patch, spu.c fails to build: /home/kwerner/dailybuild/spu-tc-2012-07-16/gcc-head/src/gcc/config/spu/spu.c:499: error: 'spu_init_cost' undeclared here (not in a function)

Re: [Ada] Optimization of the Lock-free implementation of protected object

2012-07-16 Thread Arnaud Charlet
http://gcc.gnu.org/ml/gcc-cvs/2012-07/msg00341.html This breaks ada bootstrap on powerpc-darwin9 with : Undefined symbols: ___sync_val_compare_and_swap_8, referenced from: _system__atomic_primitives__lock_free_try_write_64 in s-atopri.o ___atomic_load_8, referenced from:

Re: PATCH: PR target/53539: Different __WCHAR_TYPE__/wchar_t for gcc -m32 on Linux/i386 and Linux/x86-64

2012-07-16 Thread Richard Henderson
On 07/06/2012 11:27 AM, H.J. Lu wrote: +#undef WCHAR_TYPE +#define WCHAR_TYPE (TARGET_LP64 ? int : long int) Hi Richard, Is this patch OK for trunk? Thanks. Yes, this is fine. I'd like to backport it to 4.6/4.7 branches. Is that OK? Meh... is there a real reason for it?

Re: [Patch/RFC] SEH exceptions for Win64

2012-07-16 Thread Richard Henderson
On 07/13/2012 07:13 AM, Tristan Gingold wrote: +_Unwind_RaiseException (struct _Unwind_Exception *exc) +{ + memset (exc-private_, 0, sizeof (exc-private_)); + + RaiseException (STATUS_GCC_THROW, 0, 1, (ULONG_PTR *)exc); We almost certainly didn't want pointer-to-pointer... r~

Re: [C++ RFC / Patch] PR 51213 (access control under SFINAE)

2012-07-16 Thread Jason Merrill
On 07/14/2012 05:54 AM, Paolo Carlini wrote: The above change of yours appear to imply that, at variance with what I had in my first draft, perform_typedefs_access_check shouldn't really gain a tsubst_flags_t argument, because now it's called by instantiate_decl and instantiate_class_template_1

Re: fix thinko in tree-emutls, breaking tls support on vxworks

2012-07-16 Thread Richard Henderson
On 07/03/2012 02:09 AM, Olivier Hainque wrote: 2012-07-03 Olivier Hainque hain...@adacore.com * tree-emutls.c (new_emutls_decl): When a var_section is requested by the target, attach the new decl to that, not to the template section. Ok. r~

Re: [patch] Make mcf.c and ira-conflicts.c not include tree.h

2012-07-16 Thread Jakub Jelinek
On Sun, Jul 08, 2012 at 02:44:15PM +0200, Steven Bosscher wrote: IMHO, no RTL pass implementation file should have to include tree.h. So ira-conflicts.c shouldn't need tree.h. The reason it needed it, was because there is code to avoid putting user variables in callee clobbered registers, and

Re: [patch] Make mcf.c and ira-conflicts.c not include tree.h

2012-07-16 Thread Steven Bosscher
On Mon, Jul 16, 2012 at 9:35 PM, Jakub Jelinek ja...@redhat.com wrote: On Sun, Jul 08, 2012 at 02:44:15PM +0200, Steven Bosscher wrote: IMHO, no RTL pass implementation file should have to include tree.h. So ira-conflicts.c shouldn't need tree.h. The reason it needed it, was because there is

Re: [patch][rfc] Clean up CFG dumping

2012-07-16 Thread Steven Bosscher
On Mon, Jul 16, 2012 at 5:57 PM, Steven Bosscher stevenb@gmail.com wrote: Hello, There are comments in basic-block.h that advise to update certain parts of the compiler if a new edge flag or basic block flag is added: -/* Always update the table in cfg.c dump_edge_info. */ and -

Re: [wwwdocs] Update coding conventions for C++

2012-07-16 Thread Lawrence Crowl
On 7/10/12, Gabriel Dos Reis g...@cs.tamu.edu wrote: Jason Merrill ja...@redhat.com writes: | On 07/09/2012 06:00 PM, Lawrence Crowl wrote: | Done. New patch attached, but note that the html tags have been | stripped from the patch to avoid mailer problems. | | Thanks. If nobody else

Re: [wwwdocs] Update coding conventions for C++

2012-07-16 Thread Gabriel Dos Reis
Lawrence Crowl cr...@google.com writes: | On 7/10/12, Gabriel Dos Reis g...@cs.tamu.edu wrote: | Jason Merrill ja...@redhat.com writes: | | | On 07/09/2012 06:00 PM, Lawrence Crowl wrote: | | Done. New patch attached, but note that the html tags have been | | stripped from the patch to

[PATCH] Define FFI_SIZEOF_JAVA_RAW to 4 for x32

2012-07-16 Thread H.J. Lu
Hi, This patch defines FFI_SIZEOF_JAVA_RAW to 4 for x32, similar to MIPS n32. It fixed: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53982 Thanks. H.J. --- ChangeLog |5 + src/x86/ffitarget.h |3 ++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/ChangeLog

[v3] fix PR 53978

2012-07-16 Thread Jonathan Wakely
PR libstdc++/53978 * include/bits/alloc_traits.h (allocator_traits): Define static constants. * include/bits/ptr_traits.h (pointer_traits): Likewise. * include/ext/alloc_traits.h (__allocator_always_compares_equal): Likewise. Tested x86_64-linux,

Re: [v3] fix PR 53978

2012-07-16 Thread Jonathan Wakely
On 16 July 2012 21:44, Jonathan Wakely wrote: PR libstdc++/53978 * include/bits/alloc_traits.h (allocator_traits): Define static constants. * include/bits/ptr_traits.h (pointer_traits): Likewise. * include/ext/alloc_traits.h

PR libjava/53973: Check and and skip 67h address size prefix for x32

2012-07-16 Thread H.J. Lu
Hi, Since x32 may generate 64-bit integer divide like 67 48 f7 bd a0 fe ff ff idivq -0x160(%ebp) we need to check and skip 67h address size for x32. OK for trunk if there are no regressions on Linux/x86-64? Thanks. H.J. 2012-07-16 H.J. Lu hongjiu...@intel.com PR

C++ PATCH for better access violation diagnostic locations

2012-07-16 Thread Jason Merrill
While looking at Paolo's access sfinae patch, I noticed that we were giving poor locations for deferred access control errors. This patch fixes that by storing the location of the check to use again later. Tested x86_64-pc-linux-gnu, applying to trunk. commit

Re: [v3] PR 53270 fix hppa-linux bootstrap regression

2012-07-16 Thread Jonathan Wakely
On 7 July 2012 18:16, Jonathan Wakely wrote: My preference is (1) for 4.7 and (4) for 4.6 but the changes are not ones I feel comfortable making on release branches without other opinions. Please comment :-) No comments? Then I'm going for my preferences as stated above.

Re: [v3] PR 53270 fix hppa-linux bootstrap regression

2012-07-16 Thread Jonathan Wakely
On 16 July 2012 22:48, Jonathan Wakely wrote: On 7 July 2012 18:16, Jonathan Wakely wrote: My preference is (1) for 4.7 and (4) for 4.6 but the changes are not ones I feel comfortable making on release branches without other opinions. Please comment :-) No comments? Then I'm going for my

RE: [wwwdocs] Document ARM/embedded-x_y-branch family

2012-07-16 Thread Terry Guo
-Original Message- From: Gerald Pfeifer [mailto:ger...@pfeifer.com] Sent: Monday, July 16, 2012 8:10 AM To: Terry Guo Cc: Joey Ye; gcc-patches@gcc.gnu.org Subject: Re: [wwwdocs] Document ARM/embedded-x_y-branch family Hi Terry, On Mon, 9 Jul 2012, Terry Guo wrote: As it