Re: [google-4_6] fix broken -S in streaming LIPO (issue6115052)

2012-04-25 Thread Xinliang David Li
ok. David On Wed, Apr 25, 2012 at 1:08 PM, Rong Xu wrote: > Hi, > > This patch is for google-4_6 branch only. > It fixes the broken -S in streaming LIPO mode. > > Tested with bootstrap. > > Thanks, > > 2012-04-25   Rong Xu   > >        * gcc/gcc.c (ripa_lto_spec): Support -S. >        (cc1_optio

Re: [RS6000] Fix PR53038, cfa_restore out of order

2012-04-25 Thread Richard Henderson
On 04/20/12 18:58, Alan Modra wrote: > + add_reg_note (insn, REG_FRAME_RELATED_EXPR, set); > + RTX_FRAME_RELATED_P (insn) = 1; You're probably better off using REG_CFA_REGISTER here, rather than making dwarf2cfi.c guess what you meant. The guessing is rather tuned to prologues... r~

[Patch, fortran] PR 52428 Reading of large negative values and range checking

2012-04-25 Thread Janne Blomqvist
Hi, currently when -frange-check is enabled, we check for overflow when doing a formatted read of an integer value. This check, however, is against the Fortran numerical model (see 13.4 in F2008), which defines a symmetric interval [-huge(), huge()], whereas all targets gfortran supports use a two

Re: [Patch, libfortran] Fix handling of temporary files

2012-04-25 Thread Janne Blomqvist
PING! On Thu, Apr 19, 2012 at 01:18, Janne Blomqvist wrote: > Hi, > > the attached patch implements some fixes for handling STATUS="SCRATCH" files. > > - Currently, we check GFORTRAN_TMPDIR, TMP, and TEMP environment > variables, but not the POSIX and GNU standard TMPDIR. As specifying a > tempor

Re: [Patch, fortran] PR 49010/24518 MOD/MODULO fixes, take 2

2012-04-25 Thread Janne Blomqvist
PING! On Thu, Apr 19, 2012 at 00:46, Janne Blomqvist wrote: > Hi, > > the attached patch implements a few fixes and cleanups for the MOD and > MODULO intrinsics. > > - When the arguments are constant, use mpfr_fmod instead of the naive > algorithms which are numerically unstable for large argumen

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-25 Thread Sriraman Tallam
On Tue, Apr 24, 2012 at 7:39 PM, H.J. Lu wrote: > On Tue, Apr 24, 2012 at 7:06 PM, Sriraman Tallam wrote: >> On Tue, Apr 24, 2012 at 5:24 PM, H.J. Lu wrote: >>> On Tue, Apr 24, 2012 at 5:10 PM, Sriraman Tallam >>> wrote: Hi,   Thanks for all the comments. I have made all the cha

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-25 Thread H.J. Lu
On Wed, Apr 25, 2012 at 2:25 PM, Sriraman Tallam wrote: > On Tue, Apr 24, 2012 at 7:39 PM, H.J. Lu wrote: >> On Tue, Apr 24, 2012 at 7:06 PM, Sriraman Tallam wrote: >>> On Tue, Apr 24, 2012 at 5:24 PM, H.J. Lu wrote: On Tue, Apr 24, 2012 at 5:10 PM, Sriraman Tallam wrote: > Hi,

Re: fix incorrect SRA transformation on non-integral VIEW_CONVERT argument

2012-04-25 Thread Olivier Hainque
Thanks for your feedback Richard, On Apr 25, 2012, at 16:16 , Richard Guenther wrote: > I think much better would be to simply disallow any toplevel > VIEW_CONVERT_EXPR of BLKmode, > Does that fix your problems, too? If so I prefer that. Hmm, I think that this would fix the particular testscas

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-25 Thread Sriraman Tallam
On Wed, Apr 25, 2012 at 2:28 PM, H.J. Lu wrote: > On Wed, Apr 25, 2012 at 2:25 PM, Sriraman Tallam wrote: >> On Tue, Apr 24, 2012 at 7:39 PM, H.J. Lu wrote: >>> On Tue, Apr 24, 2012 at 7:06 PM, Sriraman Tallam >>> wrote: On Tue, Apr 24, 2012 at 5:24 PM, H.J. Lu wrote: > On Tue, Apr 2

Re: [PATCH] Fix for PR51879 - Missed tail merging with non-const/pure calls

2012-04-25 Thread Tom de Vries
On 25/04/12 11:57, Richard Guenther wrote: >>> >> Hmm. I'm not sure we can conclude that they have the same value! >>> >> >>> >> +int bar (int); >>> >> +void baz (int); >>> >> +void bla (int); >>> >> + >>> >> +void >>> >> +foo (int y) >>> >> +{ >>> >> + int a; >>> >> + if (y == 6) >>> >> +{

Re: [v3] libstdc++/52689 testcase

2012-04-25 Thread Benjamin Kosnik
On Wed, 25 Apr 2012 18:53:36 +0100 Jonathan Wakely wrote: > On 25 April 2012 08:33, Igor Zamyatin wrote: > > This testcase is reported as failed on x86 > > Yep. > > http://gcc.gnu.org/ml/gcc-testresults/2012-04/msg02547.html > > Benjamin? Ouch. Sorry about this guys, of course it wasn't thi

Re: [PR tree-optimization/52558]: RFC: questions on store data race

2012-04-25 Thread Aldy Hernandez
On 04/25/12 06:45, Richard Guenther wrote: On Tue, Apr 24, 2012 at 7:43 PM, Aldy Hernandez wrote: On 04/13/12 03:46, Richard Guenther wrote: On Fri, Apr 13, 2012 at 12:11 AM, Aldy Hernandezwrote: + /* ?? Perhaps we should cache this somewhere in the BB, or are + multiple levels of

Re: [PATCH 12/13] Adjust relevant test cases wrt -ftrack-macro-expansion=[0|2]

2012-04-25 Thread Benjamin Kosnik
Hey hey! Looking good Dodji. Can't wait to use this stuff. > There are so many libstdc++ tests that were failing because of that > benign issue that I preferred to just run them with > -ftrack-macro-expansion diabled, after inspecting each of them to be > sure there was nothing more serious under

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-25 Thread Sriraman Tallam
Hi H.J, Could you please review this patch for AVX2 check? * config/i386/i386-cpuinfo.c (FEATURE_AVX2): New enum value. (get_available_features): New argument. Check for AVX2. (__cpu_indicator_init): Modify call to get_available_features . * doc/extend.texi: Doc

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-25 Thread H.J. Lu
On Wed, Apr 25, 2012 at 4:38 PM, Sriraman Tallam wrote: > Hi H.J, > >   Could you please review this patch for AVX2 check? > >        * config/i386/i386-cpuinfo.c (FEATURE_AVX2): New enum value. >        (get_available_features): New argument. Check for AVX2. >        (__cpu_indicator_init): Modif

Re: [RFA] dwarf2.h (dwarf_location_atom): Add DW_OP_GNU_addr_index.

2012-04-25 Thread Doug Evans
On Wed, Apr 18, 2012 at 12:11 PM, Doug Evans wrote: > Hi. > > This patch adds DW_OP_GNU_addr_index to dwarf2.h. > > Ok to commit? > > 2012-04-18  Doug Evans   > >        * dwarf2.h (dwarf_location_atom): Add DW_OP_GNU_addr_index. > > Index: include/dwarf2.h > ==

Re: -finstrument-functions leads to unsats to _mm instrinsic wrappers

2012-04-25 Thread Xinliang David Li
Ping .. David On Tue, Dec 27, 2011 at 10:46 AM, Xinliang David Li wrote: > Ping .. > > On Fri, Dec 9, 2011 at 9:42 AM, Xinliang David Li wrote: >> Ping .. >> >> On Wed, Dec 7, 2011 at 4:01 PM, Xinliang David Li wrote: >>> Build the test case in the patch file with -finstrument-functions, the

[patch] fix test gcc.dg/bf-ms-layout.c

2012-04-25 Thread Janis Johnson
A couple of months ago Kai Tietz fixed several tests for MS layout of bitfields, including: 2012-02-23 Kai Tietz * gcc.dg/bf-ms-layout-2.c: Adjust offsets to fit ms-bitfield structure-layout. He missed changes to gcc.dg/bf-ms-layout.c which has the same layouts as bf-ms-la

Committed: fix PR target/53120, constraint modifier "+" on operand tied by matching-constraint, "0".

2012-04-25 Thread Hans-Peter Nilsson
The subject line points out the bug. This combination of constraints is invalid, and almost no other target has it (and none together with strict_low_part): you can't reload a read/write operand (one with a "+" modifier, such as a strict_low_part destination) for which there is an input-only match

Re: [RFA] dwarf2.h (dwarf_location_atom): Add DW_OP_GNU_addr_index.

2012-04-25 Thread Cary Coutant
> Ping. > This is approved on the binutils side. > I just need approval on the gcc side. My understanding is that a patch to sync binutils/include and gcc/include falls under the "trivial and obvious" rule. True? -cary

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-25 Thread Sriraman Tallam
Patch committed. Thanks, -Sri. On Wed, Apr 25, 2012 at 4:52 PM, H.J. Lu wrote: > On Wed, Apr 25, 2012 at 4:38 PM, Sriraman Tallam wrote: >> Hi H.J, >> >>   Could you please review this patch for AVX2 check? >> >>        * config/i386/i386-cpuinfo.c (FEATURE_AVX2): New enum value. >>        (get

[v3] hashtable doc tweaks

2012-04-25 Thread Benjamin De Kosnik
... putting in line breaks for doxygen, so that each base class is on a separate line. -benjamin2012-04-25 Benjamin Kosnik * include/bits/hashtable.h: Adjust doxygen markup for base classes. * include/bits/hashtable_policy.h: Same. diff --git a/libstdc++-v3/include/bits/hashtable.h b/li

libgo patch committed: Always define syscall.SO_REUSEPORT

2012-04-25 Thread Ian Lance Taylor
This patch to libgo ensures that syscall.SO_REUSEPORT is defined. It fixes the only use of it in the standard library to check whether it has a valid value before using it. This should fix the build on Solaris 9. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-25 Thread Ollie Wild
On Mon, Apr 23, 2012 at 8:28 AM, Gabriel Dos Reis wrote: > > Thanks.  Tom, I am satisfied with the diagnostic part. > I think the CPP part makes sense but it is your call. Hi, Tom, just a quick checkin to see if you've had a chance to review this, yet. Thanks, Ollie

Re: [RS6000] Fix PR53038, cfa_restore out of order

2012-04-25 Thread Alan Modra
On Wed, Apr 25, 2012 at 02:00:48PM -0700, Richard Henderson wrote: > On 04/20/12 18:58, Alan Modra wrote: > > + add_reg_note (insn, REG_FRAME_RELATED_EXPR, set); > > + RTX_FRAME_RELATED_P (insn) = 1; > > You're probably better off using REG_CFA_REGISTER here, > rather than making dwarf2c

Re: [RFA] dwarf2.h (dwarf_location_atom): Add DW_OP_GNU_addr_index.

2012-04-25 Thread Jakub Jelinek
On Wed, Apr 25, 2012 at 04:56:17PM -0700, Doug Evans wrote: > On Wed, Apr 18, 2012 at 12:11 PM, Doug Evans wrote: > > This patch adds DW_OP_GNU_addr_index to dwarf2.h. > > > > Ok to commit? > > > > 2012-04-18  Doug Evans   > > > >        * dwarf2.h (dwarf_location_atom): Add DW_OP_GNU_addr_index.

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-04-25 Thread Marc Glisse
On Wed, 25 Apr 2012, H.J. Lu wrote: You can download an AVX2 emulator (SDE) from http://software.intel.com/en-us/avx/ to test AVX2 binaries. Hello, not related to this specific patch, but do you have a dejagnu board description that you could share that enables running the compiler normal

<    1   2