Re: PING: PATCH [9/n]: Prepare x32: PR middle-end/47383: ivopts miscompiles Pmode != ptr_mode

2011-07-06 Thread Richard Guenther
On Tue, Jul 5, 2011 at 7:07 PM, H.J. Lu hjl.to...@gmail.com wrote: On Tue, Jul 5, 2011 at 8:24 AM, Richard Guenther richard.guent...@gmail.com wrote: On Tue, Jul 5, 2011 at 4:56 PM, Ulrich Weigand uweig...@de.ibm.com wrote: H.J. Lu wrote: However, this still seems odd to me, as I had

Re: Ping^2: TARGET_HAVE_NAMED_SECTIONS cleanup

2011-07-06 Thread Richard Guenther
On Tue, Jul 5, 2011 at 9:16 PM, Joseph S. Myers jos...@codesourcery.com wrote: Ping^2.  The patch http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01642.html is still pending review. Ok if there are no objections from target maintainers in 24h. Thanks, Richard. -- Joseph S. Myers

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

2011-07-06 Thread Xinyu Qi
Hi, It is the second part of iWMMXt maintenance. *config/arm/mmintrin.h: Revise the iWMMXt intrinsics head file. Fix some intrinsics and add some new intrinsics. Thanks, Xinyu 2_mmintrin.diff Description: 2_mmintrin.diff

[PATCH, ARM, iWMMXt][0/5]: iWMMXt intrinsics maintenance and pipeline description

2011-07-06 Thread Xinyu Qi
Hi, Since the patch of iWMMXt intrinsics maintenance and pipeline description is too big to review, I subdivide it into five parts. 1 ARM generic code change 2 iWMMXt intrinsic head file change. 3 iWMMXt built in define and expand 4 WMMX machine description 5 WMMX pipeline description These

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

2011-07-06 Thread Xinyu Qi
Hi, It is the first part of iWMMXt maintenance. *config/arm/arm.c (arm_option_override): Enable iWMMXt with VFP. iWMMXt and NEON are incompatible. iWMMXt unsupported under Thumb-2 mode. (arm_expand_binop_builtin): Accept immediate op (with mode VOID) *config/arm/arm.md: Resettle include

Cleanup Solaris ASM_SPEC handling

2011-07-06 Thread Rainer Orth
As already noted in the Solaris configuration cleanup patch, the ASM_SPEC handling on Solaris can be simplified. This patch does this, also as a prerequisite for a followup to provide a 64-bit default Solaris/x86 configuration. The basic observation is that there's a common part handled by both

Provide 64-bit default Solaris/x86 configuration (PR target/39150)

2011-07-06 Thread Rainer Orth
There has long been some clamoring for a amd64-*-solaris2 configuration similar to sparcv9-sun-solaris2. I've resisted this for quite some time, primarily because it doubles the maintenance effort of testing both the 32-bit default and 64-bit default configurations. After the recent cleanup

[testsuite] Don't XFAIL gcc.dg/tree-ssa/20030807-7.c (PR tree-optimization/49647)

2011-07-06 Thread Rainer Orth
As described in the PR, gcc.dg/tree-ssa/20030807-7.c seems to XPASS everywhere. This patch removes the xfail. Tested with the appropriate runtest invocation on i386-pc-solaris2.10. Ok for mainline? Rainer 2011-07-06 Rainer Orth r...@cebitec.uni-bielefeld.de PR

[Path,AVR]: Improve loading of 32-bit constants

2011-07-06 Thread Georg-Johann Lay
For loading a 32-bit constant in a register, there is room for improvement: * SF can be handled the same way as SI and therefore the patch adds a peep2 to produce a *reload_insf analogon to *reload_insi. * If the destination register overlaps NO_LD_REGS, values already loaded into some other

Re: [testsuite] Don't XFAIL gcc.dg/tree-ssa/20030807-7.c (PR tree-optimization/49647)

2011-07-06 Thread Richard Guenther
On Wed, 6 Jul 2011, Rainer Orth wrote: As described in the PR, gcc.dg/tree-ssa/20030807-7.c seems to XPASS everywhere. This patch removes the xfail. Tested with the appropriate runtest invocation on i386-pc-solaris2.10. Ok for mainline? Ok. Thanks, Richard. Rainer

Re: [PATCH] Fix configure --with-cloog

2011-07-06 Thread Romain Geissler
Hello This patch fix an issue while building with cloog and gmp installed in a custom separate directories. How to reproduce :  - Make sure you've installed cloog and gmp in separate directories (ie ${WITH-CLOOG-PATH}/lib doesn't contain libgmp)  - Make sure neither gmp nor cloog is not

