Re: [patch, mips] Fix optimization bug involving nor instruction

2013-10-22 Thread Steve Ellcey
On Tue, 2013-10-22 at 19:12 +0100, Richard Sandiford wrote: Richard OK, but I am curious why you put parenthesis around the right hand side of the total expression. I.e. *total = (); That's the emacs formatting rule: OK, I have checked in the patch with the parenthesis (and your

Re: [PATCH][i386]Fix PR 57756

2013-10-17 Thread Steve Ellcey
print Steve Ellcey sell...@mips.com

Re: [PATCH][i386]Fix PR 57756

2013-10-17 Thread Steve Ellcey
OK too. Steve Ellcey sell...@mips.com

Re: [PATCH v2] Fix libgfortran cross compile configury w.r.t newlib

2013-10-01 Thread Steve Ellcey
) if test x${with_newlib} = xyes -a x${ac_cv_func_exit} = xno; then /Marcus This patch works on my mips-mti-elf target. Steve Ellcey sell...@mips.com

Re: [PATCH] Fix libgfortran cross compile configury w.r.t newlib

2013-09-27 Thread Steve Ellcey
; defaulting to 00400098 /tmp/ccdG78PN.o: In function `main': y.c:(.text+0x14): undefined reference to `exit' collect2: error: ld returned 1 exit status ubuntu-sellcey % echo $? 1 Steve Ellcey sell...@mips.com

Re: [PATCH] Fix libgfortran cross compile configury w.r.t newlib

2013-09-26 Thread Steve Ellcey
. The only reason they don't run into this problem too is that they don't depend on any long double functions or any other functions that are optionally built by newlib. I will test this patch on my targets and make sure it works for me, but I don't see why it would not. Steve Ellcey sell...@mips.com

Re: RFA: Store the REG_BR_PROB probability directly as an int

2013-09-25 Thread Steve Ellcey
... Works for me as well. Andreas. This patch worked for me as well on MIPS. I did a complete build and test overnight. Steve Ellcey sell...@mips.com

Re: [patch, libgfortran, configure] Cross-compile support for libgfortran

2013-09-23 Thread Steve Ellcey
On Mon, 2013-09-23 at 16:26 +0100, Marcus Shawcroft wrote: On 4 June 2013 20:49, Steve Ellcey sell...@mips.com wrote: This patch allows me to build libgfortran for a cross-compiling toolchain using newlib. Currently the checks done by AC_CHECK_FUNCS_ONCE fail with my toolchain because

Re: [PATCH] Fix PR57451 (Incorrect debug ranges emitted for -freorder-blocks-and-partition -g)

2013-08-23 Thread Steve Ellcey
. Thanks Teresa, Thanks Kaz. Steve Ellcey

Re: [PATCH] Fix PR57451 (Incorrect debug ranges emitted for -freorder-blocks-and-partition -g)

2013-08-22 Thread Steve Ellcey
insight to using next_active_insn vs. next_real_insn vs. next_insn, though it doesn't look like you have changed what is used here. Steve Ellcey sell...@mips.com

[patch, libtool] Top-level libtool.m4 patch for autoconf 2.68

2013-08-20 Thread Steve Ellcey
, is this patch OK to check in to the GCC tree? I will send seperate email to the binutils mailing list though perhaps permission to check it in to the GCC tree is sufficient to put it in binutils as well? Steve Ellcey sell...@mips.com 2013-08-20 Steve Ellcey sell...@mips.com

Re: MIPS tests (nan-legacy.c and nans-legacy.c) failing

2013-08-13 Thread Steve Ellcey
that it does? Maciej, the patch does work for me. Steve Ellcey sell...@mips.com

[patch, mips] Fix for loongson prefetching and -mabi=32

2013-08-12 Thread Steve Ellcey
and the instruction that is generated can go into a delay slot. OK for checkin? 2013-08-12 Steve Ellcey sell...@mips.com * config/mips/mips.md (prefetch): Use lw instead of ld on loongson in 32bit mode. diff --git a/gcc/config/mips/mips.md b/gcc/config/mips/mips.md index 397c40a

[patch, mips] Add nan2008 multilibs to mips-mti-* targets.

2013-08-07 Thread Steve Ellcey
for hard-float. OK to checkin? Steve Ellcey sell...@mips.com 2013-08-07 Steve Ellcey sell...@mips.com * config/mips/mti-linux.h (SYSROOT_SUFFIX_SPEC): Add nan2008. * config/mips/t-mti-elf (MULTILIB_OPTIONS): Make mips16 and micromips incompatible. Add nan2008

