Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
On 8/4/2012 13:24, H.J. Lu wrote: > On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu wrote: >> On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote: >>> On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: OK. Btw, I don't actually have write access, so please commit if the ChangeLog is good. >>

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread H.J. Lu
On Fri, Aug 3, 2012 at 10:36 PM, Ian Lance Taylor wrote: > On Fri, Aug 3, 2012 at 10:24 PM, H.J. Lu wrote: >> On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu wrote: >>> On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote: On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: > > OK. Btw, I don'

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
On 8/4/2012 13:14, H.J. Lu wrote: > On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote: >> On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: >>> >>> OK. Btw, I don't actually have write access, so please commit if the >>> ChangeLog is good. >>> >>> 2012-08-03 Jonathan Yong >>> >>> * Makefi

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread Ian Lance Taylor
On Fri, Aug 3, 2012 at 10:24 PM, H.J. Lu wrote: > On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu wrote: >> On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote: >>> On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: OK. Btw, I don't actually have write access, so please commit if the Change

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread H.J. Lu
On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu wrote: > On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote: >> On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: >>> >>> OK. Btw, I don't actually have write access, so please commit if the >>> ChangeLog is good. >>> >>> 2012-08-03 Jonathan Yong >>> >>>

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread H.J. Lu
On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote: > On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: >> >> OK. Btw, I don't actually have write access, so please commit if the >> ChangeLog is good. >> >> 2012-08-03 Jonathan Yong >> >> * Makefile.in (version): set to BASE-VER file from

[PATCH, MIPS] Netlogic XLR tuning tweaks

2012-08-03 Thread Sandra Loosemore
We noticed that, per the XLR processor documentation, clz has a different latency than other alu instructions. Also, the branch-likely instructions are documented as obsolete on this target. This patch makes some fairly obvious tweaks to address both issues. This is another patch we've had

Re: [RFA, patch] Add missing source location info to thunks

2012-08-03 Thread Cary Coutant
Following up with a simple test case. If the patch looks OK, I'll work this into a dejagnu test and add it to the patch. class A { public: A(); virtual ~A(); private: int i; }; class B { public: B(); virtual ~B(); private: int i; }; class C : public A, public B { public: C();

