Re: [Patch,AVR]: Fix PR52488 (insane stack usage)

2012-03-13 Thread Denis Chertykov
2012/3/12 Georg-Johann Lay > > ...and now to the right mailing list... > > This patch cuts down insane stack offsets for 8-bit SP targets to a value > the > involved insns can handle. > > Background is that newlib generates such insane offsets (2050 bytes of > stack > for a device with 128 bytes o

Re: PATCH: Properly generate X32 IE sequence

2012-03-13 Thread Uros Bizjak
On Tue, Mar 13, 2012 at 2:20 AM, H.J. Lu wrote: Here is the patch which is equivalent to clearing MASK_TLS_DIRECT_SEG_REFS when Pmode != word_mode.  We need to keep          else if (Pmode == SImode)            {              /* Always generate                  

Re: [build] Remove extra_passes

2012-03-13 Thread Richard Guenther
On Mon, Mar 12, 2012 at 5:11 PM, Rainer Orth wrote: > As suggested, this patch removes the now unused extra_passes mechanism. > > Bootstrapped with no regressions on i386-pc-solaris2.11, ok for > mainline? Ok. Thanks, Richard. >        Rainer > > > 2012-03-09  Rainer Orth   > >        * config.

Re: [google] Add -gfission support to GCC (issue5754090)

2012-03-13 Thread Eric Botcazou
> For google/gcc-4_6 branch. (To be submitted for trunk soon.) The option name sounds like the brainchild of the marketing department. :-) Can we avoid going nuclear in the compiler and use something more mundane? -- Eric Botcazou

RE: PING: [PATCH, ARM, iWMMXt][1/5]: ARM code generic change

2012-03-13 Thread Xinyu Qi
PING At 2012-02-03 10:02:28,"Xinyu Qi" wrote: > PING > > http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01787.html > > At 2011-12-29 14:20:20,"Xinyu Qi" wrote: > > > At 2011-12-15 00:47:48,"Richard Earnshaw" wrote: > > > > On 14/07/11 08:35, Xinyu Qi wrote: > > > > >>> Hi, > > > > >>> > > > > >

RE: PING: [PATCH, ARM, iWMMXt][2/5]: intrinsic head file change

2012-03-13 Thread Xinyu Qi
PING At 2012-02-03 10:05:22,"Xinyu Qi" wrote: > PING > > http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01788.html > > At 2011-12-29 14:22:50,"Xinyu Qi" wrote: > > * config/arm/mmintrin.h: Use __IWMMXT__ to enable iWMMXt > > intrinsics. > > Use __IWMMXT2__ to enable iWMMXt2 intrinsics. >

RE: PING: [PATCH, ARM, iWMMXt][3/5]: built in define and expand

2012-03-13 Thread Xinyu Qi
PING At 2012-02-03 10:07:56,"Xinyu Qi" wrote: > PING > > http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01789.html > > At 2011-12-29 14:25:23,"Xinyu Qi" wrote: > > > At 2011-11-24 09:27:04,"Xinyu Qi" wrote: > > > > At 2011-11-19 07:08:22,"Ramana Radhakrishnan" > > > > wrote: > > > > > On 20 Oc

RE: PING: [PATCH, ARM, iWMMXt][4/5]: WMMX machine description

2012-03-13 Thread Xinyu Qi
PING At 2012-02-03 10:09:55,"Xinyu Qi" wrote: > PING > > http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01786.html > > At 2011-12-29 14:12:44,"Xinyu Qi" wrote: > > At 2011-12-22 17:53:45,"Richard Earnshaw" wrote: > > > On 22/12/11 06:38, Xinyu Qi wrote: > > > > At 2011-12-15 01:32:13,"Richard E

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-13 Thread Tristan Gingold
Rainer, this chunk: --- a/gcc/ada/sysdep.c +++ b/gcc/ada/sysdep.c @@ -252,27 +254,27 @@ __gnat_ttyname (int filedes) #endif ^L #if defined (linux) || defined (sun) || defined (sgi) \ - || (defined (__osf__) && ! defined (__alpha_vxworks)) || defined (WINNT) \ + || ! defined (__alpha_vxworks)

[PATCH] Fix gthr changes

2012-03-13 Thread Richard Guenther
This fixes the duplicate __GTHREAD_MUTEX_INIT_FUNCTION / __gthread_mutex_init_function present after my recent gthr change. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-03-13 Richard Guenther * gthr-posix.h: Remove duplicate __GTHREAD_MUTEX_I

Re: libgcc gcov_fork failure on AIX

2012-03-13 Thread Richard Guenther
On Mon, 12 Mar 2012, David Edelsohn wrote: > Richi, > > Your patch for PR gcov/49484 now causes a bootstrap failure on AIX: > > /farm/dje/src/src/libgcc/libgcov.c: In function '__gcov_fork': > /farm/dje/src/src/libgcc/libgcov.c:1008:5: error: 'mx' undeclared > (first use in this function) > /far

Re: [PATCH] Fix PR49484, gthr requirements update (target maintainers have a looksee)

2012-03-13 Thread Richard Guenther
On Mon, Mar 12, 2012 at 6:44 PM, Gerald Pfeifer wrote: > On Mon, 12 Mar 2012, Richard Guenther wrote: 2012-01-18  Richard Guenther          * gthr.h (__GTHREAD_MUTEX_INIT_FUNCTION): Adjust specification.        * gthr-posix.h (__GTHREAD_MUTEX_INIT_FUNCTION): Define.      

[PATCH] Fix PR52569

2012-03-13 Thread Richard Guenther
Some targets avoid using gthr.h for some weird reason when compiling included unwind-dw2-fde.c. But they do so violating gthrs expectations as they neither define __GTHREAD_MUTEX_INIT nor __GTHREAD_MUTEX_INIT_FUNCTION. The following patch restores the carefully crafted workaround for that to ma

Re: [PATCH] Fix PR49484, gthr requirements update (target maintainers have a looksee)

2012-03-13 Thread Richard Guenther
On Tue, Mar 13, 2012 at 10:44 AM, Richard Guenther wrote: > On Mon, Mar 12, 2012 at 6:44 PM, Gerald Pfeifer wrote: >> On Mon, 12 Mar 2012, Richard Guenther wrote: > 2012-01-18  Richard Guenther   > >        * gthr.h (__GTHREAD_MUTEX_INIT_FUNCTION): Adjust specification. >        *

Re: [PATCH] Don't insert white space in 'orig_option_with_args_text' for OPT_l

2012-03-13 Thread Ludovic Courtès
Hi, (Cc: Paul Brook.) "Joseph S. Myers" skribis: > On Mon, 12 Mar 2012, Ludovic Courtès wrote: > >> The patch below solves the problem in a gfortran-specific way. WDYT? > > I think that's the right approach for this issue. The previous patch was produced with ‘diff -b’. Here’s a fixed one. L

Re: [RFA/ARM]: Correct Neon testsuite generation

2012-03-13 Thread Richard Earnshaw
On 12/03/12 17:19, Matthew Gretton-Dann wrote: > All, > > The commit to fix PR51534 did not update the testsuite (as no changes were > expected there). > > Unfortunately, this means that I didn't notice that the Neon testsuite > generator > is broken. The attached patch fixes the generator. >

Re: [PATCH] Fix PR49484, gthr requirements update (target maintainers have a looksee)

2012-03-13 Thread Richard Earnshaw
On 13/03/12 10:05, Richard Guenther wrote: > On Tue, Mar 13, 2012 at 10:44 AM, Richard Guenther > wrote: >> On Mon, Mar 12, 2012 at 6:44 PM, Gerald Pfeifer wrote: >>> On Mon, 12 Mar 2012, Richard Guenther wrote: >> 2012-01-18 Richard Guenther >> >>* gthr.h (__GTHREAD_MUTEX_

Re: [PATCH] Fix PR49484, gthr requirements update (target maintainers have a looksee)

2012-03-13 Thread Richard Guenther
On Tue, Mar 13, 2012 at 11:22 AM, Richard Earnshaw wrote: > On 13/03/12 10:05, Richard Guenther wrote: >> On Tue, Mar 13, 2012 at 10:44 AM, Richard Guenther >> wrote: >>> On Mon, Mar 12, 2012 at 6:44 PM, Gerald Pfeifer wrote: On Mon, 12 Mar 2012, Richard Guenther wrote: >>> 2012-01-18  

Re: PATCH: Properly generate X32 IE sequence

2012-03-13 Thread Uros Bizjak
On Tue, Mar 13, 2012 at 8:11 AM, Uros Bizjak wrote: Please try attached patch.  It introduces TARGET_TLS_INDIRECT_SEG_REFS to block only indirect seg references. >> >> There is no regression. > > Thanks, committed to mainline SVN with following ChangeLog: > > 2012-03-13  Uros Bizjak   >

Re: Remove obsolete IRIX 6.5 support

2012-03-13 Thread Arnaud Charlet
The gnattools and gcc/ada/gcc-interface changes are OK. The gcc/ada file removals (*irix*) are also OK I'd need to review the other gcc/ada changes more carefully before giving an OK for these and would rather do so with a readily available patch file uncompressed. I'd suggest sending me these p

Re: [PATCH, i386, Android] Enable __ANDROID__ macro for Android i386 target

2012-03-13 Thread Ilya Enkovich
Ping 27 февраля 2012 г. 6:39 пользователь Ilya Enkovich написал: >> >> Undef TARGET_OS_CPP_BUILTINS and define TARGET_OS_CPP_BUILTINS >> in linux.h with GNU_USER_TARGET_OS_CPP_BUILTINS and >> ANDROID_TARGET_OS_CPP_BUILTINS. >> >> >> -- >> H.J. > > Hello, > > Here is a variant with linux.h modific

Re: [PATCH, i386, Android] -mandroid support for i386 target

2012-03-13 Thread Ilya Enkovich
Ping 27 февраля 2012 г. 6:41 пользователь Ilya Enkovich написал: >> You should keep those *_SPEC and define them with new >> GNU_*_SPEC in gnu-user.h since gnu-user.h is also used >> by other non-linux targets.  In linux.h, you undef *_SPEC >> before defining them. >> >> >> -- >> H.J. > > Thanks

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-13 Thread Andreas Schwab
Tristan Gingold writes: > diff --git a/gcc/ada/gcc-interface/decl.c b/gcc/ada/gcc-interface/decl.c > index 34183ba..120628e 100644 > --- a/gcc/ada/gcc-interface/decl.c > +++ b/gcc/ada/gcc-interface/decl.c > @@ -5488,7 +5488,7 @@ gnat_to_gnu_param (Entity_Id gnat_param, Mechanism_Type > mech, >

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-13 Thread Tristan Gingold
On Mar 13, 2012, at 12:13 PM, Andreas Schwab wrote: > Tristan Gingold writes: > >> diff --git a/gcc/ada/gcc-interface/decl.c b/gcc/ada/gcc-interface/decl.c >> index 34183ba..120628e 100644 >> --- a/gcc/ada/gcc-interface/decl.c >> +++ b/gcc/ada/gcc-interface/decl.c >> @@ -5488,7 +5488,7 @@ gnat_

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-13 Thread Richard Guenther
On Tue, Mar 13, 2012 at 12:20 PM, Tristan Gingold wrote: > > On Mar 13, 2012, at 12:13 PM, Andreas Schwab wrote: > >> Tristan Gingold writes: >> >>> diff --git a/gcc/ada/gcc-interface/decl.c b/gcc/ada/gcc-interface/decl.c >>> index 34183ba..120628e 100644 >>> --- a/gcc/ada/gcc-interface/decl.c >>

Re: [google] Add -gfission support to GCC (issue5754090)

2012-03-13 Thread Diego Novillo
On 13/03/12 04:14 , Eric Botcazou wrote: For google/gcc-4_6 branch. (To be submitted for trunk soon.) The option name sounds like the brainchild of the marketing department. :-) Can we avoid going nuclear in the compiler and use something more mundane? I, for one, welcome our new nuclear over

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-13 Thread Tristan Gingold
On Mar 13, 2012, at 12:48 PM, Richard Guenther wrote: > On Tue, Mar 13, 2012 at 12:20 PM, Tristan Gingold wrote: >> >> On Mar 13, 2012, at 12:13 PM, Andreas Schwab wrote: >> >>> Tristan Gingold writes: >>> diff --git a/gcc/ada/gcc-interface/decl.c b/gcc/ada/gcc-interface/decl.c ind

Re: [PATCH 2/3] Move MEM_REF expansion to a new function

2012-03-13 Thread Michael Matz
Hi, On Mon, 12 Mar 2012, Martin Jambor wrote: > OK, the following patch changes the places where I previously called the > new function to call expand_expr with EXPAND_WRITE modifier and then > makes sure we do not perform reads into rvalues in expand_expr_real_1 in > the contexts where I need

[PATCH] Fix PR52134

2012-03-13 Thread Richard Guenther
We fail to fold round_up generated expressions during size expression folding. Fixed as follows, bootstrapped and tested on x86_64-unknwon-linux-gnu, applied to trunk. Richard. 2012-03-13 Richard Guenther PR middle-end/52134 * fold-const.c (fold_binary_loc): Fold (X * Y) & -

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-13 Thread Andreas Schwab
Tristan Gingold writes: > For which target ? ia64-*-* or alpha-*-* > On non-VMS targets, flag_vms_malloc64 is declared in gigi.h as: > > /* Let code know whether we are targetting VMS without need of >intrusive preprocessor directives. */ > #ifndef TARGET_ABI_OPEN_VMS TARGET_ABI_OPEN_VMS

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-13 Thread Tristan Gingold
On Mar 13, 2012, at 2:57 PM, Andreas Schwab wrote: > Tristan Gingold writes: > >> For which target ? > > ia64-*-* or alpha-*-* > >> On non-VMS targets, flag_vms_malloc64 is declared in gigi.h as: >> >> /* Let code know whether we are targetting VMS without need of >> intrusive preprocessor

Re: [Patch ARM] Fix definition of MALLOC_ABI_ALIGNMENT.

2012-03-13 Thread Richard Earnshaw
On 12/03/12 22:35, Ramana Radhakrishnan wrote: > Hi, > > This sets MALLOC_ABI_ALIGNMENT to 64 bits on ARM which should be safe > on all AAPCS implementations given that this is a requirement on our > alignment specifiers. I would expect this to be true for most modern > malloc implementations on A

Re: [PATCH] Fix PRs 52080, 52097 and 48124, rewrite bitfield expansion, enable the C++ memory model wrt bitfields everywhere

2012-03-13 Thread Richard Guenther
On Mon, 12 Mar 2012, Eric Botcazou wrote: > > Btw, I _think_ I want GET_MODE_BITSIZE here - we cannot allow > > GET_MODE_BITSIZE > GET_MODE_PRECISION as that would possibly > > access memory that is not allowed. Thus, what GET_MODE_* would > > identify the access size used for a MEM of that mode?

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-13 Thread Tristan Gingold
On Mar 13, 2012, at 2:57 PM, Andreas Schwab wrote: > Tristan Gingold writes: > >> For which target ? > > ia64-*-* or alpha-*-* > >> On non-VMS targets, flag_vms_malloc64 is declared in gigi.h as: >> >> /* Let code know whether we are targetting VMS without need of >> intrusive preprocessor

Re: [PATCH 2/3] Move MEM_REF expansion to a new function

2012-03-13 Thread Richard Guenther
On Tue, 13 Mar 2012, Michael Matz wrote: > Hi, > > On Mon, 12 Mar 2012, Martin Jambor wrote: > > > OK, the following patch changes the places where I previously called the > > new function to call expand_expr with EXPAND_WRITE modifier and then > > makes sure we do not perform reads into rvalu

Re: PATCH: Properly generate X32 IE sequence

2012-03-13 Thread H.J. Lu
On Tue, Mar 13, 2012 at 3:37 AM, Uros Bizjak wrote: > On Tue, Mar 13, 2012 at 8:11 AM, Uros Bizjak wrote: > > Please try attached patch.  It introduces TARGET_TLS_INDIRECT_SEG_REFS > to block only indirect seg references. >>> >>> There is no regression. >> >> Thanks, committed to mainline

Re: PATCH: Call gen_tls_global_dynamic_64_ and en_tls_local_dynamic_base_64_

2012-03-13 Thread Uros Bizjak
On Mon, Mar 12, 2012 at 10:35 PM, H.J. Lu wrote: 2012-03-12  H.J. Lu          * config/i386/i386.c (ix86_gen_tls_global_dynamic_64): New.        (ix86_gen_tls_local_dynamic_base_64): Likewise.        (ix86_option_override_internal): Set ix86_gen_tls_global_dynamic_64

[PATCH] Fix PR52578

2012-03-13 Thread Richard Guenther
This makes sure we fold conversion chains where the outermost conversion is a sign-change only and that we associate expressions in more cases when two variables appear but they have equal value. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2012-03-13 Richard Guenthe

[PATCH][Cilkplus] Fix a bug in array notations

2012-03-13 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch mainly affecting the C compiler. It fixes a bug in function calls that take in array notations. Thanks, Balaji V. Iyer.Index: gcc/c-typeck.c === --- gcc/c-typeck.c (revis

Re: [gimplefe] [patch] splits cpp_lshift and cpp_rshift tokens into discrete cpp_less and cpp_greater tokens respectively

2012-03-13 Thread Diego Novillo
On 08/03/12 20:47 , Sandeep Soni wrote: +/* Splits the token TOKEN into two tokens FIRST_TOKEN and SECOND_TOKEN. + Note that the split should work only if the type of the TOKEN is + either CPP_RSHIFT or CPP_LSHIFT which gets splitted into two tokens s/splitted/split/ while (gl_lex_tok

[PATCH, i386]: Handle subregs of AND zero extended address correctly

2012-03-13 Thread Uros Bizjak
Hello! We allow addresses, zero extended with (AND (Dimode OP) (0x0)). Since we now track address mode precisely, we should convert this operand to a SImode subreg before passing it further down of ix86_decompose_address function. This fixes a small hole, where operands in wrong mode could

Re: [PATCH] gfortran testsuite: implicitly cleanup-modules

2012-03-13 Thread Bernhard Reutner-Fischer
On Fri, Mar 09, 2012 at 01:02:01AM +0100, Mikael Morin wrote: >On 01/03/2012 22:09, Bernhard Reutner-Fischer wrote: >> Hi, >> >> By now we have quite some leftover modules in the testsuite, again. >> Given that the previous suggestion in this thread -- to have a separate >> script in contrib -- di

Re: PING PATCH: Assert DWARF register size <= saved reg size

2012-03-13 Thread Jason Merrill
Ok. Jason

[pph] Do not emit callgraph nodes to PPH images [1/2] (issue5803043)

2012-03-13 Thread Diego Novillo
Part 1: Add more trees to the preloaded tree cache. This prevents more common trees, built at compiler initialization time, to be streamed out to PPH images. These trees are in: - type_hash_table: The private hash table used in tree.c to record canonical types. To avoid exposi

[pph] Do not emit callgraph nodes to PPH images [2/2] (issue5804043)

2012-03-13 Thread Diego Novillo
Part 1: Remove callgraph nodes from PPH images. This patch fixes several failures by refusing to generat code while writing PPH images. Instead, the writer saves state information for the reader to expand and declare symbols via rest_of_decl_compilation and expand_or_defer_fn/expand_or_defer_fn_1

Re: [patch, 4.7] libitm: Fix lost wake-up in serial lock.

2012-03-13 Thread Richard Henderson
On 03/10/12 09:43, Torvald Riegel wrote: > libitm: Fix lost wake-up in serial lock. > > PR libitm/52526 > * config/linux/rwlock.cc (GTM::gtm_rwlock::read_lock): Fix lost > wake-up. Ok. Jakub needs to approve this for 4.7.0; otherwise ok for 4.7.1. r~

Re: [patch, 4.7] libitm: Fix lost wake-up in serial lock.

2012-03-13 Thread Jakub Jelinek
On Tue, Mar 13, 2012 at 09:54:36AM -0700, Richard Henderson wrote: > On 03/10/12 09:43, Torvald Riegel wrote: > > libitm: Fix lost wake-up in serial lock. > > > > PR libitm/52526 > > * config/linux/rwlock.cc (GTM::gtm_rwlock::read_lock): Fix lost > > wak

[PATCH C++] Use next_conversion for better safety

2012-03-13 Thread Dodji Seketeli
Hello, This patch was accepted[1] before stage-1. I have rebased, bootstrapped and tested it against trunk on x86_64-unknown-linux-gnu. Applying to trunk. [1]: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01145.html gcc/cp/ * call.c (standard_conversion, build_integral_nontype_arg_con

[PATCH] PR c++/51641 - Lookup finds enclosing class member

2012-03-13 Thread Dodji Seketeli
Hello, This patch was previously accepted[1] before stage 1. I have rebased, bootstrapped and tested it against trunk on x86_64-unknown-linux-gnu. I am applying it to trunk. [1]: http://gcc.gnu.org/ml/gcc-patches/2012-01/msg01571.html gcc/cp/ PR c++/51641 * cp-tree.h (template

Re: [PATCH] Fix PR 50971 and PR 35629: Only one loop detected when there should be two

2012-03-13 Thread Andrew Pinski
Ping? Rebootstrapped on x86_64-linux-gnu with no regressions. Thanks, Andrew Pinski On Sat, Jan 21, 2012 at 1:21 PM, Andrew Pinski wrote: > The problem with these two bug reports is that the cfgloop does not do > a good job for disambiguating some loops.  This patch rewrites > find_subloop_latc

[PATCH, i386]: Declare constant_call_address_operand as special predicate

2012-03-13 Thread Uros Bizjak
Hello! Similar to tls_symbolic_operand and tls_modbase_operand, we are not interested in the mode of the operand in the predicate. 2012-03-13 Uros Bizjak * config/i386/predicates.md (constant_call_address_operand): Declare as special predicate. Update all uses. * conf

Re: [google] Add -gfission support to GCC (issue5754090)

2012-03-13 Thread Eric Botcazou
> I, for one, welcome our new nuclear overlords. AFAICS the internal switch is called dwarf_split_debug_info so a short moniker based on it would be more understandable (and avoid a useless pomposity). -- Eric Botcazou

Re: [google] Add -gfission support to GCC (issue5754090)

2012-03-13 Thread Cary Coutant
>> I, for one, welcome our new nuclear overlords. > > AFAICS the internal switch is called dwarf_split_debug_info so a short moniker > based on it would be more understandable (and avoid a useless pomposity). I wasn't trying to be pompous! It's just our project name, but I thought "fission" to be

Re: [PATCH, i386] RTM support

2012-03-13 Thread Uros Bizjak
On Sun, Mar 11, 2012 at 10:16 AM, Kirill Yukhin wrote: >> >> The patch is OK for mainline, if there are no further comments in next 24h. > > According to Tobias's input, I've added few lines about RTM to > doc/invoke.texi. If no objections - I'll commit the patch tomorrow. A small no-op change -

Re: [google] Add -gfission support to GCC (issue5754090)

2012-03-13 Thread Xinliang David Li
On Tue, Mar 13, 2012 at 11:56 AM, Cary Coutant wrote: >>> I, for one, welcome our new nuclear overlords. >> >> AFAICS the internal switch is called dwarf_split_debug_info so a short >> moniker >> based on it would be more understandable (and avoid a useless pomposity). > > I wasn't trying to be p

gcc-frontend tracer and a perlscript htmltags.pl

2012-03-13 Thread hopsingk
Hi, I would like to send a gcc-frontend tracer and a perlscript called htmltags.pl that takes the trace and creates a nice dhtml page. The source is here: http://cfw.sourceforge.net/htmltags.html an example of the output is here: http://cfw.sourceforge.net/htmltag/init_32.c.pinfo.html There is a h

Re: [PATCH, i386]: Handle subregs of AND zero extended address correctly

2012-03-13 Thread H.J. Lu
On Tue, Mar 13, 2012 at 9:21 AM, Uros Bizjak wrote: > Hello! > > We allow addresses, zero extended with (AND (Dimode OP) > (0x0)). Since we now track address mode precisely, we should > convert this operand to a SImode subreg before passing it further down > of ix86_decompose_address funct

[committed] Fix up [sn]mmintrin.h

2012-03-13 Thread Jakub Jelinek
Hi! I've noticed warnings on these two headers that /* is seen inside of a comment. Usually this is silenced because the headers are system headers, but with -Wall -Wsystem-headers or when the dir with the headers isn't -isystem this shows up. Committed as obvious, queued for 4.7.1 and 4.6.4. 2

[C PATCH] Fix -Wunused-but-set-* on __builtin_shuffle (PR c/52577)

2012-03-13 Thread Jakub Jelinek
Hi! We weren't calling mark_exp_read on these and thus we got false positive -Wunused-but-set-* warnings. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/4.7.1? For trunk we should add __builtin_shuffle support even for C++ I'd say. 2012-03-13 Jakub Jelinek

Re: [PATCH, i386]: Handle subregs of AND zero extended address correctly

2012-03-13 Thread Uros Bizjak
On Tue, Mar 13, 2012 at 9:01 PM, H.J. Lu wrote: > On Tue, Mar 13, 2012 at 9:21 AM, Uros Bizjak wrote: >> Hello! >> >> We allow addresses, zero extended with (AND (Dimode OP) >> (0x0)). Since we now track address mode precisely, we should >> convert this operand to a SImode subreg before p

Re: [PATCH, i386]: Handle subregs of AND zero extended address correctly

2012-03-13 Thread Uros Bizjak
On Tue, Mar 13, 2012 at 9:09 PM, Uros Bizjak wrote: > On Tue, Mar 13, 2012 at 9:01 PM, H.J. Lu wrote: >> On Tue, Mar 13, 2012 at 9:21 AM, Uros Bizjak wrote: >>> Hello! >>> >>> We allow addresses, zero extended with (AND (Dimode OP) >>> (0x0)). Since we now track address mode precisely, w

[C++ PATCH] Fix udlit handling (PR c++/52521)

2012-03-13 Thread Jakub Jelinek
Hi! As detailed in the PR, we would return operator "" _a (const char *) when looking for 2 argument const char *, std::size_t literal operator, because the first argument matched and we ran out of arguments and didn't check we have them all. The following patch fixes it. Bootstrapped/regtested o

Re: [PATCH] gfortran testsuite: implicitly cleanup-modules

2012-03-13 Thread Mike Stump
On Mar 13, 2012, at 9:38 AM, Bernhard Reutner-Fischer wrote: > Could some of the testsuite maintainers please eyeball? I've eyed it, the only thing that stood out was: -foreach testcase [lsort [glob -nocomplain $srcdir/$subdir/*.F]] { -if ![runtest_file_p $runtests $testcase] then { - c

Re: [RFC]: Add support for pragma pointer_size

2012-03-13 Thread Gary Funck
On 03/06/12 14:09:23, Tristan Gingold wrote: > The patch is simple: the C front-end will now calls > c_build_pointer_type (instead of build_pointer_type), which in > turn calls build_pointer_type_for_mode using the right mode. [...] Joining this discussion a bit late ... I have a few questions. Th

libgo patch RFA: Fix handling of timex struct on ARM GNU/Linux

2012-03-13 Thread Ian Lance Taylor
This patch to libgo fixes the handling of the timex struct on ARM GNU/Linux. This is PR go/52557: a boostrap failure on ARM. Bootstrapped on x86_64-unknown-linux-gnu (I don't have an ARM GNU/Linux system). Committed to mainline. OK for 4.7 branch? Ian diff -r ca4aaf510222 libgo/mksysinfo.sh --

Re: [C PATCH] Fix -Wunused-but-set-* on __builtin_shuffle (PR c/52577)

2012-03-13 Thread Joseph S. Myers
On Tue, 13 Mar 2012, Jakub Jelinek wrote: > 2012-03-13 Jakub Jelinek > > PR c/52577 > * c-parser.c (c_parser_postfix_expression) > : Call mark_exp_read on argument values. > > * gcc.dg/Wunused-var-3.c: New test. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: libgo patch RFA: Fix handling of timex struct on ARM GNU/Linux

2012-03-13 Thread Jakub Jelinek
On Tue, Mar 13, 2012 at 01:59:00PM -0700, Ian Lance Taylor wrote: > This patch to libgo fixes the handling of the timex struct on ARM > GNU/Linux. This is PR go/52557: a boostrap failure on ARM. > Bootstrapped on x86_64-unknown-linux-gnu (I don't have an ARM GNU/Linux > system). Committed to main

Re: [PATCH] PR c++/50852 - Revisit dependant template parameter

2012-03-13 Thread Jason Merrill
On 03/08/2012 08:21 AM, Dodji Seketeli wrote: So I needed to say somehow that the P in the pack expansion (in the pattern) is actually a Pr "in spirit". Ah, I see. +/*we will fixup the siblings for Space and capital W. +get_root_index_same_level (tree i

[patch libffi SH] Fix failures for libffi.call/err_bad_abi.c

2012-03-13 Thread Kaz Kojima
Hi, I've applied the attached patch which fixes failures for libffi.call/err_bad_abi.c on SH. It's similar to the changes done already on other targets. Tested on sh-linux. Regards, kaz -- 2012-03-13 Kaz Kojima * src/sh/ffi.c (ffi_prep_closure_loc): Don't ASSERT ABI test,

libgo patch RFA: Export {enter,exit}syscall

2012-03-13 Thread Ian Lance Taylor
The cooperative threading model used by Go works by calling entersyscall whenever we are about to make a call to a C function that may block. That was not being done for a call to getaddrinfo used when doing a DNS lookup. This patch fixes that problem by exporting the entersyscall and exitsyscall

[Patch, fortran] PR 49010/24518 MOD/MODULO fixes

2012-03-13 Thread Janne Blomqvist
Hi, the attached patch implements a few fixes and cleanups for the MOD and MODULO intrinsics. - When the arguments are constant, use mpfr_fmod instead of the naive algorithms which are numerically unstable for large arguments. This extends the PR 24518 fix to constant arguments as well, and makes

[SH] Fix PR 49468 testcases

2012-03-13 Thread Oleg Endo
Hi, The attach patch just swaps the dg directive lines for the PR 49468 testcases. Having 'dg-skip-if' as the first directive doesn't seem to work and always gets ignored. OK to apply? Cheers, Oleg testsuite/ChangeLog PR target/49468 * gcc.target/sh/pr49468-si.c: Make dg-skip-

Re: [google] Add -gfission support to GCC (issue5754090)

2012-03-13 Thread Diego Novillo
On 13/03/12 15:43 , Xinliang David Li wrote: On Tue, Mar 13, 2012 at 11:56 AM, Cary Coutant wrote: I, for one, welcome our new nuclear overlords. AFAICS the internal switch is called dwarf_split_debug_info so a short moniker based on it would be more understandable (and avoid a useless pompos

Re: [PATCH, i386]: Declare constant_call_address_operand as special predicate

2012-03-13 Thread H.J. Lu
On Tue, Mar 13, 2012 at 11:42 AM, Uros Bizjak wrote: > Hello! > > Similar to tls_symbolic_operand and tls_modbase_operand, we are not > interested in the mode of the operand in the predicate. > > 2012-03-13  Uros Bizjak   > >        * config/i386/predicates.md (constant_call_address_operand): Decl

4.7 version of c99status.html

2012-03-13 Thread Joseph S. Myers
I've created the 4.7 version of c99status.html and made the 4.7 branch manual refer to it instead of the mainline version of the page. Doc patch tested with "make info html pdf". Index: c99status.html === RCS file: /cvs/gcc/wwwdocs

Update contrib/gennews for 4.7

2012-03-13 Thread Joseph S. Myers
I've applied this patch to contrib/gennews on trunk and 4.7 branch to make it include release notes for 4.7 in the files it processes. (Checking this file is in the release checklist, but updating it in advance means one fewer thing that could be missed when making the final release.) Index: C

Re: [SH] Fix PR 49468 testcases

2012-03-13 Thread Kaz Kojima
Oleg Endo wrote: > The attach patch just swaps the dg directive lines for the PR 49468 > testcases. Having 'dg-skip-if' as the first directive doesn't seem to > work and always gets ignored. > > OK to apply? OK. Regards, kaz

Re: [C++ PATCH] Fix udlit handling (PR c++/52521)

2012-03-13 Thread Jason Merrill
On 03/13/2012 04:17 PM, Jakub Jelinek wrote: I'm not 100% sure what should be done if some of the literal operators have default arguments. I'm not, either. Please add a comment about that; perhaps the comparison to void_list_node should be a call to sufficient_parms_p. OK with the added co

Re: [PATCH, i386] RTM support

2012-03-13 Thread Kirill Yukhin
Thanks! K On Tue, Mar 13, 2012 at 11:14 PM, Uros Bizjak wrote: > On Sun, Mar 11, 2012 at 10:16 AM, Kirill Yukhin > wrote: >>> >>> The patch is OK for mainline, if there are no further comments in next 24h. >> >> According to Tobias's input, I've added few lines about RTM to >> doc/invoke.texi.