Re: Passes are now C++ classes (was Re: [PATCH 03/11] Handwritten part of conversion of passes to C++ classes)

2013-08-06 Thread Steve Ellcey
are not building. It looks like the mips reorg pass (pass_mips_machine_reorg2) in config/mips/mips.c was not converted and/or was not converted correctly. Steve Ellcey sell...@mips.com

Re: [patch, mips] Size savings for MIPS16 switch statements

2013-07-30 Thread Steve Ellcey
was thinking it might be better to increase the size of the switch statements. Here is a patch I have tested to fix these failures. Richard, does this look OK for checkin? Steve Ellcey sell...@mips.com 2013-07-30 Steve Ellcey sell...@mips.com * gcc.target/mips/code-readable-1.c: Increase

MIPS tests (nan-legacy.c and nans-legacy.c) failing

2013-07-30 Thread Steve Ellcey
it is about my targets (mips-mti-elf and mips-mti-linux-gnu) that would make this different from yours. Should the tests be modified to allow either output? Steve Ellcey sell...@mips.com

[patch, mips] Size savings for MIPS16 switch statements

2013-07-23 Thread Steve Ellcey
some testing of mips32 code as well and found that this change did not help with that code so I restricted the change to mips16 only. Tested on mips-mti-elf target, OK for checkin? Steve Ellcey sell...@mips.com 2013-07-23 Steve Ellcey sell...@mips.com * config/mips/mips.c

Re: RFA: Fix rtl-optimization/57425 (SPEC perl problem on MIPS)

2013-07-12 Thread Steve Ellcey
a smaller example but I wanted to see if you (or someone else on gcc-patches) was seeing anything like this on any other targets. Steve Ellcey sell...@mips.com

[patch, mips] Size optimization for MIPS

2013-07-09 Thread Steve Ellcey
with the -mips16 and saw no regressions. OK to checkin? Steve Ellcey sell...@mips.com Steve Ellcey sell...@mips.com * config/mips/mips.c (mips_conditional_register_usage): Do not use t[0-7] registers in MIPS16 mode when optimizing for size. diff --git a/gcc/config/mips/mips.c b/gcc

Re: [patch, mips] Size optimization for MIPS

2013-07-09 Thread Steve Ellcey
that could handle -mabi=eabi and ran the GCC testsuite, it all looked good. I am pretty sure the GCC testsuite has tests with 5+ arguments but I did a couple of hand tests as well and they also looked good. Steve Ellcey sell...@mips.com Steve Ellcey sell...@mips.com * config/mips/mips.c

Patch ping: [patch, libgfortran, configure] Cross-compile support for libgfortran

2013-06-21 Thread Steve Ellcey
Ping... These libgfortran changes are done the same way as libstdc++ so I hope they are OK. Steve Ellcey sell...@mips.com From sell...@mips.com Tue Jun 4 12:49:55 2013 This patch allows me to build libgfortran for a cross-compiling toolchain using newlib. Currently the checks done

Re: [Patch tree-ssa] RFC: Enable path threading for control variables (PR tree-optimization/54742).

2013-06-21 Thread Steve Ellcey
not happen at -Os then I think we can probably live with the space increase. Steve Ellcey sell...@mips.com

[patch, mips] Fix switch statement for mips16 (PR target/56942)