Re: [libiberty] add obstack macros (was Re: PR #53525 - track-macro-expansion performance regression)

2012-08-03 Thread Ian Lance Taylor
> 2012-08-04 Dimitrios Apostolou > > * libiberty.h > (XOBDELETE,XOBGROW,XOBGROWVEC,XOBSHRINK,XOBSHRINKVEC): New > type-safe macros for obstack allocation. > (XOBFINISH): Renamed argument to PT since it is a pointer to T. > +/* Type-safe obstack allocator. You must

Re: [PATCH,mmix] convert to constraints.md

2012-08-03 Thread Hans-Peter Nilsson
On Thu, 2 Aug 2012, Nathan Froyd wrote: > As $SUBJECT says. There's not too much interesting here. I did a > fairly literal-minded conversion, so it's possible there's smarter ways > to do some things. Doesn't look too bad though, but ... > Compiled with cross to mmix-knuth-mmixware and spot-ch

[libiberty] add obstack macros (was Re: PR #53525 - track-macro-expansion performance regression)

2012-08-03 Thread Dimitrios Apostolou
Hi Dodji, I appreciate your review but I'm replying for now only for the libiberty.h part (attached updated patch), which I've needed elsewhere and seems the easiest to quickly apply. On Wed, 18 Jul 2012, Dodji Seketeli wrote: === modified file 'include/libiberty.h' --- include/libiberty.h 2

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread Ian Lance Taylor
On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: > > OK. Btw, I don't actually have write access, so please commit if the > ChangeLog is good. > > 2012-08-03 Jonathan Yong > > * Makefile.in (version): set to BASE-VER file from gcc > directory. Committed. Thanks. Ian

[RFA, patch] Add missing source location info to thunks

2012-08-03 Thread Cary Coutant
GCC generates non-virtual thunks directly to assembly code, which causes a couple of problems. First, it doesn't add source location information to the thunk, so the thunk simply inherits the random location from the end of the function in front of it (the function it's a thunk for). In two differe

[google] Remove deprecated pfmon-specific functions/structs from pmu-profile.c (issue6442086)

2012-08-03 Thread Chris Manghane
This patch removes functions and structs necessary for gathering pmu info with pfmon, which is now a deprecated tool. For google/main. Tested with bootstrap. 2012-08-03 Chris Manghane * libgcc/pmu-profile.c (enum pmu_tool_type): Remove pfmon-specific functions/strucs.

Re: [RFC C++ / PR51033 ] Handle __builtin_shuffle in constexpr properly in the C++ frontend.

2012-08-03 Thread Marc Glisse
On Sat, 28 Jul 2012, Marc Glisse wrote: On Mon, 18 Jun 2012, Ramana Radhakrishnan wrote: This patch following on from the fix for turning on __builtin_shuffle for c++ , enables folding of vec_perm_exprs in the front-end for constexpr and constructor style values. Hello, I took a look, and t

Re: [google][gcc-4.7]Revert Old CPU Runtime Detection Implementation (issue6458081)

2012-08-03 Thread Paul Pluzhnikov
On Fri, Aug 3, 2012 at 2:56 PM, Sriraman Tallam wrote: > This patch reverts the original implementation of CPU runtime detection Ok for google/gcc-4_7 branch -- Paul Pluzhnikov

[google][gcc-4.7]Revert Old CPU Runtime Detection Implementation (issue6458081)

2012-08-03 Thread Sriraman Tallam
This patch reverts the original implementation of CPU runtime detection in order to allow the new implementation checked in to trunk at rev. 188417 to be merged. The original implementation was obtained by backporting rev 185083 from google/main. After committing this, I will send a new patch i

[PATCH, MIPS] fix clear cache test cases

2012-08-03 Thread Sandra Loosemore
Some MIPS configurations #define CACHE_FLUSH_FUNC to something other than "_flush_cache", This fixes the two test cases that currently have that name wired in to know about the netbsd and sdemtk names too. OK for mainline? -Sandra 2012-08-03 Sandra Loosemore Catherine Moore

Re: cosmetic change - simplify cse.c:preferable()

2012-08-03 Thread Dimitrios Apostolou
On Thu, 19 Jul 2012, Richard Guenther wrote: I don't think it's any good or clearer to understand. Hi Richi, I had forgotten I prepared this for PR #19832, maybe you want to take a look. FWIW, with my patch applied there is a difference of ~3 M instr, which is almost unmeasurable in time. Bu

[rtl, i386] combine concat+permutation

2012-08-03 Thread Marc Glisse
Hello, this is a follow up to the patch applied after this discussion: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00504.html It handles the -mavx __builtin_shuffle case mentioned there. It passes bootstrap (languages=c,c++) and regtest on x86_64. 2012-08-04 Marc Glisse gcc/ * si

[lra] patch to fix ppc bootstrap after last merge

2012-08-03 Thread Vladimir Makarov
The following patch fixes a bug occuring during the ppc bootstrap. The patch was successfully bootstrapped on x86/x86-64 and ppc. Committed as rev. 190133. 2012-08-03 Vladimir Makarov * lra-assign.c (find_hard_regno_for): Check paradoxical subregs. Index: lra-assigns.c =

[lra] merged with trunk

2012-08-03 Thread Vladimir Makarov
LRA branch has been merged with trunk at 190125. To successfully bootstrap the branch the following changes were applied. The branch was successfully boostrapped on x86-64. 2012-08-02 Vladimir Makarov * lra-constraints.c: Use debug_rtl_slim instead of print_rtl_slim.

Re: Reload/i386 patch for secondary memory vs subregs

2012-08-03 Thread Uros Bizjak
On Fri, Aug 3, 2012 at 10:26 PM, Bernd Schmidt wrote: > On 08/03/2012 08:50 PM, Uros Bizjak wrote: >> Hello! >> >>> Bootstrapped and tested on i686-linux. It's also been in several of our >>> internal trees, going back to even 4.4-based ones IIRC, and has had >>> testing for several architectures.

Re: Reload/i386 patch for secondary memory vs subregs

2012-08-03 Thread Bernd Schmidt
On 08/03/2012 08:50 PM, Uros Bizjak wrote: > Hello! > >> Bootstrapped and tested on i686-linux. It's also been in several of our >> internal trees, going back to even 4.4-based ones IIRC, and has had >> testing for several architectures. Ok for the i386 part? I intend to >> check the reload bits i

Re: [ARM Patch 1/3]PR53189: optimizations of 64bit logic operation with constant

2012-08-03 Thread Ramana Radhakrishnan
On 18 July 2012 11:12, Carrot Wei wrote: > On Wed, Jul 18, 2012 at 5:39 PM, Ramana Radhakrishnan > wrote: >> On 18 July 2012 09:20, Carrot Wei wrote: >>> On Tue, Jul 17, 2012 at 9:47 PM, Ramana Radhakrishnan >>> wrote: Carrot, Sorry about the delayed response. On 3 July

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Andrew MacLeod
On 08/03/2012 02:14 PM, Uros Bizjak wrote: On Fri, Aug 3, 2012 at 8:11 PM, Richard Henderson wrote: On 2012-08-03 11:02, Uros Bizjak wrote: Nice, so we can go with full [snip] Unless we take Andrew's suggestion to simply do this in optabs.c instead. Which is probably better. Indeed. However,

Re: Reload/i386 patch for secondary memory vs subregs

2012-08-03 Thread Uros Bizjak
Hello! > Bootstrapped and tested on i686-linux. It's also been in several of our > internal trees, going back to even 4.4-based ones IIRC, and has had > testing for several architectures. Ok for the i386 part? I intend to > check the reload bits in soon if there are no objections. Index: gcc/conf

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Uros Bizjak
On Fri, Aug 3, 2012 at 8:11 PM, Richard Henderson wrote: > On 2012-08-03 11:02, Uros Bizjak wrote: >> Nice, so we can go with full [snip] > > Unless we take Andrew's suggestion to simply do this in optabs.c instead. > Which is probably better. Indeed. However, I'm not that familiar with this part

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Richard Henderson
On 2012-08-03 11:02, Uros Bizjak wrote: > Nice, so we can go with full [snip] Unless we take Andrew's suggestion to simply do this in optabs.c instead. Which is probably better. r~

Re: [ARM] Use UBFX for some and-immediate operations

2012-08-03 Thread Richard Henderson
On 2012-08-02 09:24, Richard Earnshaw wrote: > + /* Extz only supports SImode, but we can coerce the operands > +into that mode. */ > + emit_constant_insn (cond, > + gen_extzv_t2 (gen_lowpart (mode, target), > +

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Uros Bizjak
On Fri, Aug 3, 2012 at 5:08 PM, Richard Henderson wrote: The same reasoning goes for dynamic negation: for neg %eax,%eax value 0x8000 stays the same, but we have changed (x)sub to an (x)add in the code stream. >>> >>> So? Did you think the xadd will trap? >> >> No, but can we

[PATCH, i386]: Improve sync inc/dec selection, add %K RTM operand modifier

2012-08-03 Thread Uros Bizjak
Hello! Small improvement in inc/dec selection, added forgotten %K operand modifier. 2012-08-03 Uros Bizjak * config/i386/sync.md (*atomic_fetch_add_cmp): Use incdec_operand predicate to return INC or DEC insn mnemonic. Add %K operand modifier. (atomic_add): Use

[PATCH, MIPS] diagnose -fpic/-fpie incompatibility with -mno-abicalls

2012-08-03 Thread Sandra Loosemore
For all supported MIPS ABIs other than EABI, compiling with -fpic/-fPIC/-fpie/-fPIE implicitly requires abicalls support. On Linux targets, -mabicalls is the default, but bare-metal targets like mips-sde-elf default to -mno-abicalls and naively compiling with -fPIC alone doesn't result in work

Re: [patch, fortran] Fix PR 54033, problems with -I, with test cases

2012-08-03 Thread Thomas Koenig
Hi, OK for trunk? I will be on holiday starting tomorrow, so I would like to commit this today, if possible (with a mistake in the ChangeLog noted by Uros fixed. Thanks!) If not, somebody please commit this (or another fix). Thomas

Re: ping again - Show hash table stats when -fmem-report

2012-08-03 Thread Dimitrios Apostolou
I'm always forgetting something, now it was the changelog, see attached (same as old, nothing significant changed). On Fri, 3 Aug 2012, Dimitrios Apostolou wrote: Hi, I've updated this patch to trunk and rebootstrapped it, so I'm resubmitting it, I'm also making a try to CC all relevant mainta

RE: [committed] PR 51931: force non-MIPS16ness for long-branch tests (NOW RFA: MIPS16 Long Branch Patch)

2012-08-03 Thread Moore, Catherine
> -Original Message- > From: Richard Sandiford [mailto:rdsandif...@googlemail.com] > Sent: Thursday, August 02, 2012 5:33 PM > To: Moore, Catherine > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [committed] PR 51931: force non-MIPS16ness for long-branch tests > (NOW RFA: MIPS16 Long Branch Pa

[PATCH, committed] Fix PR53773

2012-08-03 Thread William J. Schmidt
Change this test case to use the optimized dump so that the unreliable vect-details dump can't cause different behavior on different targets. Verified on powerpc64-unknown-linux-gnu, committed as obvious. Thanks, Bill 2012-08-03 Bill Schmidt * testsuite/gcc.dg/vect/pr53773.c: Change

[google/main] Add powerpc-grtev3-linux-gnu.xfail to contrib/testsuite-management (issue6447087)

2012-08-03 Thread Simon Baldwin
Add powerpc-grtev3-linux-gnu.xfail to contrib/testsuite-management. Tested with build followed by validate_failures.py. 2012-08-03 Simon Baldwin * testsuite-management/powerpc-grtev3-linux-gnu.xfail: New. Index: contrib/testsuite-management/powerpc-grtev3-linux-gnu.xfail ===

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
On 8/3/2012 23:35, Ian Lance Taylor wrote: > On Fri, Aug 3, 2012 at 8:02 AM, JonY wrote: >> On 8/3/2012 22:37, Ian Lance Taylor wrote: >>> On Fri, Aug 3, 2012 at 7:01 AM, JonY wrote: For correctness sake, determine version from BASE-VER file instead of calling compiler. This allows

Re: ping again - Show hash table stats when -fmem-report

2012-08-03 Thread Dimitrios Apostolou
Hi, I've updated this patch to trunk and rebootstrapped it, so I'm resubmitting it, I'm also making a try to CC all relevant maintainers, sorry for spamming but even though this patch is simple, it touches many parts. I'm also adding stats to pointer_{set,map} but it will come in a separate p

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-03 Thread Martin Jambor
On Thu, Aug 02, 2012 at 09:28:11PM +0200, Martin Jambor wrote: > Hi, > > this patch uses the aggregate jump functions created by the previous > patch in the series to determine benefits of inlining a particular > call graph edge. It has not changed much since the last time I posted > it, except f

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread Ian Lance Taylor
On Fri, Aug 3, 2012 at 8:02 AM, JonY wrote: > On 8/3/2012 22:37, Ian Lance Taylor wrote: >> On Fri, Aug 3, 2012 at 7:01 AM, JonY wrote: >>> >>> For correctness sake, determine version from BASE-VER file instead of >>> calling compiler. This allows better toleration with bad Canadian cross >>> set

Add --no-sysroot-suffix driver option

2012-08-03 Thread Joseph S. Myers
Some GCC configurations use SYSROOT_SUFFIX_SPEC (generally generated by gcc/config/print-sysroot-suffix.sh or a variant on that script) to handle multilibs each of which uses its own sysroot to have its own variant of the C library. Usually such variants, built for different -mcpu= or similar opti

Re: [Test] Fix for PRPR53981

2012-08-03 Thread Janis Johnson
Yes, you can put it on the 4.6 branch. Janis On 08/03/2012 03:09 AM, Anna Tikhonova wrote: > Ping > > 2012/7/27 Anna Tikhonova : >> Kirill has already checked it in for me, thanks! >> Btw, I have FSF copyright assignment. >> >> Is it possible to commit this patch to 4.6 branch? Since current ndk

Re: Value type of map need not be default copyable

2012-08-03 Thread Ollie Wild
On Fri, Aug 3, 2012 at 2:39 AM, Paolo Carlini wrote: > > Ok, but, can you also double check and in case fix unordered_map too? > Looks like we have the same issue, right? Indeed, we do. I'll send a separate patch for the unordered_map problem. > > Thanks! > Paolo. > > PS: remember that all the

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Richard Henderson
On 2012-08-03 08:01, Uros Bizjak wrote: > On Fri, Aug 3, 2012 at 4:40 PM, Richard Henderson wrote: >> On 2012-08-03 01:51, Uros Bizjak wrote: >>> The same reasoning goes for dynamic negation: for neg %eax,%eax value >>> 0x8000 stays the same, but we have changed (x)sub to an (x)add in >>> the

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
On 8/3/2012 22:37, Ian Lance Taylor wrote: > On Fri, Aug 3, 2012 at 7:01 AM, JonY wrote: >> >> For correctness sake, determine version from BASE-VER file instead of >> calling compiler. This allows better toleration with bad Canadian cross >> setups where host gcc version is of different version f

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Andrew MacLeod
On 08/03/2012 10:40 AM, Richard Henderson wrote: On 2012-08-03 01:51, Uros Bizjak wrote: The same reasoning goes for dynamic negation: for neg %eax,%eax value 0x8000 stays the same, but we have changed (x)sub to an (x)add in the code stream. So? Did you think the xadd will trap? Is there

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Uros Bizjak
On Fri, Aug 3, 2012 at 4:40 PM, Richard Henderson wrote: > On 2012-08-03 01:51, Uros Bizjak wrote: >> The same reasoning goes for dynamic negation: for neg %eax,%eax value >> 0x8000 stays the same, but we have changed (x)sub to an (x)add in >> the code stream. > > So? Did you think the xadd w

Re: [Patch ARM 1/6] Canonicalize neon_vaba and neon_vabal patterns.

2012-08-03 Thread Richard Earnshaw
On 30/07/12 12:43, Ramana Radhakrishnan wrote: >> Patch 1 fixes up the vaba and vabal patterns to use a canonical RTL >> form with the first operand to the plus being the more complex one. > > This patch canonicalizes the instruction patterns for the > vaba and vabal intrinsics so that the more co

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Richard Henderson
On 2012-08-03 01:51, Uros Bizjak wrote: > The same reasoning goes for dynamic negation: for neg %eax,%eax value > 0x8000 stays the same, but we have changed (x)sub to an (x)add in > the code stream. So? Did you think the xadd will trap? r~

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread Ian Lance Taylor
On Fri, Aug 3, 2012 at 7:01 AM, JonY wrote: > > For correctness sake, determine version from BASE-VER file instead of > calling compiler. This allows better toleration with bad Canadian cross > setups where host gcc version is of different version from target. > > Patch OK? The patch is OK but yo

Re: [PATCH][2/n] Allow anonymous SSA names, a bunch of new predicates for SSA names

2012-08-03 Thread Miles Bader
Michael Matz writes: > Bikeshedding: please settle on consistent naming, either IS, or HAS or _P. > (FWIW, I suggest _P). "IS" and "HAS" have different meanings... -miles -- People who are more than casually interested in computers should have at least some idea of what the underlying hardware

Reload/i386 patch for secondary memory vs subregs

2012-08-03 Thread Bernd Schmidt
There are a number of problems in the interaction between secondary memory and subregs. If we reload the inner of a subreg, we forget about this when deciding whether to use secondary memory, and just use REGNO_REG_CLASS on the (now reloaded) inner. Also, we don't really know what to do if we get

Re: [PATCH 0/2] Convert s390 to atomic optabs, v2

2012-08-03 Thread Ulrich Weigand
I wrote: > Just a quick heads-up that something still must be broken; > I get extra test suite failures: > > FAIL: gcc.dg/atomic-compare-exchange-1.c execution test > FAIL: gcc.dg/atomic-compare-exchange-2.c execution test > FAIL: gcc.dg/atomic-compare-exchange-3.c execution test > WARNING: progra

[Patch, fortran] PR fortran/54166 ICE on array section (4.8 regression)

2012-08-03 Thread Mikael Morin
Hello, here is the fix for the regression I have introduced with my assumed rank bounds patch. Will test and commit as obvious. Mikael 2012-08-02 Mikael Morin PR fortran/54166 * trans-array.c (set_loop_bounds): Access specinfo using spec_dim. 2012-08-02 Mikael Morin

Re: [PATCH] Intrinsics for ADCX

2012-08-03 Thread Michael Zolotukhin
Thanks! On 3 August 2012 17:51, Uros Bizjak wrote: > On Fri, Aug 3, 2012 at 3:24 PM, Michael Zolotukhin > wrote: >> Hi, >> I made a new version of the patch, where I tried to take into account >> Uros' remarks - is it ok for trunk? >> >> Bootstrap and new tests are passing, testing is in progres

[PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
Hi, For correctness sake, determine version from BASE-VER file instead of calling compiler. This allows better toleration with bad Canadian cross setups where host gcc version is of different version from target. Patch OK? Index: gcc-trunk/libgcc/Makefile.in =

Re: [PATCH] Missing change from gcc/ada/system.ads

2012-08-03 Thread Arnaud Charlet
> It is however used when building libada if target specific version is not > specified > in gcc/ada/gcc_interface/Makefile.in. Well, then you should submit changes for Makefile.in for your unsupported target. That being said, see below: > In that case building libada fails due to missing Support

Re: [PATCH] Intrinsics for ADCX

2012-08-03 Thread Uros Bizjak
On Fri, Aug 3, 2012 at 3:24 PM, Michael Zolotukhin wrote: > Hi, > I made a new version of the patch, where I tried to take into account > Uros' remarks - is it ok for trunk? > > Bootstrap and new tests are passing, testing is in progress. > > Changelog entry: > 2012-08-03 Michael Zolotukhin > >

Re: [PATCH] Missing change from gcc/ada/system.ads

2012-08-03 Thread Andris Pavenis
On 08/03/2012 09:32 AM, Arnaud Charlet wrote: Revision http://gcc.gnu.org/viewcvs?view=revision&revision=189772 added constant Support_Atomic_Primitives to the target specific system-*.ads files. The default system.ads was not updated. The included patch - adds the missing constant - removes con

Re: speedup stack var conflicts (PR54146)

2012-08-03 Thread Richard Guenther
On Fri, Aug 3, 2012 at 3:14 PM, Michael Matz wrote: > Hi, > > as Steven noted in the bug report one reason of the slowness are the > myriads of calls to bitmap_set_bit, resulting from the clique generation > at the start of basic blocks. Can be sped up by using bitmap_ior. He > implemented upper

Re: [PATCH] Intrinsics for ADCX

2012-08-03 Thread Michael Zolotukhin
Hi, I made a new version of the patch, where I tried to take into account Uros' remarks - is it ok for trunk? Bootstrap and new tests are passing, testing is in progress. Changelog entry: 2012-08-03 Michael Zolotukhin * common/config/i386/i386-common.c (OPTION_MASK_ISA_ADX_SET): New.

Re: [PATCH][2/n] Allow anonymous SSA names, a bunch of new predicates for SSA names

2012-08-03 Thread Michael Matz
Hi, On Fri, 3 Aug 2012, Richard Guenther wrote: > 2012-08-03 Richard Guenther > > * tree.h (SSA_NAME_IS_VIRTUAL_OPERAND): New predicate. > (SSA_NAME_HAS_PARM_OR_RESULT_DECL): Likewise. > (SSA_NAME_HAS_RESULT_DECL): Likewise. > (SSA_NAME_HAS_PARM_DECL): Likewise. >

speedup stack var conflicts (PR54146)

2012-08-03 Thread Michael Matz
Hi, as Steven noted in the bug report one reason of the slowness are the myriads of calls to bitmap_set_bit, resulting from the clique generation at the start of basic blocks. Can be sped up by using bitmap_ior. He implemented upper triangular form for the conflict bitmap, but I couldn't mea

[PATCH][2/n] Allow anonymous SSA names, a bunch of new predicates for SSA names

2012-08-03 Thread Richard Guenther
This adds a bunch of new SSA name predicates to remove explicit uses of SSA_NAME_VAR. Bootstrapped and tested on x86_64-unknown-linux-gnu. I'm leaving this for comments. This mainly removes this cases from explicit covering in further patches. Richard. 2012-08-03 Richard Guenther

Re: VRP wrapping MULT_EXPR

2012-08-03 Thread Marc Glisse
On Fri, 3 Aug 2012, Marc Glisse wrote: On Fri, 3 Aug 2012, Richard Guenther wrote: +/* Some quadruple precision helpers. */ +static int +quad_int_cmp (double_int l0, double_int h0, + double_int l1, double_int h1, bool uns) +{ + int c = double_int_cmp (h0, h1, uns); + if (c != 0)

Re: VRP wrapping MULT_EXPR

2012-08-03 Thread Marc Glisse
On Fri, 3 Aug 2012, Richard Guenther wrote: +/* Some quadruple precision helpers. */ +static int +quad_int_cmp (double_int l0, double_int h0, + double_int l1, double_int h1, bool uns) +{ + int c = double_int_cmp (h0, h1, uns); + if (c != 0) return c; + return double_int_ucmp (l0,

Re: [PATCH 0/2] Convert s390 to atomic optabs, v2

2012-08-03 Thread Ulrich Weigand
Richard Henderson wrote: > Please try this as a followup to the previous two patches. > That should clean up the mistake with the insv change. Just a quick heads-up that something still must be broken; I get extra test suite failures: FAIL: gcc.dg/atomic-compare-exchange-1.c execution test FAIL:

[PATCH][0/n] Allow anonymous SSA names

2012-08-03 Thread Richard Guenther
This patch series will allow gimple SSA passes to create new temporary SSA names without creating a new decl. That means that SSA_NAME_VAR for such SSA names can be NULL_TREE (so we have to deal with the fallout). To make debugging dumps still readable we can put an INDENTIFIER_NODE in the place

Re: [Test] Fix for PRPR53981

2012-08-03 Thread Anna Tikhonova
Ping 2012/7/27 Anna Tikhonova : > Kirill has already checked it in for me, thanks! > Btw, I have FSF copyright assignment. > > Is it possible to commit this patch to 4.6 branch? Since current ndk > is based on 4.6 > > 2012/7/25 Janis Johnson : >> On 07/25/2012 03:58 AM, Anna Tikhonova wrote: >>> T

Re: Value type of map need not be default copyable

2012-08-03 Thread Paolo Carlini
Hi, On 08/03/2012 06:17 AM, Ollie Wild wrote: Patch courtesy Richard Smith at Google: Fix bug in the implementation of std::map's operator[]. Construct an object of type value_type, rather than using std::make_pair, so as to allow mapped_type to have an *explicit* copy constructor. See [map.ac

Re: PING [PATCH] Fix PR libstdc++/54036, problem negating DFP NaNs

2012-08-03 Thread Paolo Carlini
Hi, On 08/03/2012 07:00 AM, Peter Bergner wrote: Fixed. Ok, once more with feeling! :) How about the updated patched below? ...and thank you for the gentle review. This is my first libstdc++ patch and I guess I was used to the normal gcc/testsuite/ tests that don't include copyright headers a

[PATCH][1/n] Allow anonymous SSA names

2012-08-03 Thread Richard Guenther
This first patch simplifies code that queries SSA names for their type, whether they are a register, and other minor stuff we have other means for. This is to avoid useless references to SSA_NAME_VAR which further patches in this series will allow to be NULL. Bootstrapped and tested on x86_64-un

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Uros Bizjak
On Fri, Aug 3, 2012 at 10:02 AM, Richard Guenther wrote: > On Fri, Aug 3, 2012 at 9:24 AM, Uros Bizjak wrote: >> On Fri, Aug 3, 2012 at 8:40 AM, Uros Bizjak wrote: >> >>> We can implement atomic_fetch_sub with atomic_fetch_add and inverted >>> operand 2. However, we have to prevent overflows wit

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Richard Guenther
On Fri, Aug 3, 2012 at 9:24 AM, Uros Bizjak wrote: > On Fri, Aug 3, 2012 at 8:40 AM, Uros Bizjak wrote: > >> We can implement atomic_fetch_sub with atomic_fetch_add and inverted >> operand 2. However, we have to prevent overflows with negation, so >> only const_int operand 2 is allowed in the exp

[patch] one more sched-vis fix

2012-08-03 Thread Steven Bosscher
Hello, I also need to handle SEQUENCE, of course, or slim dumping fails in targets with branch delay slots. But who knew SEQUENCE can also appear as a REG_NOTE value?! The cfgrtl.c fix is purely cosmetic and obvious. Bootstrapped&tested on powerpc64-unknown-linux-gnu. Committed as obvious. Ciao

Re: VRP wrapping MULT_EXPR

2012-08-03 Thread Richard Guenther
On Thu, Aug 2, 2012 at 11:49 PM, Marc Glisse wrote: > Hello, > > here is a patch handling multiplication of wrapping integer types in VRP. It > passed bootstrap+testsuite limited to c,c++ and without the testcase, so > I'll retest it better during the night. For some reason the test > libgomp.grap

Re: [PATCH, i386]: Implement atomic_fetch_sub

2012-08-03 Thread Uros Bizjak
On Fri, Aug 3, 2012 at 8:40 AM, Uros Bizjak wrote: > We can implement atomic_fetch_sub with atomic_fetch_add and inverted > operand 2. However, we have to prevent overflows with negation, so > only const_int operand 2 is allowed in the expander. > > 2012-08-02 Uros Bizjak > > PR target