Re: RFC: PR rtl-optimization/52876: Sign extend 32 to 64bit then clear upper 32bits fails O1 or higher

2012-04-11 Thread Richard Sandiford
"H.J. Lu" writes: > Here is the updated patch to change set_reg_attrs_from_value to > take arbitrary value and check incompatible pointer sign > extension. OK for trunk if there are no regressions on Linux/x86-64? The new code ought to go before the HARD_REGISTER_P check, not after it. OK with t

Re: [i386, patch, RFC] HLE support in GCC

2012-04-11 Thread Uros Bizjak
On Wed, Apr 11, 2012 at 6:06 PM, Andi Kleen wrote: > +  static char buf[128], hle[16]; > > The hle buffer does not need to be static. > BTW I'm surprised there is no better way to do this in machine descriptions > than to use static buffers. Oh, there is. Since we are looking at the operands, we

Re: Setting precision for a PSImode type

2012-04-11 Thread Richard Sandiford
Bernd Schmidt writes: > On 04/11/2012 07:31 PM, Peter Bigot wrote: >> The biggest one is that widening from 20-bit to 32-bit is an extremely >> expensive operation: it was a 16-bit ISA, but some newer MCUs support >> an extension with 20 bits in each register and a set of new >> instructions that

[SH] PR 50751- add HImode testcases

2012-04-11 Thread Oleg Endo
Hi, The attached patch adds some more test cases for PR 50751. Tested on sh-sim with make check-gcc RUNTESTFLAGS="sh.exp=pr50751* --target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a-single/-mb,-m4-single/-ml, -m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" to confirm that the tests pass as expected.

Re: Setting precision for a PSImode type

2012-04-11 Thread Bernd Schmidt
On 04/11/2012 09:15 PM, Richard Sandiford wrote: > Bernd Schmidt writes: >> On 04/11/2012 07:31 PM, Peter Bigot wrote: >>> The biggest one is that widening from 20-bit to 32-bit is an extremely >>> expensive operation: it was a 16-bit ISA, but some newer MCUs support >>> an extension with 20 bits

Re: PATCH: Define TRY_EMPTY_VM_SPACE for Linux/x32

2012-04-11 Thread H.J. Lu
On Tue, Apr 10, 2012 at 7:41 AM, H.J. Lu wrote: > On Thu, Apr 5, 2012 at 8:57 AM, H.J. Lu wrote: >> On Thu, Apr 5, 2012 at 8:36 AM, Uros Bizjak wrote: >>> On Thu, Apr 5, 2012 at 3:28 PM, H.J. Lu wrote: >>> >>> Looking at how other targets implement this check, I don't think that >>> thi

Re: [PATCH] Caret diagnostics