2013-06-19 Thread Steve Ellcey
to check it in? Since Steven added an assert in tablejump_p, I did not include any here, though I could if we thought it was needed. Steve Ellcey sell...@mips.com 2013-06-19 Steve Ellcey sell...@imgtec.com PR target/56942 * config/mips/mips.md (casesi_internal_mips16_mode

Re: [Patch tree-ssa] RFC: Enable path threading for control variables (PR tree-optimization/54742).

2013-06-19 Thread Steve Ellcey
in general is turned off for -Os but it probably should be. % ll -art coremark.fsf*elf -rwxr-xr-x 1 sellcey src 413812 Jun 19 11:11 coremark.fsf.1.elf -rwxr-xr-x 1 sellcey src 414676 Jun 19 11:11 coremark.fsf.2.elf -rwxr-xr-x 1 sellcey src 416402 Jun 19 11:11 coremark.fsf.3.elf Steve Ellcey sell

Re: [Patch tree-ssa] RFC: Enable path threading for control variables (PR tree-optimization/54742).

2013-06-14 Thread Steve Ellcey
-DRTEYAMON With my version the compiler calls gimple_duplicate_sese_region from duplicate_blocks 60 times. With your patch it calls gimple_duplicate_sese_region from duplicate_thread_path 16 times. Steve Ellcey sell...@mips.com

Re: [Patch tree-ssa] RFC: Enable path threading for control variables (PR tree-optimization/54742).

2013-06-13 Thread Steve Ellcey
, but with this patch it still exists. Maybe we need an option to do multiple searches? Steve Ellcey sell...@mips.com

RE: [patch, mips] Fix for PR target/56942

2013-06-10 Thread Steve Ellcey
Steven, The assert has been in ToT for over a week now and I haven't seen any problems reported. Is it time to move on to the next step? Steve Ellcey sell...@mips.com From: Steven Bosscher [stevenb@gmail.com] Sent: Wednesday, May 29, 2013 3:15 PM

[patch, mips] Micromips delay slot fix

2013-06-10 Thread Steve Ellcey
since he doesn't have write access. OK to checkin? 2013-06-10 Andrew Bennett andrew.benn...@imgtec.com Steve Ellcey sell...@mips.com * config/mips/mips.md (single_insn): Fix attribute for micromips. diff --git a/gcc/config/mips/mips.md b/gcc/config/mips/mips.md index 2fdc79d

Re: [patch, mips] Micromips delay slot fix

2013-06-10 Thread Steve Ellcey
On Mon, 2013-06-10 at 23:13 +0100, Maciej W. Rozycki wrote: On Mon, 10 Jun 2013, Steve Ellcey wrote: We found a bug in the micromips implementation where the branch delay slot was not getting filled for micromips. You can reproduce this with a program that just has an empty main

[patch, libgfortran, configure] Cross-compile support for libgfortran

2013-06-04 Thread Steve Ellcey
in the libgfortran configure.ac, but it seems to assume that the type exists. OK to checkin? Steve Ellcey sell...@mips.com 2013-06-04 Steve Ellcey sell...@mips.com * configure.ac (AC_CHECK_FUNCS_ONCE): Put into if statement. diff --git a/libgfortran/configure.ac b/libgfortran/configure.ac index

[patch, testsuite, cilk] Fix cilk tests for simulators

2013-06-03 Thread Steve Ellcey
the constant value 1 instead of using argc and in one (where argc was being changed) I replaced the use of argc with a local variable. Tested on mips-mti-elf with the GNU simulator. OK to checkin? 2013-06-03 Steve Ellcey sell...@mips.com * c-c++-common/cilk-plus/AN/array_test2.c: Do

Re: [patch, testsuite, cilk] Fix cilk tests for simulators

2013-06-03 Thread Steve Ellcey
to avoid argc/argv than support two paths through the test, one with argc/argv, one without. jeff I'll leave fixing the tests to Balaji then instead of doing it myself since that way he can be sure that they are testing what he wants to test. Steve Ellcey sell...@mips.com

Re: [patch, mips] Fix for PR target/56942

2013-05-30 Thread Steve Ellcey
-linux-gnu and mips-mti-elf by building cross compilers and testing and I did not hit the assert either. Steve Ellcey sell...@imgtec.com

Re: [patch] Default to --enable-libstdcxx-time=auto

2013-05-28 Thread Steve Ellcey
] } I will dig around some more but I thought I would see if you had any ideas on what the problem was and how to fix it. Steve Ellcey sell...@imgtec.com

Re: [patch] Default to --enable-libstdcxx-time=auto

2013-05-28 Thread Steve Ellcey
. Steve Ellcey sell...@imgtec.com

Re: [patch, mips] Patch for mips generic scheduler

2013-05-28 Thread Steve Ellcey
. Thanks, Richard OK, I did some testing and benchmarking and did not find a clear best choice so I think I will go with PROCESSOR_74KF2_1. Here is the patch I will check in shortly: 2013-05-28 Steve Ellcey sell...@mips.com * config/mips/mips-cpus.def (mips32r2): Change processor type

[patch, mips] Update multilibs for mips-mti-* targets

2013-05-28 Thread Steve Ellcey
mips targets, OK to checkin? Steve Ellcey sell...@imgtec.com 2013-05-28 Steve Ellcey sell...@imgtec.com * config/mips/mti-linux.h (SYSROOT_SUFFIX_SPEC): Add micromips and mips16 directories. * config/mips/t-mti-linux (MULTILIB_OPTIONS): Add micromips and mips16

Re: [patch, mips] Patch for mips generic scheduler

2013-05-21 Thread Steve Ellcey
to PROCESSOR_M4K and mips32 defaults to PROCESSOR_4KC. I don't see any special scheduler for m4k. Is there supposed to be a scheduler for m4k? Steve Ellcey

Re: [PATCH][gensupport] Add optional attributes field to define_cond_exec

