Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)

2011-12-13 Thread tmsriram
On 2011/12/14 00:04:10, davidxl wrote: http://codereview.appspot.com/5488054/diff/4002/tree-vect-stmts.c File tree-vect-stmts.c (right): http://codereview.appspot.com/5488054/diff/4002/tree-vect-stmts.c#newcode3712 tree-vect-stmts.c:3712: } The check can be put into a helper function. Fixed

Re: [google] dump inline decisions to stderr under -fopt-info

2011-12-13 Thread Dehao Chen
I've updated the patch to fix a bug in dump_inline_decision. Thanks, Dehao On Thu, Dec 1, 2011 at 9:59 AM, Dehao Chen wrote: > > This patch is for google-{main|gcc_4.6} only. > > Tested with bootstrap and regression tests. > > Dump inline decisions, also output the inline chain. > > Dehao > > 20

Go patch committed: Move reading export data to gcc interface

2011-12-13 Thread Ian Lance Taylor
This patch moves reading export data from an object file out of the Go frontend proper into the gcc side of the interface. The purpose of this move is to let the gcc side entirely control the name of the Mach-O segment and section where the export data is stored. Bootstrapped and ran Go testsuite

[PATCH] MIPS16: Fix truncated DWARF-2 line information

2011-12-13 Thread Maciej W. Rozycki
Hi, I've noticed in the presence of a specific MIPS16 function thunk, GCC fails to emit suitable DWARF-2 location directives, which in turn causes DWARF-2 line records to provide truncated information. This is probably best illustrated with an example. Given the following source code: $ ca

C++ PATCH for c++/51406, 51161 (wrong-code with static cast to rvalue ref)

2011-12-13 Thread Jason Merrill
The code for casting to rvalue ref was assuming that no base adjustment would be necessary. This patch delegates to the normal lvalue binding code, and then changes the result to be an rvalue reference. Tested x86_64-pc-linux-gnu, applying to trunk. Will apply to 4.5/4.6 as well after testin

Re: Fix flags for edges from/to entry/exit basic blocks (issue5486043)

2011-12-13 Thread Hans-Peter Nilsson
> From: Dmitry Vyukov > Date: Tue, 13 Dec 2011 11:43:18 +0100 > On Tue, Dec 13, 2011 at 3:04 AM, Hans-Peter Nilsson > wrote: > > Details in PR51521. > Sorry for the trouble. What should I do now? Do what Diego and others asked, roll back your patch immediately; I see it hasn't happened yet. No

Re: [Patch,AVR] ad PR50931: Implement mulpsi3

2011-12-13 Thread Denis Chertykov
2011/12/13 Georg-Johann Lay : > This adds missing 24-Bit multiplication. > > Besides implementing __mulpsi3 in libgcc the patch adds some minor tweaks to > multiply with small numbers and to represent the reduced register footprint of > asm implementations. > > With this patch PR50931 is complete f

Re: [google] dump inline decisions to stderr under -fopt-info

2011-12-13 Thread Dehao Chen
Sorry, forgot to attach the patch... Dehao On Wed, Dec 14, 2011 at 9:13 AM, Dehao Chen wrote: > I've updated the patch to fix a bug in dump_inline_decision. > > Thanks, > Dehao > > On Thu, Dec 1, 2011 at 9:59 AM, Dehao Chen wrote: >> >> This patch is for google-{main|gcc_4.6} only. >> >> Tested

Re: [google] dump inline decisions to stderr under -fopt-info

2011-12-13 Thread Xinliang David Li
There are a couple of problems with the patch (the patch is only compatible with 4_6 branch) 1) the dump_inline_decision should be called inside cgraph_mark_inline_edge when the edge is finally picked (and when the callee node is cloned) 2) The source location is not printed which makes the dumpin

<    1   2