Re: RFC: extend cprop_hardreg into a global pass

2012-07-25 Thread Steven Bosscher
On Wed, Jul 25, 2012 at 3:35 AM, Bin.Cheng amker.ch...@gmail.com wrote: On Wed, Jul 25, 2012 at 2:14 AM, Steven Bosscher stevenb@gmail.com wrote: Bin Cheng wrote: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44025 You could foster-parent and fix the attached patch to address this issue.

Integer promotion for register based arguments

2012-07-25 Thread Jon Beniston
Hi, I've tried compiling the following program targeting both MIPS, LM32 and ARM. long a, b; void func(short p) { b = (long)p; } int main() { if(a 2) func((short)a); return 0; } For MIPS and LM32, truncation is performed in the calling function and

Re: Integer promotion for register based arguments

2012-07-25 Thread Andrew Haley
On 07/25/2012 12:15 PM, Jon Beniston wrote: For MIPS and LM32, truncation is performed in the calling function and sign extension in the called function. One of these operations seems redundant. For ARM, truncation is performed in the caller, but sign-extension isn't performed in the callee,

RE: Integer promotion for register based arguments

2012-07-25 Thread Jon Beniston
Hi Andrew, On 07/25/2012 12:15 PM, Jon Beniston wrote: For MIPS and LM32, truncation is performed in the calling function and sign extension in the called function. One of these operations seems redundant. For ARM, truncation is performed in the caller, but sign-extension isn't

Problems with pragma and attribute optimize.

2012-07-25 Thread Allan Sandfeld Jensen
Hi, I have been experimenting with marking specific functions to be auto- vectorized in GCC, but have had problems getting it to work. It seems the optimize attribute works sometimes, but only if the function it is used on is not static, but pragma optimize never seems to work. See the

Re: Problems with pragma and attribute optimize.

2012-07-25 Thread Richard Guenther
On Wed, Jul 25, 2012 at 2:23 PM, Allan Sandfeld Jensen carew...@gmail.com wrote: Hi, I have been experimenting with marking specific functions to be auto- vectorized in GCC, but have had problems getting it to work. It seems the optimize attribute works sometimes, but only if the function it

Optimize attribute and inlining