2013-05-20 Thread Steve Ellcey
and don't have access to any IA64 boxes (other than the one in the GCC compile farm), so I don't think I can confirm this for you. Steve Ellcey sell...@imgtec.com

[patch, mips] Patch for mips generic scheduler

2013-05-20 Thread Steve Ellcey
that the integer madd instruction uses the alu and the imuldiv functional units? Tested on 74kc and proAptive with no regressions and better performance. OK to checkin? Steve Ellcey sell...@imgtec.com 2013-05-20 Steve Ellcey sell...@imgtec.com * config/mips/generic.md (generic_imul): Remove imadd

Re: [PATCH] New switch optimization pass (PR tree-optimization/54742)

2013-05-16 Thread Steve Ellcey
updates that Jeff wanted but left the name of the function as is. Steve Ellcey sell...@imgtec.com

Re: [PATCH] New switch optimization pass (PR tree-optimization/54742)

2013-05-15 Thread Steve Ellcey
a flag to gimple_duplicate_sese_region to tell it whether or not to update the dominator information. I had to add the same flag to copy_bbs to make it all work. How does this look? I tested it with a bootstrap and test on x86 (with my optimization enabled) and got no regressions. 2013-05-15 Steve

Re: [patch] Hash table changes from cxx-conversion branch - config part

2013-05-14 Thread Steve Ellcey
On Mon, 2013-05-13 at 15:03 -0700, Lawrence Crowl wrote: I still have not heard from i386 or ia64 folks. Anyone? The IA64 part looks OK to me. Steve Ellcey sell...@imgtec.com (sell...@mips.com)

Re: [PATCH] New switch optimization pass (PR tree-optimization/54742)

2013-05-14 Thread Steve Ellcey
that. Steve Ellcey sell...@imgtec.com 2013-05-14 Steve Ellcey sell...@imgtec.com * dominance.c (iterate_fix_dominators): Add null check. diff --git a/gcc/dominance.c b/gcc/dominance.c index 5c96dad..d858ad1 100644 --- a/gcc/dominance.c +++ b/gcc/dominance.c @@ -1251,6 +1251,7

[PATCH] New switch optimization pass (PR tree-optimization/54742)

2013-05-13 Thread Steve Ellcey
? OK to checkin? Steve Ellcey sell...@imgtec.com 2013-05-13 Steve Ellcey sell...@imgtec.com PR tree-optimization/54742 * Makefile.in (OBJS): Add tree-switch-shortcut.o. * common.opt (ftree-switch-shortcut): New. * dominance.c (iterate_fix_dominators): Add null

RE: [patch][mips] split mips_reorg in pre- and post-dbr_schedule parts

2013-05-07 Thread Steve Ellcey
Steven, This patch has broken the GCC build for my mips-mti-linux-gnu and mips-mti-elf builds. The GCC build dies when trying to configure libgcc. Do you have any idea what might be going on? Steve Ellcey sell...@imgtec.com checking for mips-mti-linux-gnu-gcc... /local/home/sellcey

Re: [patch][mips] split mips_reorg in pre- and post-dbr_schedule parts

2013-05-07 Thread Steve Ellcey
On Tue, 2013-05-07 at 18:39 +0200, Steven Bosscher wrote: On Tue, May 7, 2013 at 5:56 PM, Steve Ellcey wrote: Steven, This patch has broken the GCC build for my mips-mti-linux-gnu and mips-mti-elf builds. The GCC build dies when trying to configure libgcc. Do you have any idea what

Re: [patch][mips] split mips_reorg in pre- and post-dbr_schedule parts

2013-05-07 Thread Steve Ellcey
case anyhow. */ mips_reorg_process_insns (); mips16_split_long_branches (); + return 0; } struct rtl_opt_pass pass_mips_machine_reorg2 = Yes, this fixed my build. Steve Ellcey sell...@imgtec.com

Re: [patch, mips] Allow users to avoid promoting prototypes.