2012-04-11 Thread Mike Stump
On Apr 11, 2012, at 11:15 AM, Manuel López-Ibáñez wrote: > contrib/compare_tests does not seem to detect this :-( I'd bet, either, there were no tests after that point, or, compare_tests pointed it out. Feel free to send the two .sum files and and I will investigate if you think that isn't the

[PATCH, powerpc] Fix PR52775, enable FCFID on power4

2012-04-11 Thread Michael Meissner
It was brought to my attention that when I rewrote the floating point conversion operations for power7, I did not notice that the power4 and 970 powerpc's actually support the FCFID (floating point convert) instruciton in 32-bit mode. This patch fixes it. It is ok to apply? I did bootstraps with

PATCH: Simplify TRY_EMPTY_VM_SPACE for Linux hosts

2012-04-11 Thread H.J. Lu
Hi, This patch simplifies TRY_EMPTY_VM_SPACE for Linux hosts by checking pointer size when appropriate. Tested on Linux/x86-64, Linux/ia32 and Linux/x32. OK for trunk? Thanks. H.J. --- 2012-04-11 H.J. Lu * config/host-linux.c (TRY_EMPTY_VM_SPACE): Check pointer size for alp

[v3] fix broken performance test

2012-04-11 Thread Jonathan Wakely
This fixes a check-performance failure caused by my changes for libstdc++/49204 * testsuite/performance/30_threads/future/polling.cc: Adjust. Tested x86_64-linux, committing to trunk and 4.7 branch. commit fbb91a75e318226e666967a28883483a027f1f07 Author: Jonathan Wakely Date: Wed Apr 1

Re: [PATCH] Caret diagnostics

2012-04-11 Thread Manuel López-Ibáñez
On 11 April 2012 18:40, H.J. Lu wrote: > > It breaks library tests: > > ERROR: tcl error sourcing > ../../../../src-trunk/boehm-gc/testsuite/../../gcc/testsuite/lib/prune.exp. > ERROR: tcl error sourcing > ../../../../src-trunk/libgomp/testsuite/../../gcc/testsuite/lib/prune.exp. > ERROR: tcl erro

[SH] Use more bool, NULL_RTX

2012-04-11 Thread Oleg Endo
Hello, The attached patch make the code use some more 'bool' and 'NULL_RTX'. Tested with 'make all-gcc'. OK? ChangeLog: * config/sh/sh.h (RETURN_ADDR_RTX): Use NULL_RTX instead of 0. * config/sh/sh.c (INSN_REGMODE_WEIGHT, CURR_REGMODE_PRESSURE): Fix line width. (d

[IA-64] Fix PR target/48496

2012-04-11 Thread Eric Botcazou
Hi, this is the spurious error on asm statements of the form: error: 'asm' operand requires impossible reload present for IA-64 on mainline and 4.7 branch. As diagnosed by Ulrich, the code responsible for the error implicitly assumes that constraints accepting memory operands also accept ps

Re: [i386, patch, RFC] HLE support in GCC

2012-04-11 Thread Andi Kleen
> > That's true. Actually I see the values are defined by the compiler > > at compile time, so it would be possible to move all one up? > > No, that is IMHO not possible. They need to match the enum values that are > part of libstdc++ ABI already, and not everybody is going to use the > __ATOMIC_

[patch] Update email address

2012-04-11 Thread Steve Ellcey
I have checked in the following patch to update my email address. I am currently working on getting a new copyright assignment filed as my previous one was based on my employment at HP. Steve Ellcey sell...@mips.com 2012-04-11 Steve Ellcey * MAINTAINERS: Changed email address. Ind

Re: [SH] PR 50751- add HImode testcases

2012-04-11 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch adds some more test cases for PR 50751. > > Tested on sh-sim with > make check-gcc RUNTESTFLAGS="sh.exp=pr50751* --target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a-single/-mb,-m4-single/-ml, > -m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" > > to confirm that

Re: [SH] Use more bool, NULL_RTX

2012-04-11 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch make the code use some more 'bool' and 'NULL_RTX'. > Tested with 'make all-gcc'. > OK? OK. Regards, kaz

Re: [Patch, Fortran, F03] PR52909: Procedure pointers not private to modules

2012-04-11 Thread Janus Weil
>> 3) As you mentioned, the .mod version incompatibility also severely >> limits the mixing of code compiled with different compiler versions. >> And the proc-pointer name mangling (which is under discussion here) >> *only* concerns proc-pointers inside modules. > > Note however, that GCC 4.7 and 4

m68k: add bswapsi2 expander

2012-04-11 Thread Andreas Schwab
The default expansion of bswapsi2 (and bswapdi2) is pretty suboptimal. Tested on m68k-linux. Andreas. * config/m68k/m68k.md (rotrhi3+1): Name it rotrhi_lowpart. (bswapsi2): New expander. diff --git a/gcc/config/m68k/m68k.md b/gcc/config/m68k/m68k.md index e4b4b59..8104e75 100644

[v3] tweak docs

2012-04-11 Thread Jonathan Wakely
I think this makes the text read a bit better. * doc/xml/manual/debug.xml (Debug Versions of Library Binary Files): Re-arrange text slightly. Committed to trunk commit 74b28e0fa40289525b44c79920cbd64a36a0cb52 Author: Jonathan Wakely Date: Wed Apr 11 23:13:49 2012 +0100

Re: RFC: IRA patch to reduce lifetimes

2012-04-11 Thread Vladimir Makarov
On 04/11/2012 10:35 AM, Bernd Schmidt wrote: On 12/23/2011 05:31 PM, Vladimir Makarov wrote: On 12/21/2011 09:09 AM, Bernd Schmidt wrote: This patch was an experiment to see if we can get the same improvement with modifications to IRA, making it more tolerant to over-aggressive scheduling. THe

Re: [i386, patch, RFC] HLE support in GCC

2012-04-11 Thread Torvald Riegel
On Wed, 2012-04-11 at 15:06 +0200, Andi Kleen wrote: > > Tests passing, bootstrap in progress. > > > > Comments? > > Do you really imply ACQUIRE/RELEASE with HLE_ACQUIRE/RELEASE now? I don't > see that in the code. I think that's really required, otherwise the optimizer > will do the wrong thing

fix libstdc++/52924

2012-04-11 Thread Jonathan Wakely
PR libstdc++/52924 * include/bits/shared_ptr_base.h (_Sp_counted_deleter): Add user-defined destructor. (_Sp_counted_inplace): Likewise. * testsuite/20_util/shared_ptr/cons/52924.cc: New. * testsuite/20_util/shared_ptr/cons/43820_neg.cc: Adjust dg-err

Re: Phone call (was Re: Armhf dynamic linker path)

2012-04-11 Thread Michael Hope
On 12 April 2012 10:38, Steve McIntyre wrote: > On Wed, Apr 11, 2012 at 02:06:09AM +0100, Steve McIntyre wrote: >> >>And here's the details as promised. >> >>I've started a wiki page at >> >>https://wiki.linaro.org/OfficeofCTO/HardFloat/LinkerPathCallApr2012 >> >>with a strawman agenda for now, an

Re: Phone call (was Re: Armhf dynamic linker path)

2012-04-11 Thread Michael Hope
2012/4/12 Paulo César Pereira de Andrade : > Em 11 de abril de 2012 20:22, Michael Hope escreveu: >> On 12 April 2012 10:38, Steve McIntyre wrote: >>> On Wed, Apr 11, 2012 at 02:06:09AM +0100, Steve McIntyre wrote: And here's the details as promised. I've started a wiki page at

Re: Phone call (was Re: Armhf dynamic linker path)

2012-04-11 Thread Michael Hope
On 12 April 2012 12:38, Wookey wrote: > +++ Michael Hope [2012-04-12 12:16 +1200]: >> 2012/4/12 Paulo César Pereira de Andrade >> : > >> >> All good.  My vote is for /lib/ld-arm-linux-gnueabihf.so.3 as it: >> >  Sorry for more bikeshedding, >> > /lib/ld-linux-armv7hl.so.3 >>  I'd rather drop the '

Re: Phone call (was Re: Armhf dynamic linker path)

2012-04-11 Thread Michael Hope
2012/4/12 Paulo César Pereira de Andrade : > Em 11 de abril de 2012 21:16, Michael Hope escreveu: >> 2012/4/12 Paulo César Pereira de Andrade >> : >>> Em 11 de abril de 2012 20:22, Michael Hope >>> escreveu: On 12 April 2012 10:38, Steve McIntyre wrote: > On Wed, Apr 11, 2012 at 02:06:

[PATCH] [ARM] thumb1 imm move 256-510

2012-04-11 Thread Joey Ye
For thumb1 use move + add instructions for immediate move [256-510]. Following is a complete range if combine an imm mov with listed instructions. Among them, lsls and neg have already been implemented. The only missing opportunity is add, in which I enabled in this patch. Others are replicated w

Re: Phone call (was Re: Armhf dynamic linker path)

2012-04-11 Thread Jakub Jelinek
On Thu, Apr 12, 2012 at 11:22:13AM +1200, Michael Hope wrote: > All good. My vote is for /lib/ld-arm-linux-gnueabihf.so.3 as it: The directory should be /libhf/ or /libhfp/ for that for consistency with all the other architectures. Note e.g. x86_64 dynamic linker is /lib64/ld-linux-x86-64.so.2,

<    1   2