2012-07-25 Thread Selvaraj, Senthil_Kumar
Declaring a function with __attribute__((optimize(O0)) turns off inlining for the translation unit (atleast) containing the function (see output at the end). Is this expected behavior? I tracked this down to the fact that when processing the optimize attribute with O0, flag_no_inline is set to

Re: Problems with pragma and attribute optimize.

2012-07-25 Thread Allan Sandfeld Jensen
On Wednesday 25 July 2012, Richard Guenther wrote: On Wed, Jul 25, 2012 at 2:23 PM, Allan Sandfeld Jensen carew...@gmail.com wrote: Hi, I have been experimenting with marking specific functions to be auto- vectorized in GCC, but have had problems getting it to work. It seems the

Re: Optimize attribute and inlining

2012-07-25 Thread Richard Guenther
On Wed, Jul 25, 2012 at 4:07 PM, Selvaraj, Senthil_Kumar senthil_kumar.selva...@atmel.com wrote: Declaring a function with __attribute__((optimize(O0)) turns off inlining for the translation unit (atleast) containing the function (see output at the end). Is this expected behavior? Not

Re: Problems with pragma and attribute optimize.

2012-07-25 Thread Richard Guenther
On Wed, Jul 25, 2012 at 4:25 PM, Allan Sandfeld Jensen carew...@gmail.com wrote: On Wednesday 25 July 2012, Richard Guenther wrote: On Wed, Jul 25, 2012 at 2:23 PM, Allan Sandfeld Jensen carew...@gmail.com wrote: Hi, I have been experimenting with marking specific functions to be auto-

Re: Integer promotion for register based arguments

2012-07-25 Thread Eric Botcazou
I guess my question is what would I need to change to make it work like the ARM port? I can't see how this is being controlled. Try TARGET_PROMOTE_PROTOTYPES. -- Eric Botcazou

RE: Integer promotion for register based arguments

2012-07-25 Thread Jon Beniston
Hi Eric, I guess my question is what would I need to change to make it work like the ARM port? I can't see how this is being controlled. Try TARGET_PROMOTE_PROTOTYPES. For all 3 targets I believe this returns true (Both MIPS and LM32 use hook_bool_const_tree_true), so I presume it must

Re: Integer promotion for register based arguments

2012-07-25 Thread Andrew Haley
On 07/25/2012 04:52 PM, Jon Beniston wrote: Hi Eric, I guess my question is what would I need to change to make it work like the ARM port? I can't see how this is being controlled. Try TARGET_PROMOTE_PROTOTYPES. For all 3 targets I believe this returns true (Both MIPS and LM32 use

RE: Identifying Compiler Options to Minimize Energy Consumption by Embedded Programs

2012-07-25 Thread Jon Beniston
Hi James, - Which set of benchmarks are suitable for embedded applications and representative of possible applications? Have a look at CoreMark: http://www.coremark.org/ EEMBC also have EnergyBench: http://www.eembc.org/benchmark/power_sl.php although I think that might be commercial, but it

Re: Optimize attribute and inlining

2012-07-25 Thread David Brown
On 25/07/12 17:30, Richard Guenther wrote: On Wed, Jul 25, 2012 at 4:07 PM, Selvaraj, Senthil_Kumar senthil_kumar.selva...@atmel.com wrote: Declaring a function with __attribute__((optimize(O0)) turns off inlining for the translation unit (atleast) containing the function (see output at the

Re: Reserving a bit in ELF segment flags for huge page mappings

2012-07-25 Thread Sriraman Tallam
On Tue, Jul 24, 2012 at 1:40 PM, Cary Coutant ccout...@google.com wrote: To do this, I would like to reserve a bit in the segment flags to indicate that this segment is to be mapped to huge pages if possible. Can I reserve something like a PF_LARGE_PAGE bit? HP-UX has a PF_HP_PAGE_SIZE

[Bug gcov-profile/47618] Collecting multiple profiles and using all for PGO

2012-07-25 Thread steven at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47618 --- Comment #12 from Steven Bosscher steven at gcc dot gnu.org 2012-07-25 08:24:49 UTC --- (In reply to comment #9) I think a tool to merge would be a good partial solution. We will go with the tool solution. I'll take care of the tool before

[Bug c++/53839] [4.7/4.8 Regression] [C++11] internal compiler error: in adjust_temp_type, at cp/semantics.c:6391

2012-07-25 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53839 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added Status|UNCONFIRMED |NEW

[Bug c++/54020] [c++0x] incorrectly accepted constexpr functions

2012-07-25 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54020 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added CC||jason at

[Bug c++/53654] move constructor incorrectly delete copy constructor defined by template

2012-07-25 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53654 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[Bug c++/53786] [C++11] alias template causes g++ segfault

2012-07-25 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53786 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added Status|UNCONFIRMED |NEW

[Bug libstdc++/54075] [4.7.1] unordered_map 3x slower than 4.6.2

2012-07-25 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54075 --- Comment #10 from Paolo Carlini paolo.carlini at oracle dot com 2012-07-25 09:56:15 UTC --- A patch is available here: http://gcc.gnu.org/ml/libstdc++/2012-07/msg00051.html Submitter and interested people can give it a try before it goes

[Bug c++/54090] New: internal compiler error: in unify, at cp/pt.c:15731

2012-07-25 Thread dicomj23 at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54090 Bug #: 54090 Summary: internal compiler error: in unify, at cp/pt.c:15731 Classification: Unclassified Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal

[Bug c++/54091] New: internal compiler error in class method with many string objects

2012-07-25 Thread bugzilla-gcc at thewrittenword dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54091 Bug #: 54091 Summary: internal compiler error in class method with many string objects Classification: Unclassified Product: gcc Version: 4.4.6 Status: UNCONFIRMED

[Bug c++/54091] internal compiler error in class method with many string objects

2012-07-25 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54091 --- Comment #1 from Jonathan Wakely redi at gcc dot gnu.org 2012-07-25 11:35:00 UTC --- Have you tried with a current release, 4.6 or 4.7? The 4.4 release series is closed and no longer maintained.

[Bug bootstrap/54092] New: [4.8 Regression] Bootstrap fails on x86_64-apple-darwin10 while building Ada at stage 1

2012-07-25 Thread dominiq at lps dot ens.fr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54092 Bug #: 54092 Summary: [4.8 Regression] Bootstrap fails on x86_64-apple-darwin10 while building Ada at stage 1 Classification: Unclassified Product: gcc Version: 4.8.0

[Bug c++/48026] #pragma optimize ignored for C++

2012-07-25 Thread linux at carewolf dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48026 Allan Jensen linux at carewolf dot com changed: What|Removed |Added CC||linux at carewolf

[Bug c++/54020] [c++0x] incorrectly accepted constexpr functions

2012-07-25 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54020 Jason Merrill jason at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last

[Bug bootstrap/54092] [4.8 Regression] Bootstrap fails while building Ada at stage 1

2012-07-25 Thread rth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54092 Richard Henderson rth at gcc dot gnu.org changed: What|Removed |Added Target|x86_64-apple-darwin10 |

[Bug c++/54020] [c++0x] incorrectly accepted constexpr functions

2012-07-25 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54020 --- Comment #5 from Jason Merrill jason at gcc dot gnu.org 2012-07-25 14:57:01 UTC --- Author: jason Date: Wed Jul 25 14:56:57 2012 New Revision: 189851 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189851 Log: PR c++/54020 *

[Bug c++/54086] GCC should allow constexpr and const together

2012-07-25 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54086 --- Comment #2 from Jason Merrill jason at gcc dot gnu.org 2012-07-25 14:57:11 UTC --- Author: jason Date: Wed Jul 25 14:57:06 2012 New Revision: 189852 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189852 Log: PR c++/54086 *

[Bug c++/54020] [c++0x] incorrectly accepted constexpr functions

2012-07-25 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54020 Jason Merrill jason at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug bootstrap/54092] [4.8 Regression] Bootstrap fails while building Ada at stage 1

2012-07-25 Thread rth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54092 --- Comment #2 from Richard Henderson rth at gcc dot gnu.org 2012-07-25 15:10:53 UTC --- Author: rth Date: Wed Jul 25 15:10:44 2012 New Revision: 189853 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189853 Log: PR bootstrap/54092

[Bug bootstrap/54092] [4.8 Regression] Bootstrap fails while building Ada at stage 1

2012-07-25 Thread rth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54092 Richard Henderson rth at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug c++/54086] GCC should allow constexpr and const together

2012-07-25 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54086 Jason Merrill jason at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[Bug target/54093] New: ICE in in extract_insn, at recog.c:2129

2012-07-25 Thread rmansfield at qnx dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54093 Bug #: 54093 Summary: ICE in in extract_insn, at recog.c:2129 Classification: Unclassified Product: gcc Version: 4.8.0 Status: UNCONFIRMED Severity: normal Priority:

[Bug tree-optimization/54094] New: [4.8 regression] ICE in graphite-dependences.c:320 : isl_constraint.c:497: position out of bounds

2012-07-25 Thread jojelino at gmail dot com
(GCC) version 4.8.0 20120725 (experimental) (i686-pc-cygwin) compiled by GNU C version 4.8.0 20120725 (experimental), GMP version 5.0.2, MPFR version 3.2.0-dev, MPC version 0.9 GGC heuristics: --param ggc-min-expand=30 --param ggc-min-heapsize=4096 ignoring duplicate directory /usr/include

[Bug regression/54084] Bunch of fails for x86

2012-07-25 Thread wschmidt at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54084 William J. Schmidt wschmidt at gcc dot gnu.org changed: What|Removed |Added CC||wschmidt

[Bug libstdc++/54036] Negating a DFP NAN in C++ produces NAN not -NAN

2012-07-25 Thread bergner at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54036 --- Comment #2 from Peter Bergner bergner at gcc dot gnu.org 2012-07-25 17:02:39 UTC --- Author: bergner Date: Wed Jul 25 17:02:27 2012 New Revision: 189857 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189857 Log: Backport prospective

[Bug lto/54095] New: Unnecessary static variable renaming

2012-07-25 Thread rth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54095 Bug #: 54095 Summary: Unnecessary static variable renaming Classification: Unclassified Product: gcc Version: unknown Status: UNCONFIRMED Severity: enhancement

[Bug lto/54095] Unnecessary static variable renaming

2012-07-25 Thread andi-gcc at firstfloor dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54095 Andi Kleen andi-gcc at firstfloor dot org changed: What|Removed |Added CC||andi-gcc at

[Bug target/53633] __attribute__((naked)) should disable -Wreturn-type

2012-07-25 Thread sandra at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53633 --- Comment #3 from sandra at gcc dot gnu.org 2012-07-25 18:08:11 UTC --- Author: sandra Date: Wed Jul 25 18:08:06 2012 New Revision: 189860 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189860 Log: 2012-07-25 Sandra Loosemore

[Bug tree-optimization/30318] VRP does not create ANTI_RANGEs on overflow

2012-07-25 Thread glisse at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30318 --- Comment #20 from Marc Glisse glisse at gcc dot gnu.org 2012-07-25 18:26:18 UTC --- Author: glisse Date: Wed Jul 25 18:26:12 2012 New Revision: 189861 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189861 Log: 2012-07-25 Marc Glisse

[Bug fortran/54096] New: Type bound procedures

2012-07-25 Thread badass at vt dot edu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54096 Bug #: 54096 Summary: Type bound procedures Classification: Unclassified Product: gcc Version: 4.5.3 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug c++/54090] internal compiler error: in unify, at cp/pt.c:15731

2012-07-25 Thread daniel.kruegler at googlemail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54090 Daniel Krügler daniel.kruegler at googlemail dot com changed: What|Removed |Added CC|

[Bug c++/54090] internal compiler error: in unify, at cp/pt.c:15731

2012-07-25 Thread ubizjak at gmail dot com
reconfirmed||2012-07-25 Ever Confirmed|0 |1 --- Comment #2 from Uros Bizjak ubizjak at gmail dot com 2012-07-25 18:56:57 UTC --- Confirmed on x86_64-unknown-linux-gnu, with: gcc version 4.8.0 20120725 (experimental) [trunk revision 189856] (GCC

[Bug fortran/54096] Type bound procedures

2012-07-25 Thread kargl at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54096 kargl at gcc dot gnu.org changed: What|Removed |Added CC||kargl at gcc dot gnu.org ---

[Bug target/54093] ICE in in extract_insn, at recog.c:2129

2012-07-25 Thread rmansfield at qnx dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54093 --- Comment #1 from Ryan Mansfield rmansfield at qnx dot com 2012-07-25 19:27:00 UTC --- Created attachment 27873 -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=27873 2nd preprocessed src example Happens at -O2

[Bug target/54093] ICE in in extract_insn, at recog.c:2129

2012-07-25 Thread rmansfield at qnx dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54093 --- Comment #2 from Ryan Mansfield rmansfield at qnx dot com 2012-07-25 19:28:40 UTC --- Created attachment 27874 -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=27874 3rd preprocessed src example Happens at -O1

[Bug libstdc++/54075] [4.7.1] unordered_map 3x slower than 4.6.2

2012-07-25 Thread fdumont at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54075 --- Comment #11 from François Dumont fdumont at gcc dot gnu.org 2012-07-25 19:32:53 UTC --- Author: fdumont Date: Wed Jul 25 19:32:48 2012 New Revision: 189863 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189863 Log: 2012-07-25 François

[Bug bootstrap/50461] mpfr.h found in mpfr-3.1.0/src instead of mpfr-3.0.1/. as previously

2012-07-25 Thread nightstrike at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50461 nightstrike nightstrike at gmail dot com changed: What|Removed |Added CC||nightstrike at

[Bug target/53110] GCC-4.7 generates stupid x86_64 asm

2012-07-25 Thread tejohnson at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53110 --- Comment #13 from tejohnson at gcc dot gnu.org 2012-07-25 20:11:23 UTC --- Author: tejohnson Date: Wed Jul 25 20:11:13 2012 New Revision: 189866 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189866 Log: Backport the following patches

[Bug fortran/54096] Type bound procedures

2012-07-25 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54096 Tobias Burnus burnus at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last

[Bug libfortran/54097] New: configure: error: GNU Fortran is not working (CPU you selected does not support x86-64 instruction set)

2012-07-25 Thread pentium4borg at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54097 Bug #: 54097 Summary: configure: error: GNU Fortran is not working (CPU you selected does not support x86-64 instruction set) Classification: Unclassified Product: gcc Version:

[Bug libfortran/54097] configure: error: GNU Fortran is not working (CPU you selected does not support x86-64 instruction set)

2012-07-25 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54097 Andrew Pinski pinskia at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[Bug libfortran/54097] configure: error: GNU Fortran is not working (CPU you selected does not support x86-64 instruction set)

2012-07-25 Thread pentium4borg at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54097 pentium4borg at gmail dot com changed: What|Removed |Added CC||pentium4borg at gmail dot

[Bug target/54089] [SH] Refactor shift patterns

2012-07-25 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54089 --- Comment #1 from Oleg Endo olegendo at gcc dot gnu.org 2012-07-25 23:03:12 UTC --- Author: olegendo Date: Wed Jul 25 23:03:06 2012 New Revision: 189872 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189872 Log: PR target/54089 *

[Bug libfortran/54097] configure: error: GNU Fortran is not working (CPU you selected does not support x86-64 instruction set)

2012-07-25 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54097 --- Comment #3 from Andrew Pinski pinskia at gcc dot gnu.org 2012-07-25 23:11:53 UTC --- (In reply to comment #2) (In reply to comment #1) Simple answer don't use -march=i686 in any of the *C*FLAGS env variables. Why is that? I thought

[Bug fortran/54096] Type bound procedures

2012-07-25 Thread badass at vt dot edu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54096 --- Comment #3 from badass at vt dot edu 2012-07-25 23:42:16 UTC --- Sorry about that, I though i had included the source code. I'm working on getting an updated compiler, but IT here is being difficult and I could take a day. Thanks for the help,

[Bug target/51244] SH Target: Inefficient conditional branch

2012-07-25 Thread olegendo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51244 --- Comment #44 from Oleg Endo olegendo at gcc dot gnu.org 2012-07-26 00:20:05 UTC --- Author: olegendo Date: Thu Jul 26 00:19:58 2012 New Revision: 189877 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189877 Log: PR target/51244 *

[Bug target/54093] ICE in in extract_insn, at recog.c:2129

2012-07-25 Thread amodra at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54093 Alan Modra amodra at gmail dot com changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last

Re: [patch] un-#ifdef GATHER_STATISTICS

2012-07-25 Thread Andrey Belevantsev
On 24.07.2012 21:13, Steven Bosscher wrote: AFAIR the qsort is just for getting a stable ordering over two pools to find the leaked regsets afterwards, the type of ordering doesn't matter. What matters is that the compare function gives a reliable result. You can't subtract pointers like that

Re: [PATCH] Detect loops in find_comparison_args

2012-07-25 Thread Paolo Bonzini
Il 24/07/2012 22:17, Sandra Loosemore ha scritto: I was looking to see what needs to be done to un-stick this previously submitted patch: http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01419.html Paolo's suggestion was to re-write this to use a tortoise-and-hare algorithm to detect the

[SH] PR 54089 - Reorg left shifts

2012-07-25 Thread Oleg Endo
Hello, The attached patch reorganizes the SH left shift patterns. The final generated shift insns don't differ much, but it seems to have some positive side effects on register allocation in some cases. Tested with make -k check RUNTESTFLAGS=--target_board=sh-sim

Re: [Patch/RFC] SEH exceptions for Win64

2012-07-25 Thread Tristan Gingold
Gerald, it is ok ? Tristan. On Jul 19, 2012, at 12:15 PM, Tristan Gingold wrote: On Jul 19, 2012, at 11:37 AM, Pedro Alves wrote: On 07/19/2012 08:30 AM, Tristan Gingold wrote: No regression on i386 GNU/Linux. Committed. Nice. Is this NEWS / docs --or wherever release notes are

VRP PLUS/MINUS_EXPR

2012-07-25 Thread Marc Glisse
Hello, here is a slight improvement to VRP for sum and difference of intervals. There are some things I left because I didn't understand them enough: * range_int_cst_p (vr0): I thought it was always true by that time, but it isn't obvious * TYPE_PRECISION (expr_type) =

Re: [Test] Fix for PRPR53981

2012-07-25 Thread Anna Tikhonova
Thanks! I've removed declarations. New patch attached. testsuite/ChangeLog: 2012-07-20 Anna Tikhonova anna.m.tikhon...@gmail.com * gcc.dg/20020201-1.c: Remove declarations for exit, abort, rand, srand. Include stdlib.h. 2012/7/24 Janis Johnson janis_john...@mentor.com: On

Re: [PATCH] Fix PR 52631 (VN does not use simplified expression for lookup)

2012-07-25 Thread Richard Guenther
On Tue, Jul 24, 2012 at 5:50 PM, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: Hi, Before tuples was introduced, VN used to lookup the simplified expression to see if it was available already and use that instead of the non simplified one. This patch adds the support back to VN to

Re: [PATCH] Fix comment in cgraphunit.c

2012-07-25 Thread Richard Guenther
On Tue, Jul 24, 2012 at 9:27 PM, Marek Polacek pola...@redhat.com wrote: Ping. Ok. Thanks, Richard. On Fri, Jul 13, 2012 at 02:00:54PM +0200, Marek Polacek wrote: I think the comment at the start of the file is wrong, since it speaks about varpool_finalize_variable, but there's no such

Re: [4.7 RFT PATCH, i386]: Backport recent LEA improvements to 4.7 branch

2012-07-25 Thread Richard Guenther
On Tue, Jul 24, 2012 at 9:32 PM, Uros Bizjak ubiz...@gmail.com wrote: Hello! Attached patch backports recent LEA improvements to 4.7 branch. As in regression fixes, or wrong-code fixes? This looks suspiciously not appropriate for 4.7 ... Richard. The patch unifies handling of LEA

Re: [patch] un-#ifdef GATHER_STATISTICS

2012-07-25 Thread Richard Guenther
On Tue, Jul 24, 2012 at 7:13 PM, Steven Bosscher stevenb@gmail.com wrote: AFAIR the qsort is just for getting a stable ordering over two pools to find the leaked regsets afterwards, the type of ordering doesn't matter. What matters is that the compare function gives a reliable result. You

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Kirill Yukhin
Please see attached patch that does all this with minimum surgery and also sets missing SSE prefetch for x86_32 with -mprfchw Thanks, applied! +++ b/gcc/config/i386/prfchwintrin.h +#if !defined _X86INTRIN_H_INCLUDED !defined _MM3DNOW_H_INCLUDED +# error Never use prfchwintrin.h directly;

Re: [patch] un-#ifdef GATHER_STATISTICS

2012-07-25 Thread Steven Bosscher
On Wed, Jul 25, 2012 at 1:49 PM, Richard Guenther richard.guent...@gmail.com wrote: Oh, bigger bitmap_head? That's bad ... :/ So much for '#ifdefs are bad' :/ Bigger bitmap_head isn't a problem. A bigger bitmap_element would be bad. For GGC allocated bitmaps, nothing changed (rounding, etc.).

Re: [4.7 RFT PATCH, i386]: Backport recent LEA improvements to 4.7 branch

2012-07-25 Thread Uros Bizjak
On Wed, Jul 25, 2012 at 1:45 PM, Richard Guenther richard.guent...@gmail.com wrote: On Tue, Jul 24, 2012 at 9:32 PM, Uros Bizjak ubiz...@gmail.com wrote: Hello! Attached patch backports recent LEA improvements to 4.7 branch. As in regression fixes, or wrong-code fixes? This looks

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Uros Bizjak
On Wed, Jul 25, 2012 at 1:51 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Please see attached patch that does all this with minimum surgery and also sets missing SSE prefetch for x86_32 with -mprfchw Thanks, applied! +++ b/gcc/config/i386/prfchwintrin.h +#if !defined

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Kirill Yukhin
Is it OK for trunk? OK. Thanks! Checked in. http://gcc.gnu.org/viewcvs?view=revisionrevision=189844 Next I think would be rdseed* insns. Thanks, K

[PATCH] Fix typo in comment in basic-block.h

2012-07-25 Thread Dodji Seketeli
This fixes a typo in a comment. Applied to trunk as obvious. gcc/ * basic-block.c: Fix typo in comment. diff --git a/gcc/basic-block.h b/gcc/basic-block.h index 15d34e0..dff06e4 100644 --- a/gcc/basic-block.h +++ b/gcc/basic-block.h @@ -108,7 +108,7 @@ struct GTY(()) gimple_bb_info {

Re: VRP PLUS/MINUS_EXPR

2012-07-25 Thread Richard Guenther
On Wed, Jul 25, 2012 at 12:21 PM, Marc Glisse marc.gli...@inria.fr wrote: Hello, here is a slight improvement to VRP for sum and difference of intervals. There are some things I left because I didn't understand them enough: * range_int_cst_p (vr0): I thought it was always true by that time,

Re: [patch] un-#ifdef GATHER_STATISTICS

2012-07-25 Thread Richard Guenther
On Wed, Jul 25, 2012 at 1:59 PM, Steven Bosscher stevenb@gmail.com wrote: On Wed, Jul 25, 2012 at 1:49 PM, Richard Guenther richard.guent...@gmail.com wrote: Oh, bigger bitmap_head? That's bad ... :/ So much for '#ifdefs are bad' :/ Bigger bitmap_head isn't a problem. A bigger

Re: [PATCH] PR target/53633; disable return value warnings for naked functions

2012-07-25 Thread nick clifton
Hi Sandra, One suggestion - rather than having architecture specific test files, why not just have a single generic test case with a new dg-require-naked-attribute qualifier. That way the mcore port would be tested as well as the ARM port. Something like this? The code part of the patch is

[PATCH] Remove FIXUNS_TRUNC_LIKE_FIX_TRUNC

2012-07-25 Thread Richard Henderson
The optabs rewrite removed the special handling for FIXUNS_TRUNC_LIKE_FIX_TRUNC. This was a minor short-hand only used by two targets, and can be replaced by a grand total of two patterns in each. Cross-compiled to vax-linux and v850-elf and committed. r~ * system.h

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Kirill Yukhin
Hi again, Here is second patch which adds support of rdseed[16|32|64] insn. Changelog entry: 2012-07-25 Kirill Yukhin kirill.yuk...@intel.com Michael Zolotukhin michael.v.zolotuk...@intel.com * common/config/i386/i386-common.c (OPTION_MASK_ISA_RDSEED_SET): New.

C++ PATCH to remove tf_no_access_control

2012-07-25 Thread Jason Merrill
Now that access control is subject to SFINAE, I was worried that tf_no_access_control might cause wrong overload resolution. On further investigation I think that it wasn't a problem, but it is unnecessary. To preserve the access control checking on default arguments, I changed

Re: PR 54075 Fix hashtable::reserve

2012-07-25 Thread Jonathan Wakely
(CC gcc-patches) On 25 July 2012 10:26, François Dumont wrote: Hi Here is a patch proposal for PR 54075. I also took the occasion to fix something that has been delay so far which is usage of std::max to get the number of buckets to use. The problem of using std::max when using the hash

C++ PATCHes for c++/54020, 54086 (minor constexpr issues)

2012-07-25 Thread Jason Merrill
54020 notes that we were accepting a constexpr function that is ill-formed, no diagnostic required, but it would be easy to give a diagnostic for. 54086 suggests that the standard seems to allow constexpr and const together. Tested x86_64-pc-linux-gnu, applying to trunk. commit

[PATCH] Fix assembly dialect handling in asm_fprintf

2012-07-25 Thread Siddhesh Poyarekar
Hi, While reviewing code to fix a previous patch submission around assembly dialect handling[1], I found that the dialect handling in asm_fprintf is different from that in output_asm_insn and it might be broken too, since encountering a '|' simply leads to skipping all of the string until a '}'

[PATCH] Fix Ada bootstrap (PR 54092)

2012-07-25 Thread Richard Henderson
That's what I get for assuming front-ends don't touch back-end things like optabs and libcalls... I'm reasonably certain that a better solution would be to move the set_stack_check_libfunc declaration elsewhere and/or totally rearrange that specific interface: c.f.

Re: [PATCH] Fix assembly dialect handling in asm_fprintf

2012-07-25 Thread Richard Henderson
On 07/25/2012 08:10 AM, Siddhesh Poyarekar wrote: 2012-07-25 Siddhesh Poyarekar siddh...@redhat.com * final.c [ASSEMBLER_DIALECT](do_assembler_dialects): New function to implement assembler dialects. (output_asm_insn): Use do_assembler_dialects. (asm_fprintf):

Re: [PATCH] Fix assembly dialect handling in asm_fprintf

2012-07-25 Thread Siddhesh Poyarekar
On Wed, 25 Jul 2012 08:31:14 -0700, Richard wrote: * gcc.dg/asm-dialect-1.c: New test case. ... except this should go in gcc.target/i386/ without the { target } qualifier. Thanks, here's the updated version. Regards, Siddhesh gcc/ChangeLog: 2012-07-25 Siddhesh Poyarekar

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Uros Bizjak
On Wed, Jul 25, 2012 at 4:07 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Here is second patch which adds support of rdseed[16|32|64] insn. Changelog entry: 2012-07-25 Kirill Yukhin kirill.yuk...@intel.com Michael Zolotukhin michael.v.zolotuk...@intel.com *

Re: [PATCH] PR target/53633; disable return value warnings for naked functions

2012-07-25 Thread Richard Henderson
On 07/24/2012 09:40 PM, Sandra Loosemore wrote: PR target/53633 gcc/ * target.def (warn_func_return): New hook. * doc/tm.texi.in (TARGET_WARN_FUNC_RETURN): New hook. * doc/tm.texi: Regenerate. * doc/sourcebuild.texi (Effective-Target Keywords): Document

Re: [google/gcc-4_7] Add -gfission as alias for -gsplit-dwarf (issue6444043)

2012-07-25 Thread Sterling Augustine
On Tue, Jul 24, 2012 at 5:51 PM, Cary Coutant ccout...@google.com wrote: This patch is for the google/gcc-4_7 branch. Add -gfission as alias for -gsplit-dwarf. Tested by bootstrap and regression testing. * common.opt (-gfission): Alias for -gsplit-dwarf. (-gno-fission):

[PATCH] Change IVOPTS and strength reduction to use expmed cost model

2012-07-25 Thread William J. Schmidt
Per Richard Henderson's suggestion (http://gcc.gnu.org/ml/gcc-patches/2012-06/msg01370.html), this patch changes the IVOPTS and straight-line strength reduction passes to make use of data computed by init_expmed. This required adding a new convert_cost array in expmed to store the costs of

Re: [PATCH] Fix Ada bootstrap (PR 54092)

2012-07-25 Thread Steven Bosscher
On Wed, Jul 25, 2012 at 5:20 PM, Richard Henderson r...@redhat.com wrote: That's what I get for assuming front-ends don't touch back-end things like optabs and libcalls... That is _almost_ the case. In fact, this is one of the last problems... $ egrep

[patch] Tidy up BIT_FIELD_REF handling code

2012-07-25 Thread Eric Botcazou
As documented in tree.def and checked in tree-cfg.c, the second and third operands of BIT_FIELD_REF are INTEGER_CSTs so there is no need to do fancy things on them. The patch also removes the now unused build6 stuff and fixes a buglet in the folder. Tested on x86_64-suse-linux, OK for the

Re: [PATCH] Change IVOPTS and strength reduction to use expmed cost model

2012-07-25 Thread Richard Henderson
On 07/25/2012 09:13 AM, William J. Schmidt wrote: Per Richard Henderson's suggestion (http://gcc.gnu.org/ml/gcc-patches/2012-06/msg01370.html), this patch changes the IVOPTS and straight-line strength reduction passes to make use of data computed by init_expmed. This required adding a new

Re: [Patch ARM] Pass --be8 option for big endian targets by default

2012-07-25 Thread Richard Earnshaw
On 24/07/12 23:26, Bharathi Seshadri (bseshadr) wrote: Hi, Attached is a patch that modifies BE8_LINK_SPEC to pass -be8 option to the linker by default if the target is big endian, and to not pass -be8 option if -mlittle-endian is used. It also preserves the existing usage whereby -be8

Re: [patch] Tidy up BIT_FIELD_REF handling code

2012-07-25 Thread Richard Henderson
On 07/25/2012 09:30 AM, Eric Botcazou wrote: 2012-07-25 Eric Botcazou ebotca...@adacore.com * expr.c (expand_expr_real_1): Do not expand operand #1 and #2 of BIT_FIELD_REF for ignored results. * fold-const.c (fold_ternary_loc) BIT_FIELD_REF: Check that the

Re: [Ada] Lock-free implementation of protected objects

2012-07-25 Thread Richard Henderson
On 07/23/2012 08:13 AM, Geert Bosch wrote: IIUC, all ports are supposed to implement the atomic built-ins. If they are not supported in hardware, there should be a library function for it that uses locking. The problem we're trying to address is builds failing because of undefined

[Patch, Fortran, committed] Some spell fixes

2012-07-25 Thread Tobias Burnus
Committed as obvious (Rev. 189859). Tobias 2012-07-25 Tobias Burnus bur...@net-b.de * trans-types.c (gfc_real16_is_float128): Fix spelling in a comment. * trans.h (struct gfc_array_info): Ditto. * gfortran.h (gfc_expr): Ditto. * simplify.c

  1   2   >