Re: [google][4.6]Make option -freorder-functions= invoke function reordering linker plugin (issue 5825054)

2012-03-18 Thread davidxl
Ok for google branches after updating the doc/invoke.texi file. David http://codereview.appspot.com/5825054/

Re: remove wrong code in immed_double_const

2012-03-18 Thread Richard Sandiford
Mike Stump mikest...@comcast.net writes: We currently only support constant integer widths = 2*HOST_BITS_PER_WIDE_INT, and the assert is correctly triggering if we try to build a wider constant. Can you give me a concrete example of what will fail with the constant 0 generated by:

Re: Remove anachronistic docs about G++ template instantiation

2012-03-18 Thread Gerald Pfeifer
On Thu, 8 Mar 2012, Jonathan Wakely wrote: The manual claims a future version of G++ will support a hybrid instantiation model, which I don't think is still planned, and describes extern templates as an extension when they are in C++11. * doc/extend.texi (Template Instantiation):

Re: [C++ Patch] PR 14710 (add -Wuseless-cast)

2012-03-18 Thread Paolo Carlini
Hi, That won't catch something like int i; static_castint(i); which is also a useless cast, because i is already an int lvalue; not all lvalues are derived from references. Note that something like static_castint(42); is not a useless cast, because it turns a prvalue into an xvalue.

Re: PATCH: Properly generate X32 IE sequence

2012-03-18 Thread Uros Bizjak
On Sat, Mar 17, 2012 at 10:50 PM, H.J. Lu hjl.to...@gmail.com wrote: Since we must use reg64 in %fs:(%reg) memory operand like movq x@gottpoff(%rip),%reg64; mov %fs:(%reg64),%reg this patch optimizes x32 TLS IE load and store by wrapping %reg64 inside of UNSPEC when Pmode == SImode.  OK

Re: remove wrong code in immed_double_const

2012-03-18 Thread Mike Stump
On Mar 18, 2012, at 3:16 AM, Richard Sandiford wrote: Mike Stump mikest...@comcast.net writes: We currently only support constant integer widths = 2*HOST_BITS_PER_WIDE_INT, and the assert is correctly triggering if we try to build a wider constant. Can you give me a concrete example of what

Re: [wwwdocs] Buildstat update for 4.6

2012-03-18 Thread Gerald Pfeifer
On Thu, 15 Mar 2012, Tom G. Christensen wrote: Latest results for 4.6.x -tgc Testresults for 4.6.2: hppa2.0w-hp-hpux11.11 i386-pc-solaris2.8 i686-pc-linux-gnu powerpc-apple-darwin8.11.0 sparc-sun-solaris2.8 (2) x86_64-apple-darwin10.8.0 x86_64-apple-darwin11.3.0

[Patch, fortran] PR41600 - [OOP] SELECT TYPE with associate-name = exp: Arrays not supported

2012-03-18 Thread Paul Richard Thomas
Dear All, Please find attached a fix for PR41600 plus some. It is reasonably straightforward but the following should be noted: (i) gfc_get_vptr_from_expr exploits that seeming fact that tracing back any class expression through TREE_OPERAND 0 eventually gets one back to the class expression. I

Re: PATCH: Properly generate X32 IE sequence

2012-03-18 Thread Uros Bizjak
On Sun, Mar 18, 2012 at 5:01 PM, Uros Bizjak ubiz...@gmail.com wrote: I am testing this patch.  OK for trunk if it passes all tests? No, force_reg will generate a pseudo, so this conversion is valid only for !can_create_pseudo (). At least for *tls_initial_exec_x32_store, you will need a

Re: [patch, committed] invoke.texi: DWARF, stabs, and ELF

2012-03-18 Thread Gerald Pfeifer
On Fri, 9 Mar 2012, Sandra Loosemore wrote: Per the DWARF web site http://dwarfstd.org/Download.php the correct names of the various versions of the DWARF standard appear to be either DWARF Version N or DWARF N, rather than e.g. DWARF2, DWARF-2, dwarf2, or whatever. This patch to

Re: [PATCH] Straight line strength reduction, part 1

2012-03-18 Thread Andrew Pinski
On Sun, Mar 18, 2012 at 6:12 PM, William J. Schmidt wschm...@linux.vnet.ibm.com wrote: Greetings, Now that we're into stage 1 again, I'd like to submit the first round of changes for dominator-based strength reduction, which will address issues from PR22586, PR35308, PR46556, and perhaps

Re: [google][4.6]Make option -freorder-functions= invoke function reordering linker plugin (issue 5825054)

2012-03-18 Thread Sriraman Tallam
Updated doc/invoke.texi and submitted. Thanks, -Sri. On Sat, Mar 17, 2012 at 11:58 PM, davi...@google.com wrote: Ok for google branches after updating the doc/invoke.texi file. David http://codereview.appspot.com/5825054/