Re: [ping] couple of fixes

2012-10-21 Thread Paolo Bonzini
Il 19/10/2012 19:01, Eric Botcazou ha scritto: > PR bootstrap/54820 (stage #1 bootstrap failure) > http://gcc.gnu.org/ml/gcc-patches/2012-10/msg01093.html This one is okay, thanks. Paolo

[lra] patch to fix a gfortran test failure

2012-10-21 Thread Vladimir Makarov
The following patch fixes a gfortran test failure. LRA tried to put a split code after sibling call. Sibling calls mean no return. Committed as rev. 192657. 2012-10-21 Vladimir Makarov * lra-constraints.c (inherit_in_ebb): Check sibling call. Index: lra-constraints.c =

[lra] Merged with trunk

2012-10-21 Thread Vladimir Makarov
LRA branch was merged with the trunk @ rev. 192652. Committed as rev. 192655. One change was necessary: 2012-10-21 Vladimir Makarov * gcse.c (calculate_bb_reg_pressure): Add parameter to ira_setup_eliminable_regset call.

[PATCH GCC]Fix test case failure reported in PR54989

2012-10-21 Thread Bin Cheng
Hi, The test case "gcc/testsuite/gcc.dg/hoist-register-pressure.c" is failed on x86_64-apple-darwin because it uses more registers than x86_64-linux. This can be fixed by simplifying the case using fewer registers. Tested on x86_64-apple-darwin/x86_64-linux, is it OK? Thanks gcc/testsuite/Chan

Re: [PATCH] A steadier steady_clock

2012-10-21 Thread Sam Varshavchik
Florian Weimer writes: * Sam Varshavchik: > Based on a casual browsing of clock_gettime(3), CLOCK_MONOTONIC_RAW > seems to be a better fit for std::chrono::steady_clock's requirements > as given in 20.11.7.2, with recent Linux kernels, Are the Linux clock semantics documented somewhere in det

Re: [PING] [PATCH, libstdc++-v3] Add newlib specific ctype_members.cc

2012-10-21 Thread Jonathan Wakely
On 19 October 2012 18:04, Yufeng Zhang wrote: > > Please find the attached patch which documents the 'newlib' option for the > configuration option '--enable-clocale'. > > Feel free to improve it. Thanks for the description. I changed it slightly as I realised the 'darwin' option was also undocum

Re: [PATCH] A steadier steady_clock

2012-10-21 Thread Ulrich Drepper
On Sun, Oct 21, 2012 at 12:11 PM, Paolo Carlini wrote: \>> @@ -70,7 +70,11 @@ >> { >> timespec tp; >> // -EINVAL, -EFAULT >> +#ifdef CLOCK_MONOTONIC_RAW >> + clock_gettime(CLOCK_MONOTONIC_RAW, &tp); >> +#else >> clock_gettime(CLOCK_MONOTONIC, &tp); >> +#endif >> re

[wwwdocs] gcc-3.3/gcj-status.html: omit two links to sources.redhat.com

2012-10-21 Thread Gerald Pfeifer
We've got plenty of other links, and this is mostly archeologic, but still shouldn't feature a broken link when that hostname goes away. Applied. Gerald Index: gcj-status.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-3.3/gcj-status

[Patch, Fortran] PR 54997: -Wunused-function gives false warnings for procedures passed as actual argument

2012-10-21 Thread Janus Weil
Hi all, here is another patch to silence some more of the bogus warnings about unused functions that gfortran is currently throwing (cf. also the previous patch for PR 54224). It fixes the usage of the 'referenced' attribute, which should only be given to procedures which are actually 'used' (cal

[PATCH, i386]: Fix length attribute calculation for LEA and addr32 addresses

2012-10-21 Thread Uros Bizjak
Hello! Attached patch fixes length attribute calculation for LEA insn and addr32 addresses: - REX prefix was computed in a wrong way for zero-extended LEA instructions. Addresses with SUBREG, ZERO_EXTEND and AND result in SImode target register without REX prefix. - addr32 handling was wrong for

Re: [patch] fix outdated path in cpp.texi

2012-10-21 Thread Jonathan Wakely
On 21 October 2012 20:34, Gerald Pfeifer wrote: > On Thu, 18 Oct 2012, Jonathan Wakely wrote: >> http://gcc.gnu.org/onlinedocs/cpp/Search-Path.html claims the search >> path for C++ headers starts with /usr/include/g++-v3 which hasn't been >> true for many years. >> >> 2012-10-18 Jonathan Wakely

Re: [patch] PR 54930: add -Wreturn-local-addr

2012-10-21 Thread Dodji Seketeli
The C++ bits look good to my casual non-maintainer eyes. Let's CC Jason. Thanks. Jonathan Wakely a écrit: > This adds a warning switch for the existing "returning address of > local variable" warnings in the C and C++ FEs which are enabled by > default but have no switch controlling them. Addi

Re: [PATCH] A steadier steady_clock

2012-10-21 Thread Florian Weimer
* Sam Varshavchik: > Based on a casual browsing of clock_gettime(3), CLOCK_MONOTONIC_RAW > seems to be a better fit for std::chrono::steady_clock's requirements > as given in 20.11.7.2, with recent Linux kernels, Are the Linux clock semantics documented somewhere in detail? > +#ifdef CLOCK_MONO

[MIPS] Implement static stack checking

2012-10-21 Thread Eric Botcazou
This implements static stack checking for MIPS, i.e. checking of the static part of the frame in the prologue when -fstack-check is specified. This is very similar to the PowerPC and SPARC implementations and makes it possible to pass the full ACATS testsuite with -fstack-check. Tested on mips

XFAIL a couple of gnat.dg testcases on MIPS

2012-10-21 Thread Eric Botcazou
They are reported as failing with the n32 ABI, but the failures are spurious. Tested on mips64el-linux-gnu, applied on the mainline and 4.7 branch. 2012-10-21 Eric Botcazou * gnat.dg/specs/atomic1.ads: XFAIL on MIPS. * gnat.dg/specs/addr1.ads: Likewise. -- Eric Botcazoudif

RFA: replace #ifdef with if/#if for HAVE_ATTR_*

2012-10-21 Thread Joern Rennecke
As discussed on the gcc mailing list ("RFD: HAVE_* pattern flags"), we would like to have fewer #ifdefs littered throughout our functions. And in order to get there, we first need flags with values that are suitable for the C / C++ parsers, i.e. values 1 or 0 instead of defined or not. This patch

Re: [patch] fix outdated path in cpp.texi

2012-10-21 Thread Gerald Pfeifer
On Thu, 18 Oct 2012, Jonathan Wakely wrote: > http://gcc.gnu.org/onlinedocs/cpp/Search-Path.html claims the search > path for C++ headers starts with /usr/include/g++-v3 which hasn't been > true for many years. > > 2012-10-18 Jonathan Wakely > > * doc/cpp.texi (Search Path): Fix outdat

Re: unordered set design modification

2012-10-21 Thread François Dumont
On 10/21/2012 11:47 AM, Paolo Carlini wrote: On 10/20/2012 11:07 PM, François Dumont wrote: I have also put a fix in profile/map.h even if it is unrelated to this evolution. Should I commit it outside this patch ? Yes please! Thanks, Paolo. Attached patch applied. 2012-10-21 François

Re: [PATCH] A steadier steady_clock

2012-10-21 Thread Paolo Carlini
On 10/21/2012 04:00 PM, Sam Varshavchik wrote: Based on a casual browsing of clock_gettime(3), CLOCK_MONOTONIC_RAW seems to be a better fit for std::chrono::steady_clock's requirements as given in 20.11.7.2, with recent Linux kernels, Something like this: Please always CC library patches to l

committed: Fix ADDR_DIFF_VEC comment typo.

2012-10-21 Thread Joern Rennecke
Committed as obvious. 2012-10-21 Joern Rennecke * rtl.def (ADDR_DIFF_VEC): Fix comment typo. Index: rtl.def === --- rtl.def (revision 192653) +++ rtl.def (working copy) @@ -241,7 +241,7 @@ DEF_RTL_EXPR(ADDR_VEC, "

[wwwdocs, Java] Remove obsolete FAQ entry re old GNU/Linux versions

2012-10-21 Thread Gerald Pfeifer
While looking for something else I found this (and in general it looks like the GCJ FAQ could use some spring cleaning). Lots of "might be possible" in addition to this being really about quite older versions of GNU/Linux. Applied. Gerald 2012-10-21 Gerald Pfeifer * faq.html (My li

[PATCH] A steadier steady_clock

2012-10-21 Thread Sam Varshavchik
Based on a casual browsing of clock_gettime(3), CLOCK_MONOTONIC_RAW seems to be a better fit for std::chrono::steady_clock's requirements as given in 20.11.7.2, with recent Linux kernels, Something like this: Index: libstdc++-v3/src/c++11/chrono.cc ===

Re: Committed: fix gcc.dg/webizer.c (the trivial part :)

2012-10-21 Thread Hans-Peter Nilsson
On Sun, 21 Oct 2012, Andreas Schwab wrote: > Hans-Peter Nilsson writes: > > > I used __builtin_exit just to avoid declarations or includes; > > A "return 0" would have worked just as well. It's not the preferred method. Fail: abort(). Pass: exit(0). Don't remember where I read it, though. Brgd

Re: Committed: fix gcc.dg/webizer.c (the trivial part :)

2012-10-21 Thread Andreas Schwab
Hans-Peter Nilsson writes: > I used __builtin_exit just to avoid declarations or includes; A "return 0" would have worked just as well. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely

Committed: fix gcc.dg/webizer.c (the trivial part :)

2012-10-21 Thread Hans-Peter Nilsson
Committed as obvious. Without this, mmix-knuth-mmixware exited with 2256, likely a return-address left over in the return-value-register. I used __builtin_exit just to avoid declarations or includes; the default implicit declaration apparently incompatible with this use. I saw nothing in the relat

Follow-up to PR rtl-optimization/44194

2012-10-21 Thread Eric Botcazou
As remarked by Richard S., we can now allow sibling calls in the PARALLEL case after the fix for PR rtl-optimization/44194. Tested on x86_64-suse-linux and mips64el-linux-gnu, applied on the mainline. 2012-10-21 Eric Botcazou PR rtl-optimization/44194 * calls.c (expand_call)

[wwwdocs] testing/index.h: sources.redhat.com -> sourceware.org

2012-10-21 Thread Gerald Pfeifer
Similar to the patch last night, replacing usage of sources.redhat.com. Installed. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/testing/index.html,v retrieving revision 1.27 diff -u -3 -p -r1.27 index.html --- inde

[patch, wwwdocs, committed] Fortran changes for 4.8

2012-10-21 Thread Thomas Koenig
Hello world, I have committed the attached patch as obvious after checking with "tidy". Best regards Thomas 2012-10-21 Thomas Koenig * gcc-4.8/changes.html: Document that -Wc-binding-type is enabled by -Wall. Document that -Wcompare-reals is enabled by -Wex

Re: *ping* [patch, fortran] Handle -Wextra, -fcompare-reals is implied with -Wextra

2012-10-21 Thread Thomas Koenig
Hi Janus, In the docu of -Wall, there is a list of which switches are included, but -Wc-binding-type is missing. Maybe you can add it? And how about also adding a short docu paragraph for Wextra? I've added both items to the documentation as you suggested, and committed as rev. 192649. Here

Re: [Patch, Build+Fortran] PR54725 - correctly set TARGET_SYSTEM_ROOT for CPP

2012-10-21 Thread Paul Richard Thomas
Dear Tobias, I had already reviewed this patch in its previous accidental manifestation :-) OK for trunk Thanks for the patch Paul On 19 October 2012 10:22, Tobias Burnus wrote: > gfortran was ignoring the TARGET_SYSTEM_ROOT and thus searched in > /usr/include for files specified with "#inclu

Re: [lra] patch to fix testsuite regressions

2012-10-21 Thread Richard Sandiford
Hi Vlad, Vladimir Makarov writes: > @@ -543,23 +544,34 @@ extract_loc_address_regs (bool top_p, en > code1 = GET_CODE (arg1); > } > > + if (CONSTANT_P (arg0) > + || code1 == PLUS || code1 == MULT || code1 == ASHIFT) > + { > + tloc = arg1_loc; > +