2013-05-03 Thread Steve Ellcey
On Thu, 2013-05-02 at 23:26 +0100, Richard Sandiford wrote: 2013-05-02 Steve Ellcey sell...@imgtec.com * config/mips/mips.c (mips_promote_prototypes) :New. (TARGET_PROMOTE_PROTOTYPES): Change to use mips_promote_prototypes. * config/mips/mips.opt (mpromote-prototypes

Re: [patch, mips] Fix for PR target/56942

2013-05-03 Thread Steve Ellcey
read: active was a _more_ restrictive condition than real. Richard Steven, Do you have any comment on Richard's question? I am basically stuck between the two of you with no way to fix my problem at this point unless you allow the generic fix or Richard allows the MIPS specific fix. Steve

[patch, mips] Allow users to avoid promoting prototypes.

2013-05-02 Thread Steve Ellcey
of turning if off if desired. Tested on mips-mti-elf. OK for checkin? Steve Ellcey sell...@imgtec.com 2013-05-02 Steve Ellcey sell...@imgtec.com * config/mips/mips.c (mips_promote_prototypes) :New. (TARGET_PROMOTE_PROTOTYPES): Change to use mips_promote_prototypes

Re: [patch, mips] Fix for PR target/56942

2013-04-29 Thread Steve Ellcey
should keep that if possible, even during the transition period. Thanks, Richard OK, here is patch to next_real_insn to keep the ordering property intact and fix the bug. OK for checkin? Steve Ellcey sell...@imgtec.com 2013-04-29 Andrew Bennett andrew.benn...@imgtec.com Steve

Re: [patch, mips] Fix for PR target/56942

2013-04-26 Thread Steve Ellcey
On Wed, 2013-04-24 at 07:45 +0100, Richard Sandiford wrote: Steve Ellcey sell...@imgtec.com writes: 2013-04-19 Andrew Bennett andrew.benn...@imgtec.com Steve Ellcey sell...@imgtec.com PR target/56942 * config/mips/mips.md (casesi_internal_mips16_mode): Use

[patch, mips] Fix for PR target/56942

2013-04-19 Thread Steve Ellcey
Andrew Bennett found this fix to my MIPS build problem (PR target/56942). He does not have write access so I am submitting it for checkin, is this a simple enough fix for an 'obvious' checkin? I did a build and regression test to verify the fix. Steve Ellcey sell...@imgtec.com 2013-04-19

Re: [patch, mips] Patch to control the use of integer madd/msub instructions

2013-03-25 Thread Steve Ellcey
be right. Actually, one of my reasons for adding this flag is to make it easier for me to do 74k runs with and without madd/msub and see where we are using (but shouldn't) and hopefully improve the current implementation. Steve Ellcey sell...@mips.com

Re: [patch, mips] Patch to control the use of integer madd/msub instructions

2013-03-25 Thread Steve Ellcey
to leave -mllsc as it is but do -mimadd in the same way as -mbranch-likely. Thanks, Richard OK, Here is a patch that implements -mimadd in the same manner as -mbranch-likely. Steve Ellcey sell...@imgtec.com 2013-03-25 Steve Ellcey sell...@mips.com * config/mips/mmips-cpus.def (74kc

[patch, mips] Patch to control the use of integer madd/msub instructions

2013-03-22 Thread Steve Ellcey
but does not change that default behaviour. OK for checkin? Steve Ellcey sell...@imgtec.com 2013-03-22 Steve Ellcey sell...@mips.com * config/mips/mips.md (mimadd): New flag for integer madd/msub. * config/mips/mips.h (GENERATE_MADD_MSUB): Check -mimadd flag. diff --git a/gcc

[patch, testsuite] Obvious patch for gcc.dg/pr56396.c

2013-03-01 Thread Steve Ellcey
I am going to check in this testsuite patch as an obvious fix. We should make sure a platform supports PIC before compiling with the -fpic flag. Steve Ellcey sell...@imgtec.com 2013-03-01 Steve Ellcey sell...@mips.com * gcc.dg/pr56396.c: Require pic support. diff --git a/gcc

RE: PR target/52555: attribute optimize is overriding command line options

2013-03-01 Thread Steve Ellcey
to be a valid pointer but it is not pointing at default_target_optabs addr and so I get the assert. I am still trying to dig out more information and create a good test case. Steve Ellcey sell...@imgtec.com

RE: [PATCH] MIPS: MIPS32r2 FP MADD instruction set support

2013-02-27 Thread Steve Ellcey
. They are also required for mips64 release 1 implementations. The AVPs check for this. Steve Ellcey

Re: PR target/52555: attribute optimize is overriding command line options

2013-02-22 Thread Steve Ellcey
. Jakub, I built with this patch and ran the GCC testsuite (using the target mips-mti-linux-gnu), it fixed the problem and caused no regressions for me. Steve Ellcey sell...@imgtec.com

RE: PR target/52555: attribute optimize is overriding command line options

2013-02-21 Thread Steve Ellcey
the test times out. I will keep digging and see if I can figure out what is going on but I wanted to see if anyone else has reported this problem. Steve Ellcey sell...@imgtec.com

[patch, libstdc++, testsuite] Fix simulator timouts.

2013-01-14 Thread Steve Ellcey
to checkin? Steve Ellcey sell...@mips.com 2013-01-14 Steve Ellcey sell...@mips.com * testsuite/libstdc++-v3/testsuite/23_containers/vector/profile/vector.cc: Shrink memory usage under simulator. * testsuite/libstdc++-v3/testsuite/26_numerics/valarray/28277.cc: Ditto

Re: [patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-08 Thread Steve Ellcey
On Mon, 2013-01-07 at 12:54 -0800, Steve Ellcey wrote: On Mon, 2013-01-07 at 20:38 +, Richard Sandiford wrote: Ah, sorry, this reminded me of a patch I'd written a year ago and never got around to sanity-checking. Does it fix the problem for you? Richard It fixes my small

[patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-07 Thread Steve Ellcey
removing STABS completely from GCC, etc. but it looks like there is still some users who want it on some targets. Here is my default patch which just undef's it for the mti targets. Steve Ellcey sell...@mips.com 2013-01-07 Steve Ellcey sell...@mips.com * config/mips/mips-mti-elf.h

[patch, mips] Change -mad to -mmad to match documentation

2013-01-07 Thread Steve Ellcey
could leave -mad and -mno-ad as aliases for -mmad and -mno-ad but I don't think that is necessary. If someone does think we should do allow -mad, please let me know. OK to checkin? Steve Ellcey sell...@mips.com 2013-01-07 Steve Ellcey sell...@mips.com PR target/42661 * config

Re: [patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-07 Thread Steve Ellcey
On Mon, 2013-01-07 at 20:30 +, Richard Sandiford wrote: Steve Ellcey sell...@mips.com writes: While testing all the variations of my mips-mti-elf target I found that a number of debug tests like gcc.dg/debug/trivial.c fail when compiled using a stabs debug flag (-gstabs3 for example

Re: [patch, mips, stabs] Do any MIPS targets use stabs?

2013-01-07 Thread Steve Ellcey
On Mon, 2013-01-07 at 20:38 +, Richard Sandiford wrote: Ah, sorry, this reminded me of a patch I'd written a year ago and never got around to sanity-checking. Does it fix the problem for you? Richard It fixes my small test case, I will try it on the full testsuite overnight. Steve

Re: [patch] Performance patch for MIPS conditional move in expr.c

2013-01-07 Thread Steve Ellcey
running with your new *movGPR:mode_on_GPR2:mode_ne instruction and that part at least works fine. I would like to get at least that much checked in for GCC 4.8. Steve Ellcey sell...@mips.com

Re: [Patch, PR 54128] ira.c change to fix mips bootstrap

2012-12-21 Thread Steve Ellcey
On Fri, 2012-12-21 at 08:46 +0100, Jakub Jelinek wrote: Given Alex' comments in the PR, the second hunk is definitely ok for trunk, the first one can be applied too (but you can skip it too if you want, it shouldn't make a difference). Thanks, I checked in both chunks. Steve Ellcey sell

[patch, libstdc++ testsuite] Shrink more tests for simulator

2012-12-14 Thread Steve Ellcey
Here are four more C++ tests that fail for me when run under the GNU simulator. I would like to shrink them to use less memory in the same way as the other tests that I modified earlier. OK to checkin? Steve Ellcey sell...@mips.com 2012-12-14 Steve Ellcey sell...@mips.com

[patch, testsuite, committed] Obvious fix for targets with no profiling

2012-12-14 Thread Steve Ellcey
I am going to check in this patch as the obvious fix for PR 55688. I tested it on mips-mti-elf. Steve Ellcey sell...@mips.com 2012-12-14 Steve Ellcey sell...@mips.com PR regression/55688 * g++.dg/other/pr55650.C: Add dg-require-profiling. diff --git a/gcc/testsuite/g++.dg

Re: [patch, libstdc++ testsuite] Shrink tests for simulator

2012-12-13 Thread Steve Ellcey
On Thu, 2012-12-13 at 20:27 +0100, Paolo Carlini wrote: It's the same approach that we are following in many other places, right? Yes, it is the same approach. OK to checkin? Yes, thanks. Done. Steve Ellcey sell...@mips.com

Re: [patch, mips] Follow up to pr54061 patch, fix another abort.

2012-12-12 Thread Steve Ellcey
distinguish between registers that have been disabled through -msoft-float, -mips3 and below (for ST_REGS other than $fcc0), -mno-dsp, etc. Richard OK, you have convinced me. Here is what I am testing, OK to checkin once I have run it through the testsuite? Steve Ellcey sell...@mips.com

[patch, testsuite] Check for pic support on tests that use -fPIC

2012-12-12 Thread Steve Ellcey
I think this patch qualifies as obvious so I will check it in later today if I don't get any objections. These two tests use the -fPIC flag but do not check for pic support. Tested on mips-mti-elf. Steve Ellcey sell...@mips.com 2012-12-12 Steve Ellcey sell...@mips.com * gcc.dg

[patch, mips] Patch to allow madd, etc without 64 bit floating point

2012-12-12 Thread Steve Ellcey
to reflect that we can now generate and use a rsqrt.d instruction. I put Chao-ying and Maciej's names on the patch since they did the original changes in GCC but those changes were never checked in upstream. OK to checkin? Steve Ellcey sell...@mips.com In the spec MD00086, page 347 Revision History

Re: [patch, mips] Patch to allow madd, etc without 64 bit floating point

2012-12-12 Thread Steve Ellcey
On Wed, 2012-12-12 at 22:21 +, Maciej W. Rozycki wrote: Given that, how about we coordinate these submissions -- I'll dig out the most recent version of the madd change and push it through testing before I post it and you do the reciprocals? Maciej That sounds good to me. Steve

[patch, mips] Follow up to pr54061 patch, fix another abort.

2012-12-11 Thread Steve Ellcey
)) __chk_fail (); while (c != '\0'); return s; } aborts when compiled with -O2 -g -fpic -mips16. The following patch fixes it. OK to checkin? Steve Ellcey sell...@mips.com 2012-12-11 Steve Ellcey sell...@mips.com * config/mips/mips.c (mips_option_override): Set

Re: [patch, mips] Follow up to pr54061 patch, fix another abort.

2012-12-11 Thread Steve Ellcey
not sure if we want to set all fake registers to IGNORED because of this one case. If more popped up then I could see us making that change. Steve Ellcey sell...@mips.com

Re: [patch, mips, debug] Fix PR 54061, mips compiler aborts in testsuite

2012-12-10 Thread Steve Ellcey
to dbx_reg_number too early in the last patch and getting aborts during testing due to that. I also used a variable to avoid calling it twice in a row now. This version passed the GCC testsuite with no regressions. Steve Ellcey sell...@mips.com 2012-12-07 Steve Ellcey sell...@mips.com PR

[patch, mips, testsuite] Fix scan on gcc.target/mips/ext-2.c

2012-12-10 Thread Steve Ellcey
the failure for me and I see no reason why GCC should not use the ext instruction instead of dext in this case. Steve Ellcey sell...@mips.com 2012-12-10 Steve Ellcey sell...@mips.com * gcc.target/mips/ext-2.c: Scan for ext or dext. diff --git a/gcc/testsuite/gcc.target/mips/ext-2.c b/gcc

[patch, mips] Fix -mno-shared option (gcc.target/mips/pr35802.c)

2012-12-10 Thread Steve Ellcey
with no regressions. 2012-12-10 Steve Ellcey sell...@mips.com * config/mips/gnu-user.h (NO_SHARED_SPECS): Add space before option. diff --git a/gcc/config/mips/gnu-user.h b/gcc/config/mips/gnu-user.h index d35ddac..55ca662 100644 --- a/gcc/config/mips/gnu-user.h +++ b/gcc/config/mips/gnu

[patch, testsuite] Fix pr35634 tests when char is unsigned.

2012-12-10 Thread Steve Ellcey
' is still a good idea I made that change as well as making 'c' an explicitly signed char. This now passes on mips-mti-elf. OK for checkin? Steve Ellcey sell...@mips.com 2012-12-10 Steve Ellcey sell...@mips.com * gcc.dg/torture/pr35634.c: Initialize n, make c signed. * g++.dg

Re: [patch, testsuite] Fix pr35634 tests when char is unsigned.

2012-12-10 Thread Steve Ellcey
tree. Steve Ellcey sell...@mips.com

[patch, mips, debug] Fix PR 54061, mips compiler aborts in testsuite

2012-12-07 Thread Steve Ellcey
haven't done a full regression test but will do that shortly, in the mean time I wanted to see if this approach was considered acceptable and if not, how I should fix it? Steve Ellcey sell...@mips.com 2012-12-07 Steve Ellcey sell...@mips.com PR target/54061 * dwarfwout.c

Re: [patch, mips, debug] Fix PR 54061, mips compiler aborts in testsuite

2012-12-07 Thread Steve Ellcey
be a target macro but so should DBX_REGISTER_NUMBER and DWARF_FRAME_REGNUM, we probably don't want to be making that change at this time in the release cycle. Steve Ellcey sell...@mips.com

Re: [patch, mips, debug] Fix PR 54061, mips compiler aborts in testsuite

2012-12-07 Thread Steve Ellcey
On Fri, 2012-12-07 at 14:43 -0800, H.J. Lu wrote: I think you need a way to tell a bad register from a good register which doesn't have DWARF register number. Here is a patch using your IGNORED_DWARF_REGNUM idea. Steve Ellcey sell...@mips.com 2012-12-07 Steve Ellcey sell...@mips.com

Re: [IA-64] Fix dynamic allocation in leaf functions

2012-12-04 Thread Steve Ellcey
to above change. Ping: http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01617.html Thanks in advance. Eric, this looks OK to me. As for back porting it, that is OK with me too as long as it is OK with the release/branch managers. Steve Ellcey sell...@mips.com

[patch, libgcc] Bug in fp-bit.c when NO_NANS is defined

2012-11-30 Thread Steve Ellcey
. isinf is always going to be false when NO_NANS is defined so it is only a question of what to do for iszero(a) that we need to worry about. b at this point may or may not be zero. OK to checkin or should I return something else? Steve Ellcey sell...@mips.com 2012-11-30 Steve Ellcey sell

[patch, pch] Fix pch failure on mips-mti-linux-gnu

2012-11-21 Thread Steve Ellcey
no regressions. OK to checkin? Steve Ellcey sell...@mips.com 2012-11-21 Steve Ellcey sell...@mips.com PR pch/55399 * files.c (pch_open_file): Fix check for implicit_preinclude. diff --git a/libcpp/files.c b/libcpp/files.c index a8288dc..9f84d8c 100644 --- a/libcpp/files.c +++ b/libcpp

Re: ping - Re: [patch] [ia64] add multiarch definitions for ia64-linux-gnu

2012-11-21 Thread Steve Ellcey
. updated patch attached below. OK for ia64. Steve Ellcey sell...@mips.com

[patch] Performance patch for MIPS conditional move in expr.c

2012-11-14 Thread Steve Ellcey
mode in the temp that we are creating for the conditional move. Tested on mips-mti-elf with no regressions. OK for checkin? Steve Ellcey sell...@mips.com 2012-11-14 Steve Ellcey sell...@mips.com * expr.c (expand_cond_expr_using_cmove): Use promoted mode for temp. diff --git a/gcc

Re: [patch] Performance patch for MIPS conditional move in expr.c

2012-11-14 Thread Steve Ellcey
will see fewer 'andi' instructions with 0x are generated after my patch is applied. Steve Ellcey sell...@mips.com unsigned short foo(unsigned short a1, unsigned short a2) { unsigned short i, x; for (i = 0; i 8; i++) { x = (a1 1) ^ (a2 1); a1 = 1; if (x == 1) a2 ^= 0x2006

Re: [patch] Performance patch for MIPS conditional move in expr.c

2012-11-14 Thread Steve Ellcey
When it should just do: andi$5,$5,0x1 movn$2,$4,$5 Steve Ellcey sell...@mips.com

[patch, mips] Do not make -Os the default with mips-mti-elf target.

2012-11-07 Thread Steve Ellcey
the existing --enable-target-optspace configure option to get it. This patch creates config/mt-mti that is like mt-sde but without -Os and changes the mips*-mti-elf* target to use that include file instead of mt-sde. Tested with the mips-mti-elf target. OK to checkin? Steve Ellcey sell...@mips.com

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-06 Thread Steve Ellcey
of the problem so my patch is not needed. Steve Ellcey sell...@mips.com

[patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Steve Ellcey
The test gcc.dg/torture/mips-sdata-1.c fails when compiled with -fno-fat-lto-objects because when that option is used no assembly code is output and thus the scan does not find the '.sdata' line. Checked with the mips-mti-elf target. OK to checkin? Steve Ellcey sell...@mips.com 2012-11-05

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Steve Ellcey
On Mon, 2012-11-05 at 20:38 +, Richard Sandiford wrote: This sort of thing should usually be handled automatically by scan-assembler, and is for me: /foo/gcc/xgcc -B/foo/gcc/ /bar/gcc/testsuite/gcc.dg/torture/mips-sdata-1.c -fno-diagnostics-show-caret -O2 -flto -fuse-linker-plugin

Re: [patch, testsuite, mips] Fix gcc.dg/torture/mips-sdata-1.c

2012-11-05 Thread Steve Ellcey
' and cause check_effective_target_lto to return FALSE and cause the -ffat-lto-objects flag to not be added. Of course this raises the question of why it is trying to do lto testing if I didn't configure it in. I will add 'lto' to my configure line and see if that fixes the problem. Steve Ellcey sell

<    1   2   3   4   5   6   7   >