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: >

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

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.

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 f84991f247261f8e9d8c57115398f2fcd6

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 PR libjava/53973 * i

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 (__allocator_always_comp

[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, com

[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

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

2012-07-16 Thread Gabriel Dos Reis
Lawrence Crowl writes: | On 7/10/12, Gabriel Dos Reis wrote: | > Jason Merrill writes: | > | > | On 07/09/2012 06:00 PM, Lawrence Crowl wrote: | > | > Done. New patch attached, but note that the tags have been | > | > stripped from the patch to avoid mailer problems. | > | | > | Thanks. If n

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

2012-07-16 Thread Lawrence Crowl
On 7/10/12, Gabriel Dos Reis wrote: > Jason Merrill writes: > > | On 07/09/2012 06:00 PM, Lawrence Crowl wrote: > | > Done. New patch attached, but note that the tags have been > | > stripped from the patch to avoid mailer problems. > | > | Thanks. If nobody else has any comments, I think this

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

2012-07-16 Thread Steven Bosscher
On Mon, Jul 16, 2012 at 5:57 PM, Steven Bosscher 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 > > - Always update

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 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 code to avo

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, a

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 > > * 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: [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: [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: 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 branche

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", refere

[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) /home/kwerner/dailybuild/spu-tc-2012-07-16/gcc-head/src/gcc/config/spu/

[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 wh

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

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] 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 m

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: [PATCH 1/2] gcc symbol database

2012-07-16 Thread Dodji Seketeli
Yunfeng ZHANG 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 macro_st

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: [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: [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 wrote: > On Mon, Jul 16, 2012 at 7:14 AM, Kirill Yukhin > 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.

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 i

Re: [PATCH] Fix up _xabort for -O0

2012-07-16 Thread Uros Bizjak
On Mon, Jul 16, 2012 at 4:45 PM, Jakub Jelinek 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 > > * config/i386/rtmintrin.h (_xabort): Remove trailing semicolon. OK,

[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 * config/i386/rtmintrin.h (_xabort): Remove trailing semicolon. --- gcc/config/i386/rtmintrin.h 2012-03-14 09:39:41.167265085 +01

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* ari

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: [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 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 Kirill Yukhin > >

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 bui

[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 PR target/53877 * config/i386/lzcntintrin.h (_lzcnt_u32):

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 wrote: > Richard Guenther wrote: >> On Fri, Jul 6, 2012 at 6:36 PM, Tom de Vries wrote: >> > Bootstrapped and reg-tested (ada inclusive) on x86_64. >> > >> > OK for trunk? >> >> Ok. >> Thanks, >> Richard. >> >> > 2012-07-06 Tom de Vries >> >

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 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 mingw targets. so t

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: [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

[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 wrote: > > Bootstrapped and reg-tested (ada inclusive) on x86_64. > > > > OK for trunk? > > Ok. > Thanks, > Richard. > > > 2012-07-06 Tom de Vries > > Richard Guenther > > > > * tree-ssa-ccp.c (optimi

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

[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 in

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

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 n

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 wer

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 sinc

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 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 () > at /export/gnu

[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; pra

[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 inf

[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, committed

[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 * a-exexpr.adb (Propagate_Exception): Adjust call to Exception_Traces procedures. *

[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 * seh_init.c (__gnat_map_SEH): New function extracted

[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 setup

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 - -DHAVE_CONFI

[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 * s-crtl.ads (mkdir): New routine, support encoding. * adaint.h (__gnat_mkdir): Update spec to pass encoding. * mkdir.c (__gnat_mkdir): Add encoding parameter. * a-direct.adb (Create_Direct

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 wrote: > On Mon, Jul 16, 2012 at 11:12 AM, Steven Bosscher > wrote: >> On Mon, Jul 16, 2012 at 11:00 AM, Richard Guenther >> wrote: >>> On Fri, Jul 13, 2012 at 3:38 PM, Steven Bosscher >>> wrote: Hello, The attached monster patc

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 wrote: > On Mon, Jul 16, 2012 at 11:00 AM, Richard Guenther > wrote: >> On Fri, Jul 13, 2012 at 3:38 PM, Steven Bosscher >> wrote: >>> Hello, >>> >>> The attached monster patch re-organizes a lot of includes to avoid >>> dependencies on tree-pas

[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, 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 >> attribute

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 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 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() instead of NULL?

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 wrote: > On Mon, Jul 16, 2012 at 10:53 AM, Richard Guenther > 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 n

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 wrote: > Hello, > > Just another small cleanup. This probably made sense before > function-at-a-time became the norm, 10 years ago. > > Bootstrapped&tested on powerpc64-unknown-linux-gnu and > x86_64-unknown-linux-gnu. > OK for trunk? Ok. Thanks,

Re: [patch] Remove expand_expr_stmt

2012-07-16 Thread Richard Guenther
On Sat, Jul 14, 2012 at 7:34 PM, Steven Bosscher 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 involve a > call to a

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 PROCE

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 wrote: > On Fri, Jul 13, 2012 at 3:38 PM, Steven Bosscher > 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 followi

[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 t

Re: Fix PR53908

2012-07-16 Thread Richard Guenther
On Fri, Jul 13, 2012 at 11:49 PM, Steven Bosscher wrote: > On Fri, Jul 13, 2012 at 10:56 AM, Hans-Peter Nilsson > wrote: >>> From: Richard Guenther >>> Date: Fri, 13 Jul 2012 10:08:05 +0200 >>> On Fri, Jul 13, 2012 at 9:59 AM, Hans-Peter Nilsson >>> wrote: >>> > Ok for 4.7 too? >>> >>> Of cours

Re: [PATCH][MIPS] NetLogic XLP scheduling

2012-07-16 Thread Richard Sandiford
Maxim Kuvyrkov 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 writes: >>> Tested on mips64-elf, and by making sure that there were no changes in >>> -O2 output for a recent set of cc1 .ii files. Applied. >>> >>> I

Re: [patch] Call free_after_parsing earlier

2012-07-16 Thread Richard Guenther
On Fri, Jul 13, 2012 at 4:33 PM, Steven Bosscher wrote: > On Thu, Jul 12, 2012 at 9:21 AM, Richard Guenther > wrote: >> On Wed, Jul 11, 2012 at 9:39 PM, Steven Bosscher >> wrote: >>> Hello, >>> >>> GCC calls free_after_parsing in rest_of_clean_state. >>> That's way too late, it can be done in f

Re: [PATCH][MIPS] NetLogic XLP scheduling

2012-07-16 Thread Richard Sandiford
Chung-Lin Tang 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] 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 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 to do a lot more clean

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 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: > > * tree-dump.h should

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 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 > doesn't clean u

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] 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 e

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 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 > attribute compariso

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 h

[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 * gcc.dg/torture/b