PATCH TRUNK: [gcc/configure.ac] Generate GCCPLUGIN_VERSION_* macros

2011-07-06 Thread Basile Starynkevitch
Hello All, The fie plugin-version.h is now generated by gcc/configure.ac. It contains version information (about the GCC supposed to load the plugin compiled with it) as constant strings. But I think it will also help some plugin developers if that file (which is packaged in gcc-4.6-plugin-dev

[Patch,testsuite]: target-supports.exp: Disable -fprofile-generate for AVR

2011-07-06 Thread Georg-Johann Lay
AVR tests will fail if -fprofile-generate is given because that is not (yet) implemented. CCed avr port maintainer in the case they have objections. Ok to commit? Johann * lib/target-supports.exp (check_profiling_available): Disable profiling with -fprofile-generate for target

Re: [PATCH] Address lowering [1/3] Main patch

2011-07-06 Thread Richard Guenther
On Tue, Jul 5, 2011 at 3:59 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: (Sorry for the late response; yesterday was a holiday here.) On Mon, 2011-07-04 at 16:21 +0200, Richard Guenther wrote: On Thu, Jun 30, 2011 at 4:39 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote:

Re: PATCH TRUNK: [gcc/configure.ac] Generate GCCPLUGIN_VERSION_* macros

2011-07-06 Thread Richard Guenther
On Wed, Jul 6, 2011 at 2:50 PM, Basile Starynkevitch bas...@starynkevitch.net wrote: Hello All, The fie plugin-version.h is now generated by gcc/configure.ac. It contains version information (about the GCC supposed to load the plugin compiled with it) as constant strings. But I think it

[PATCH] Fix PR49645, with C FE pieces

2011-07-06 Thread Richard Guenther
This fixes PR49645 - with MEM_REF the value-numbering machinery to look through aggregate copies wasn't working reliably as we have two representations for X, X and MEM[X]. The following patch fixes that by internally always using the more complicated representation. The patch needs consistent

Re: [Patch,testsuite]: target-supports.exp: Disable -fprofile-generate for AVR

2011-07-06 Thread Rainer Orth
Georg-Johann Lay a...@gjlay.de writes: Index: lib/target-supports.exp === --- lib/target-supports.exp (revision 175811) +++ lib/target-supports.exp (working copy) @@ -497,6 +497,13 @@ proc check_profiling_available { test_wh

Re: [PATCH] Fix PR49645, with C FE pieces

2011-07-06 Thread Michael Matz
Hi, On Wed, 6 Jul 2011, Richard Guenther wrote: *** copy_reference_ops_from_ref (tree ref, V *** 579,585 memset (temp, 0, sizeof (temp)); /* We do not care for spurious type qualifications. */ ! temp.type = TYPE_MAIN_VARIANT (TREE_TYPE (ref));

Re: [Path,AVR]: Improve loading of 32-bit constants

2011-07-06 Thread Denis Chertykov
2011/7/6 Georg-Johann Lay a...@gjlay.de: For loading a 32-bit constant in a register, there is room for improvement: * SF can be handled the same way as SI and therefore the patch  adds a peep2 to produce a *reload_insf analogon to *reload_insi. * If the destination register overlaps

Re: [PATCH] Fix PR49645, with C FE pieces

2011-07-06 Thread Joseph S. Myers
On Wed, 6 Jul 2011, Richard Guenther wrote: * c-decl.c (finish_decl): Also set DECL_HARD_REGISTER for global register variables. OK. -- Joseph S. Myers jos...@codesourcery.com

--enable-gnu-indirect-function patch

2011-07-06 Thread Ian Lance Taylor
This patch: http://gcc.gnu.org/ml/gcc-patches/2010-09/msg02070.html for x86_64-*-linux* sets the default for --enable-gnu-indirect-function to glibc-2011. This string is not used anywhere else in gcc as far as I can see. What is the purpose of that? The original version of the patch

patch committed: Correct configure option name in docs

2011-07-06 Thread Ian Lance Taylor
This patch corrects the name of the configure option --enable-gnu-indirect-function in the docs to correspond to the source. Committed as obvious. Ian 2011-07-06 Ian Lance Taylor i...@google.com * doc/install.texi (Configuration): It's --enable-gnu-indirect-function, not

Re: PATCH TRUNK: [gcc/configure.ac] Generate GCCPLUGIN_VERSION_* macros

2011-07-06 Thread Richard Guenther
On Wed, Jul 6, 2011 at 3:46 PM, Basile Starynkevitch bas...@starynkevitch.net wrote: On Wed, Jul 06, 2011 at 03:21:47PM +0200, Richard Guenther wrote: On Wed, Jul 6, 2011 at 2:50 PM, Basile Starynkevitch bas...@starynkevitch.net wrote: I belive it can help to make plugin code more robust. A

Re: --enable-gnu-indirect-function patch

2011-07-06 Thread Nathan Sidwell
On 07/06/11 14:57, Ian Lance Taylor wrote: This patch: http://gcc.gnu.org/ml/gcc-patches/2010-09/msg02070.html for x86_64-*-linux* sets the default for --enable-gnu-indirect-function to glibc-2011. This string is not used anywhere else in gcc as far as I can see. What is the purpose of that?

Define WORDS_BIG_ENDIAN in rs6000/vxworks.h

2011-07-06 Thread Joseph S. Myers
At http://gcc.gnu.org/ml/gcc-patches/2010-10/msg01104.html I enumerated the cases in GCC where WORDS_BIG_ENDIAN and BYTES_BIG_ENDIAN may differ. The ARM -mwords-little-endian case has now had a deprecation patch submitted http://gcc.gnu.org/ml/gcc-patches/2011-06/msg02217.html and approved (with

Re: [PATCH] Address lowering [1/3] Main patch

2011-07-06 Thread William J. Schmidt
On Wed, 2011-07-06 at 15:16 +0200, Richard Guenther wrote: On Tue, Jul 5, 2011 at 3:59 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: (Sorry for the late response; yesterday was a holiday here.) On Mon, 2011-07-04 at 16:21 +0200, Richard Guenther wrote: On Thu, Jun 30, 2011 at

Re: PATCH TRUNK: [gcc/configure.ac] Generate GCCPLUGIN_VERSION_* macros

2011-07-06 Thread Basile Starynkevitch
On Wed, Jul 06, 2011 at 04:02:48PM +0200, Richard Guenther wrote: On Wed, Jul 6, 2011 at 3:46 PM, Basile Starynkevitch bas...@starynkevitch.net wrote: On Wed, Jul 06, 2011 at 03:21:47PM +0200, Richard Guenther wrote: On Wed, Jul 6, 2011 at 2:50 PM, Basile Starynkevitch

Re: PATCH [1/n] X32: Add initial -x32 support

2011-07-06 Thread H.J. Lu
Hi Paolo, DJ, Nathanael, Alexandre, Ralf, Is the change . * configure.ac: Support --enable-x32. * configure: Regenerated. diff --git a/gcc/configure.ac b/gcc/configure.ac index 5f3641b..bddabeb 100644 --- a/gcc/configure.ac +++ b/gcc/configure.ac @@ -611,6 +611,11 @@

Re: [Patch,testsuite]: target-supports.exp: Disable -fprofile-generate for AVR

2011-07-06 Thread Georg-Johann Lay
Rainer Orth wrote: Georg-Johann Lay writes: Index: lib/target-supports.exp === --- lib/target-supports.exp (revision 175811) +++ lib/target-supports.exp (working copy) @@ -497,6 +497,13 @@ proc check_profiling_available {

Re: [PATCH] Address lowering [1/3] Main patch

2011-07-06 Thread Richard Guenther
On Wed, Jul 6, 2011 at 4:28 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: On Wed, 2011-07-06 at 15:16 +0200, Richard Guenther wrote: On Tue, Jul 5, 2011 at 3:59 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: (Sorry for the late response; yesterday was a holiday here.)

Re: PATCH [1/n] X32: Add initial -x32 support

2011-07-06 Thread Richard Guenther
On Wed, Jul 6, 2011 at 4:48 PM, H.J. Lu hjl.to...@gmail.com wrote: Hi Paolo, DJ, Nathanael, Alexandre, Ralf, Is the change .        * configure.ac: Support --enable-x32.        * configure: Regenerated. diff --git a/gcc/configure.ac b/gcc/configure.ac index 5f3641b..bddabeb 100644 ---

Re: PATCH TRUNK: [gcc/configure.ac] Generate GCCPLUGIN_VERSION_* macros

2011-07-06 Thread Diego Novillo
On Wed, Jul 6, 2011 at 10:37, Basile Starynkevitch bas...@starynkevitch.net wrote: On Wed, Jul 06, 2011 at 04:02:48PM +0200, Richard Guenther wrote: On Wed, Jul 6, 2011 at 3:46 PM, Basile Starynkevitch bas...@starynkevitch.net wrote: On Wed, Jul 06, 2011 at 03:21:47PM +0200, Richard Guenther

Re: [Patch,testsuite]: target-supports.exp: Disable -fprofile-generate for AVR

2011-07-06 Thread Rainer Orth
Georg-Johann Lay a...@gjlay.de writes: Here is a revised patch. Ok? I'd like to defer to the target maintainers here: they know their port, while I don't. Rainer -- - Rainer Orth, Center for Biotechnology,

Re: Provide 64-bit default Solaris/x86 configuration (PR target/39150)

2011-07-06 Thread Eric Botcazou
* In the sparcv9-sun-solaris2.11 builds, the 32-bit libgo tests fail to link since they have unresolved references to __sync_bool_compare_and_swap_8 and __sync_add_and_fetch_8. I could trace this to -mv8plus being missing in that configuration. I'm uncertain where best to handle

Re: Provide 64-bit default Solaris/x86 configuration (PR target/39150)

2011-07-06 Thread Rainer Orth
Eric Botcazou ebotca...@adacore.com writes: * In the sparcv9-sun-solaris2.11 builds, the 32-bit libgo tests fail to link since they have unresolved references to __sync_bool_compare_and_swap_8 and __sync_add_and_fetch_8. I could trace this to -mv8plus being missing in that

Re: [Patch,testsuite]: target-supports.exp: Disable -fprofile-generate for AVR

2011-07-06 Thread Denis Chertykov
2011/7/6 Rainer Orth r...@cebitec.uni-bielefeld.de: Georg-Johann Lay a...@gjlay.de writes: Here is a revised patch. Ok? I'd like to defer to the target maintainers here: they know their port, while I don't. Approved. Denis.

Re: [Path,AVR]: Improve loading of 32-bit constants

2011-07-06 Thread Georg-Johann Lay
Denis Chertykov wrote: 2011/7/6 Georg-Johann Lay a...@gjlay.de: For loading a 32-bit constant in a register, there is room for improvement: * SF can be handled the same way as SI and therefore the patch adds a peep2 to produce a *reload_insf analogon to *reload_insi. * If the destination

[PATCH, testsuite] Fix for PR49519, miscompiled 447.dealII in SPEC CPU 2006

2011-07-06 Thread Kirill Yukhin
Hi, I've prepared a patch for: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49519 I've also prepared a test which reproduces the error. ChangeLog entry: 2011-07-06 Kirill Yukhin kirill.yuk...@intel.com PR tailcall-optimization/49519 * calls.c

Re: Improve Solaris mudflap support (PR libmudflap/49550)

2011-07-06 Thread Frank Ch. Eigler
Hi, Rainer - It addresses a couple of testsuite failures: [...] where the registration of __iob has been done automatically by the compiler. I avoid this problem by not registering stdin, stdout, and stderr separately on Solaris. OK. * Some tests were failing while calling

Re: Improve Solaris mudflap support (PR libmudflap/49550)

2011-07-06 Thread Rainer Orth
Hi Frank, * Some tests were failing while calling unregister in munmap. It turned out that there had been no corresponding mmap registration before. This occurs because Solaris has mmap64 for largefile-aware programs instead. Fixed by wrapping mmap64, too. What I don't know is if

Re: Provide 64-bit default Solaris/x86 configuration (PR target/39150)

2011-07-06 Thread Eric Botcazou
Thanks, I'll give it a try. I just seemd weird to have MASK_V8PLUS and MASK_V9 at the same time. Yes, that's why the existing comment should also be enhanced. I'll fix it. -- Eric Botcazou

[v3] Correctly determine baseline_subdir for 64-bit default Solaris gcc

2011-07-06 Thread Rainer Orth
As alluded to in Provide 64-bit default Solaris/x86 configuration (PR target/39150) http://gcc.gnu.org/ml/gcc-patches/2011-07/msg00327.html (which was meant to be Cc'ed to libstdc++, but bounced due to a stupid typo), there are now to variant bi-arch gcc configurations for

Re: C++ mangling, function name to mangled name (or tree)

2011-07-06 Thread Pierre Vittet
On 06/07/2011 18:25, Kevin André wrote: On Wed, Jul 6, 2011 at 18:00, Pierre Vittetpier...@pvittet.com wrote: I would like user of the plugin to give in arguments the name of the functions on which he would like a test to be run. That means that I must convert the string containing a

Re: Improve Solaris mudflap support (PR libmudflap/49550)

2011-07-06 Thread Frank Ch. Eigler
Hi, Rainer - If so, I'd rather do it by adding some MFWRAP_OS_SPEC to avoid having to duplicate the whole spec in the Solaris config headers. Why would solaris have to duplicate MFWRAP_SPEC if mmap64 is added to the default gcc.c one? I assumed that you wanted to keep the

Re: C++ mangling, function name to mangled name (or tree)

2011-07-06 Thread Romain Geissler
Le 6 juil. 2011 à 18:40, Pierre Vittet a écrit : On 06/07/2011 18:25, Kevin André wrote: On Wed, Jul 6, 2011 at 18:00, Pierre Vittetpier...@pvittet.com wrote: I would like user of the plugin to give in arguments the name of the functions on which he would like a test to be run. That

Re: Improve Solaris mudflap support (PR libmudflap/49550)

2011-07-06 Thread Rainer Orth
Hi Frank, Why would solaris have to duplicate MFWRAP_SPEC if mmap64 is added to the default gcc.c one? I assumed that you wanted to keep the default generic, and meant to separate target specific additions from the generic part. I don't have a strong opinion on this, but if you add

Re: [pph] Stream and restore static_aggregates (issue4626096)

2011-07-06 Thread Gabriel Charette
On Tue, Jul 5, 2011 at 9:08 PM, Diego Novillo dnovi...@google.com wrote:        * pph-streamer-in.c (pph_add_bindings_to_namespace): diff --git a/gcc/cp/pph-streamer-in.c b/gcc/cp/pph-streamer-in.c index 72536a5..0bab93b 100644 --- a/gcc/cp/pph-streamer-in.c +++ b/gcc/cp/pph-streamer-in.c

DOC patch: about gengtype plugins

2011-07-06 Thread Basile Starynkevitch
Hello All, The attached documentation patch is nearly trivial, I was tempted to apply it without review. ### gcc/ChangeLog entry ### 2011-07-06 Basile Starynkevitch bas...@starynkevitch.net * doc/plugins.texi (Building GCC plugins): gengtype needs its gtype.state ### end

Re: [Path,AVR]: Improve loading of 32-bit constants

2011-07-06 Thread Denis Chertykov
2011/7/6 Georg-Johann Lay a...@gjlay.de: Denis Chertykov wrote: 2011/7/6 Georg-Johann Lay a...@gjlay.de: For loading a 32-bit constant in a register, there is room for improvement: * SF can be handled the same way as SI and therefore the patch  adds a peep2 to produce a *reload_insf

[Patch,testsuite]: Filter more test cases to fit target capabilities

2011-07-06 Thread Georg-Johann Lay
Hi, I am struggling against hundreds of fails in the testsuite because many cases are not carefully written, e.g. stull like shifting an int by 19 bits if int is only 16 bits wide. This patch adds some additional tests to avoid FAILs that are confusing. Sorry for gathering it in one patch, other

Re: [testsuite] ARM wmul tests: require arm_dsp_multiply

2011-07-06 Thread Janis Johnson
On 06/29/2011 06:25 AM, Richard Earnshaw wrote: On 23/06/11 22:38, Janis Johnson wrote: Tests wmul-[1234].c and mla-2.c in gcc.target/arm require support that the arm backend identifies as TARGET_DSP_MULTIPLY. The tests all specify a -march option with that support, but it is overridden by

Re: [RFC PATCH, go]: Port to ALPHA arch - sysinfo.go fixup

2011-07-06 Thread Ian Lance Taylor
Uros Bizjak ubiz...@gmail.com writes: On Tue, Jul 5, 2011 at 7:17 PM, Mike Stump mikest...@comcast.net wrote: Please note that we set -mieee flag to compile .go files from library and also we add this flag to default testsuite compile flags. Ick, I think this patch might be expedient,

Re: [RFC PATCH, go]: Port to ALPHA arch - sysinfo.go fixup

2011-07-06 Thread Uros Bizjak
On Wed, Jul 6, 2011 at 7:34 PM, Ian Lance Taylor i...@google.com wrote: This seems like a reasonable patch to me, but technically speaking it is incomplete.  Go should have IEEE floating point behaviour by default.  I believe Java is the same.  Ideally there would be a target-independent way

[testsuite] fixes for gcc.target/arm/mla-1.c

2011-07-06 Thread Janis Johnson
Test gcc.target/arm/mla-1.c scans the assembly file for a string that is part of the name, which always succeeds. This patch adds a tab to the search target to avoid that. It also removes the -march option and pruning of warnings about conflicts, and restricts the tests to targets that support

Re: [Path,AVR]: Improve loading of 32-bit constants

2011-07-06 Thread Georg-Johann Lay
Denis Chertykov wrote: 2011/7/6 Georg-Johann Lay a...@gjlay.de: Denis Chertykov wrote: 2011/7/6 Georg-Johann Lay a...@gjlay.de: For loading a 32-bit constant in a register, there is room for improvement: * SF can be handled the same way as SI and therefore the patch adds a peep2 to

Re: [1/11] Use targetm.shift_truncation_mask more consistently

2011-07-06 Thread Richard Sandiford
Bernd Schmidt ber...@codesourcery.com writes: At some point we've grown a shift_truncation_mask hook, but we're not using it everywhere we're masking shift counts. This patch changes the instances I found. The documentation reads: Note that, unlike @code{SHIFT_COUNT_TRUNCATED}, this function

Re: [7/11] rtl optimizer changes

2011-07-06 Thread Richard Henderson
On 07/01/2011 10:35 AM, Bernd Schmidt wrote: * explow.c (trunc_int_for_mode): Use GET_MODE_PRECISION instead of GET_MODE_BITSIZE where appropriate. * rtlanal.c (subreg_lsb_1, subreg_get_info, nonzero_bits1, num_sign_bit_copies1, canonicalize_condition, low_bitmask_len,

Re: [8/11] Expander changes

2011-07-06 Thread Richard Henderson
On 07/01/2011 10:36 AM, Bernd Schmidt wrote: * optabs.c (expand_binop): Use GET_MODE_PRECISION instead of GET_MODE_BITSIZE where appropriate. (widen_leading, expand_parity, expand_ctz, expand_ffs, expand_unop, expand_abs_nojump, expand_one_cmpl_abs_nojump,

Re: [PATCH] Fix PR49645, with C FE pieces

2011-07-06 Thread H.J. Lu
On Wed, Jul 6, 2011 at 6:26 AM, Richard Guenther rguent...@suse.de wrote: This fixes PR49645 - with MEM_REF the value-numbering machinery to look through aggregate copies wasn't working reliably as we have two representations for X, X and MEM[X].  The following patch fixes that by internally

Re: [9/11] Fix units mismatch in comparison

2011-07-06 Thread Richard Henderson
On 07/01/2011 10:38 AM, Bernd Schmidt wrote: * rtlanal.c (nonzero_bits1): Don't compare GET_MODE_SIZE against a bitsize. Ok. r~

Re: [10/11] Expander fixes for 40-bit integers

2011-07-06 Thread Richard Henderson
On 07/01/2011 10:41 AM, Bernd Schmidt wrote: * optabs.c (expand_binop): Tighten conditions for doubleword expansions. (widen_bswap): Assert that mode bitsize and precision are the same. * stor-layout.c (get_best_mode): Skip modes that have lower precision

Re: [11/11] Fix get_mode_bounds

2011-07-06 Thread Richard Henderson
On 07/01/2011 10:42 AM, Bernd Schmidt wrote: get_mode_bounds should also use GET_MODE_PRECISION, but this exposes a problem on ia64 - BImode needs to be handled specially here to work around another preexisting special case in gen_int_mode. Would it be better to remove the trunc_int_for_mode

libgo patch committed: Fix json test when rand returns 0

2011-07-06 Thread Ian Lance Taylor
This patch is necessary when compiling the test cases with optimization, which changes the order of the calls to rand. I proposed the same patch to the upstream library. Bootstrapped and tested on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 48658f7ed377

Re: [PATCH] Use ira_reg_class_max_nregs array instead of CLASS_MAX_NREGS macro

2011-07-06 Thread Anatoly Sokolov
Hi. The patch http://gcc.gnu.org/ml/gcc-patches/2011-06/msg02405.html is still pending review. Anatoly.

Re: [Path,AVR]: Improve loading of 32-bit constants

2011-07-06 Thread Georg-Johann Lay
Denis Chertykov wrote: I have asked about example of *d instead of !d. Just svn GCC with *d vs svn GCC !d. Denis. Is the patch ok with the original !d instead of *d ? It still improves and the !d vs. *d don't matter because there's always r I think. Johann

[patch] PR other/49658 fix typo in extend.texi

2011-07-06 Thread Jonathan Wakely
Index: extend.texi === --- extend.texi (revision 175887) +++ extend.texi (working copy) @@ -1629,7 +1629,7 @@ char **foo = (char *[]) @{ x, y, z @}; @end smallexample -Compound literals for scalar types and union types are is

Re: [PATCH, testsuite] Fix for PR49519, miscompiled 447.dealII in SPEC CPU 2006

2011-07-06 Thread Eric Botcazou
2011-07-06 Kirill Yukhin kirill.yuk...@intel.com PR tailcall-optimization/49519 Please do not invent components, this will disable the automatic xref of the commit in bugzilla. Copy the Component field of the PR, middle-end here. * calls.c

[PATCH] Fix dead_debug_insert_before ICE (PR debug/49522, take 2)

2011-07-06 Thread Jakub Jelinek
On Tue, Jul 05, 2011 at 10:06:51PM +0200, Jakub Jelinek wrote: On Tue, Jul 05, 2011 at 10:35:11AM +0200, Eric Botcazou wrote: There are two kinds of changes we do on the debug insns without immediate rescanning: 1) reset the debug insn 2) replace a reg use with DEBUG_EXPR of the same

[wwwdocs] Buildstat update for 4.5

2011-07-06 Thread Tom G. Christensen
Latest results for 4.5.x -tgc Testresults for 4.5.3: powerpc-apple-darwin8.11.0 sparc-sun-solaris2.7 Testresults for 4.5.2 powerpc-apple-darwin8.11.0 Testresults for 4.5.1 powerpc-apple-darwin8.11.0 Index: buildstat.html

[wwwdocs] Buildstat update for 4.4

2011-07-06 Thread Tom G. Christensen
Latest results for 4.4.x. -tgc Testresults for 4.4.6: powerpc-apple-darwin8.11.0 Testresults for 4.4.4: powerpc-apple-darwin8.11.0 Index: buildstat.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.4/buildstat.html,v retrieving

Re: [PATCH, go]: Compile go checks with $(GOCFLAGS)

2011-07-06 Thread Ian Lance Taylor
Uros Bizjak ubiz...@gmail.com writes: IMO, it makes sense to compile go tests with the same pack of flags as the library. Additionally, this solves an issue with extra compile flags (i.e. -mieee) that needs to be added to handle NaN/Inf. Patch was tested on x86_64-pc-linux-gnu {, -m32} and

Remove unused t-* fragments

2011-07-06 Thread Joseph S. Myers
This patch removes three unused t-* makefile fragments. (t-pa is unused because no target uses it explicitly and all PA targets define nonempty tmake_file; t-$cpu_type is is only used implicitly if tmake_file is empty after config.gcc.) Bootstrapped with no regressions on

[lra] initial support of debug info and some fixes

2011-07-06 Thread Vladimir Makarov
The following patch contains some of my work since the last lra patch: o initial support of debug info in LRA. o code size improvement for i386 (changes in register banks). o code improvement by better reallocation of non-reload pseudos in LRA. by using allocno class which is usually wider

Re: Remove unused t-* fragments

2011-07-06 Thread Uros Bizjak
On Wed, Jul 6, 2011 at 10:14 PM, Joseph S. Myers jos...@codesourcery.com wrote: This patch removes three unused t-* makefile fragments.  (t-pa is unused because no target uses it explicitly and all PA targets define nonempty tmake_file; t-$cpu_type is is only used implicitly if tmake_file is

Re: [PATCH] Fix dead_debug_insert_before ICE (PR debug/49522)

2011-07-06 Thread Eric Botcazou
So you prefer something like this (untested) instead? I think that, ideally, we should avoid leaving the dead_debug chain in the semi-broken state that we currently have. Without the second loop I have no idea how to make it work in dead_debug_reset, the other dead_debug_use referencing the

Re: [PATCH] Fix dead_debug_insert_before ICE (PR debug/49522, take 2)

2011-07-06 Thread Eric Botcazou
And here is a version that passed bootstrap/regtest on x86_64-linux and i686-linux: 2011-07-06 Jakub Jelinek ja...@redhat.com PR debug/49522 * df-problems.c (dead_debug_reset): Remove dead_debug_uses referencing debug insns that have been reset.

[pph] Add FIXME comment to avoid finalizing decls when generating pph image. (issue4626099)

2011-07-06 Thread Gabriel Charette
We do not need to finalize decls and add them to the varpool when generating the pph image as we will do this when streaming in (lto also already does it this way). I simply added a comment for now, because this will not fix anything, it will simply avoid streaming out unecessary stuff. Since

Re: [Patch, Fortran] Register allocatable coarrays.

2011-07-06 Thread Tobias Burnus
Daniel Carrera wrote: On 07/05/2011 09:57 AM, Tobias Burnus wrote: On 07/04/2011 11:34 PM, Daniel Carrera wrote: The test compiles, but there are expected failures because gcc doesn't think that allocatable scalar coarrays are supported. [...] I really don't want to add knowingly a failing

Re: [Patch, Fortran] Register allocatable coarrays.

2011-07-06 Thread Steve Kargl
On Wed, Jul 06, 2011 at 10:57:35PM +0200, Tobias Burnus wrote: I have now committed it as Rev. 175937 Does Daniel have write-after-approval svn access? If not, we should probably get him access. -- Steve

PING: PATCH [8/n]: Prepare x32: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD sizeof (void *)

2011-07-06 Thread H.J. Lu
PING. On Thu, Jun 30, 2011 at 1:47 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Jun 30, 2011 at 12:02 PM, Richard Henderson r...@redhat.com wrote: On 06/30/2011 11:23 AM, H.J. Lu wrote: +#ifdef REG_VALUE_IN_UNWIND_CONTEXT +typedef _Unwind_Word _Unwind_Context_Reg_Val; +/* Signal frame

Re: [pph] Test cleanup (issue4572050)

2011-07-06 Thread Gabriel Charette
After having a look at how pph.exp works last Friday I think I could do this myself easily enough. Or are you still modifying the tests and want me to avoid touching this for now? Gab On Fri, Jul 1, 2011 at 5:51 PM, Lawrence Crowl cr...@google.com wrote: On 7/1/11, Gabriel Charette

[PATCH] Add -mno-r11 option to suppress load of ppc64 static chain in indirect calls

2011-07-06 Thread Michael Meissner
This patch adds an option to not load the static chain (r11) for 64-bit PowerPC calls through function pointers (or virtual function). Most of the languages on the PowerPC do not need the static chain being loaded when called, and adding this instruction can slow down code that calls very short

Re: [PATCH] [Annotalysis] Change to get_virtual_function_decl

2011-07-06 Thread Diego Novillo
On Wed, Jul 6, 2011 at 17:56, Delesley Hutchins deles...@google.com wrote: This patch modifies the behavior of cp_get_virtual_function_decl in gcc/cp/class.c so that it returns NULL if the function declaration cannot be found.  The previous behavior was to fail with a segmentation fault.  The

[testsuite] arm tests: remove -march= and dg-prune-output from 3 tests

2011-07-06 Thread Janis Johnson
For three tests in gcc.target/arm that don't depend on processor-specific behavior, don't specify the -march option. This makes dg-prune-output for warnings about conflicts unnecessary, so remove it. Two of these tests are for internal compiler errors that showed up with particular values of

[testsuite] arm thumb tests: remove -march= and dg-prune-output from 9 tests

2011-07-06 Thread Janis Johnson
This patch removes -march= from nine tests that also check for relevant effective targets. If -march is removed there is no need to ignore compiler warnings about conflicting options with dg-prune-output, so the patch removes that from the tests. OK for trunk, and for 4.6 in a few days?

Re: [pph] Add FIXME comment to avoid finalizing decls when generating pph image. (issue4626099)

2011-07-06 Thread Diego Novillo
On Wed, Jul 6, 2011 at 16:55, Gabriel Charette gch...@google.com wrote: +2011-07-06  Gabriel Charette  gch...@google.com + +       * passes.c (rest_of_decl_compilation): Add FIXME pph comment. + OK under the obvious rule (small patches like this one that make obvious fixes to documentation

Re: [11/11] Fix get_mode_bounds

2011-07-06 Thread Richard Henderson
On 07/06/2011 04:04 PM, Bernd Schmidt wrote: That might require target specific changes if there are assumptions that a BImode value is either 0 or 1, not 0 or -1. For now I'd prefer to minimize the impact. Systems that set STORE_FLAG_VALUE to -1: m68k spu Systems that use

[PATCH] Update html docs for -mno-r11 and --param case-value-threshold

2011-07-06 Thread Michael Meissner
I updated the html documents for my two recent changes: *** changes.html.~1~2011-07-06 19:26:37.0 -0400 --- changes.html2011-07-06 19:35:22.0 -0400 *** *** 48,54 h2General Optimizer Improvements/h2 ul ! li.../li /ul h2New

Re: [1/11] Use targetm.shift_truncation_mask more consistently

2011-07-06 Thread Bernd Schmidt
On 07/06/11 20:06, Richard Sandiford wrote: Bernd Schmidt ber...@codesourcery.com writes: At some point we've grown a shift_truncation_mask hook, but we're not using it everywhere we're masking shift counts. This patch changes the instances I found. The documentation reads: Note that,

C++ PATCH for c++/49353 (emitting functions with DECL_EXTERNAL set)

2011-07-06 Thread Jason Merrill
The C++ front end sets DECL_EXTERNAL on functions and variables with vague linkage during most of compilation, and then clears the flag at EOF if we actually want to emit them. But we were failing to clear DECL_EXTERNAL in the case of inlines that we are emitting because of

[Patch, Fortran] Add stat=/errmsg= support to _gfortran_caf_register

2011-07-06 Thread Tobias Burnus
This patch cleans up the ABI mess, I created at some point. The initial version of _gfortran_caf_register didn't handle stat/errmsg as one could leave it to the front end: The the returned memory is NULL, it's an error. However, as Nick pointed out, for stat= one can also return