Re: [patch, mips] delete bit-rotten ADJUST_REG_ALLOC_ORDER definition

2014-06-22 Thread Sandra Loosemore
On 05/14/2014 12:49 PM, Richard Sandiford wrote: Jeff Law l...@redhat.com writes: On 05/13/14 14:11, Sandra Loosemore wrote: 2014-05-13 Catherine Moore c...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.c

Re: [patch, aarch64] additional bics patterns

2014-11-13 Thread Sandra Loosemore
On 11/13/2014 10:27 AM, Richard Earnshaw wrote: On 13/11/14 17:05, Ramana Radhakrishnan wrote: On Thu, Nov 13, 2014 at 4:55 PM, Sandra Loosemore san...@codesourcery.com wrote: This patch to the AArch64 back end adds a couple of additional bics patterns to match code of the form if ((x y

Re: [patch, aarch64] additional bics patterns

2014-11-13 Thread Sandra Loosemore
On 11/13/2014 10:47 AM, Andrew Pinski wrote: On Thu, Nov 13, 2014 at 9:42 AM, Sandra Loosemore san...@codesourcery.com wrote: On 11/13/2014 10:27 AM, Richard Earnshaw wrote: On 13/11/14 17:05, Ramana Radhakrishnan wrote: On Thu, Nov 13, 2014 at 4:55 PM, Sandra Loosemore san

[patch, arm] align saved FP regs on stack

2014-11-14 Thread Sandra Loosemore
things. OK to commit? -Sandra 2014-11-14 Sandra Loosemore san...@codesourcery.com Joshua Conner jcon...@nvidia.com Chris Jones chr...@nvidia.com gcc/ * doc/invoke.texi (Option Summary): Add -malign-saved-fp-regs. (ARM options): Document it. * config/arm/arm.h (arm_stack_offsets

Re: [BUILDROBOT] nios2: build breakage

2014-11-15 Thread Sandra Loosemore
On 11/15/2014 04:49 PM, Jan-Benedict Glaw wrote: Hi, On Sun, 2014-11-16 00:36:27 +0100, Jan Hubicka hubi...@ucw.cz wrote: Yep, it is because my code does not handle streaming of arrays into the target optimization nodes. I will take a look on why that array is really needed. It seems like a

Re: [patch v2, aarch64] additional bics patterns

2014-11-19 Thread Sandra Loosemore
On 11/13/2014 10:47 AM, Andrew Pinski wrote: On Thu, Nov 13, 2014 at 9:42 AM, Sandra Loosemore san...@codesourcery.com wrote: On 11/13/2014 10:27 AM, Richard Earnshaw wrote: On 13/11/14 17:05, Ramana Radhakrishnan wrote: On Thu, Nov 13, 2014 at 4:55 PM, Sandra Loosemore san

Re: [BUILDROBOT] nios2: build breakage

2014-11-19 Thread Sandra Loosemore
On 11/15/2014 06:46 PM, Sandra Loosemore wrote: On 11/15/2014 04:49 PM, Jan-Benedict Glaw wrote: Hi, On Sun, 2014-11-16 00:36:27 +0100, Jan Hubicka hubi...@ucw.cz wrote: Yep, it is because my code does not handle streaming of arrays into the target optimization nodes. I will take a look

Re: [BUILDROBOT] nios2: build breakage

2014-11-19 Thread Sandra Loosemore
On 11/19/2014 09:34 PM, Jan Hubicka wrote: [snip] I see three possible fixes: 1) extend the AWk script to recognize arrays and stream them specially (it already recognizes string so it is not hard to do, just bit wasteful) 2) add attribute to .opt file allowing user to specify his own

[patch, testsuite] don't use dg-run in gcc.dg/vect tests

2014-09-25 Thread Sandra Loosemore
unconditionally instead of allowing the dg-require-effective-target mechanism to decide whether the target can run code compiled with the vectorization options added by vect.exp. This patch fixes the bad tests. OK to check in? -Sandra 2014-09-25 Sandra Loosemore san...@codesourcery.com

Re: [patch, testsuite] don't use dg-run in gcc.dg/vect tests

2014-09-25 Thread Sandra Loosemore
On 09/25/2014 02:04 PM, Sandra Loosemore wrote: While doing some arm-none-eabi testing, I noticed that a bunch of gcc.dg/vect tests were causing the target to hang from trying to execute code compiled with -mfpu=neon -mfloat-abi=softfp, on a target that doesn't support those instructions

[patch, nios2] support for custom round instructions

2014-04-22 Thread Sandra Loosemore
restrictions requiring -ffinite-math-only, etc., so that part was also just following current practice and building on existing framework, just adding a new flag. I've checked this in. -Sandra 2014-04-22 Sandra Loosemore san...@codesourcery.com gcc/ * config/nios2/nios2.md (UNSPEC_ROUND

[patch, toplevel] configure nios2-elf libraries to build with -mno-gpopt

2014-04-24 Thread Sandra Loosemore
-specific, needs to be approved by a global reviewer and then propagated to the sourceware.org binutils-gdb and newlib repositories as well. So, OK to commit? -Sandra 2014-04-24 Sandra Loosemore san...@codesourcery.com * configure.ac (target_makefile_frag): Set for nios2-*-elf

Ping [patch, toplevel] configure nios2-elf libraries to build with -mno-gpopt

2014-05-05 Thread Sandra Loosemore
Ping! http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01618.html -Sandra

[patch, mips] fix JALR test in configure.ac

2014-05-12 Thread Sandra Loosemore
the configure test to pass. OK to commit? -Sandra 2014-05-12 Catherine Moore c...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * configure.ac: Fix assembly for explicit JALR relocation check. * configure: Regenerate. Index: gcc/configure.ac

[patch, mips] small -mmicromips testsuite cleanup

2014-05-12 Thread Sandra Loosemore
The test case gcc.target/mips/loongson-simd.c fails when the multilib options include -mmicromips. This patch fixes it analogously to how this test case already ignores MIPS16ness. OK to commit? -Sandra 2014-05-12 Nathan Sidwell nat...@codesourcery.com Sandra Loosemore san

Ping^2 [patch, toplevel] configure nios2-elf libraries to build with -mno-gpopt

2014-05-13 Thread Sandra Loosemore
On 05/05/2014 02:32 PM, Sandra Loosemore wrote: Ping! http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01618.html And ping again -Sandra

[patch,mips] avoid invalid register for JALR

2014-05-13 Thread Sandra Loosemore
up testing this patch on a more stable 4.9.0 checkout modified to support Mentor's extended set of mips-sde-elf multilibs instead. OK to commit? -Sandra 2014-05-13 Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.h (enum reg_class): Add JALR_REGS

[patch, mips] delete bit-rotten ADJUST_REG_ALLOC_ORDER definition

2014-05-13 Thread Sandra Loosemore
Catherine Moore c...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.c (mips_order_regs_for_local_alloc): Delete. * config/mips/mips.h (ADJUST_REG_ALLOC_ORDER): Delete. * config/mips/mips-protos.h

Re: [patch,mips] avoid invalid register for JALR

2014-05-13 Thread Sandra Loosemore
On 05/13/2014 03:41 PM, Richard Sandiford wrote: Sandra Loosemore san...@codesourcery.com writes: When I was trying to benchmark another patch (which I'll be sending along shortly) with CSiBE for -mabi=64, I ran into an assembler error like this: /tmp/ccJv2faG.s: Assembler messages: /tmp

Re: [patch, mips] delete bit-rotten ADJUST_REG_ALLOC_ORDER definition

2014-05-14 Thread Sandra Loosemore
On 05/14/2014 12:49 PM, Richard Sandiford wrote: Jeff Law l...@redhat.com writes: On 05/13/14 14:11, Sandra Loosemore wrote: 2014-05-13 Catherine Moore c...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.c

[patch, nios2] unbreak nios2 build

2014-05-14 Thread Sandra Loosemore
that was not there before; previously the failure mode was to quietly generate long conditional branches everywhere instead of an ICE. -Sandra 2014-05-14 Sandra Loosemore san...@codesourcery.com gcc/ * config/nios2/nios2.md (nios2_cbranch): Fix paste-o in length attribute

[patch, libstdc++] fix TCL error in abi.exp

2014-05-16 Thread Sandra Loosemore
of before. OK to commit? -Sandra 2014-05-16 Iain Sandoe i...@codesourcery.com Sandra Loosemore san...@codesourcery.com libstdc++-v3/ * testsuite/libstdc++-abi/abi.exp: Defer setting of baseline_subdir until after checking that the test is eligible to be run

[patch ping] libstdc++ testsuite cxxflags

2014-05-16 Thread Sandra Loosemore
It appears that this patch from last fall never got reviewed. https://gcc.gnu.org/ml/gcc-patches/2013-10/msg02340.html Can someone take a look? I'll commit the patch on Cesar's behalf if it's approved. -Sandra

Re: Eliminate write-only variables

2014-05-18 Thread Sandra Loosemore
On 05/16/2014 11:25 AM, Jan Hubicka wrote: Hi, this patch adds code to remove write only static variables. While analyzing effectivity of LTO on firefox, I noticed that surprisingly large part of binary's data segment is occupied by these. Fixed thus. (this is quite trivial transformation, I

Re: Eliminate write-only variables

2014-05-18 Thread Sandra Loosemore
On 05/18/2014 02:59 PM, Jan Hubicka wrote: Sandra, This patch seems quite similar in purpose to the remove_local_statics optimization that Mentor has proposed, although the implementation is quite different. Here is the last version of our patch, prepared by Bernd Schmidt last year:

[patch, mips, tree] align microMIPS functions to 16 bits with -Os

2014-05-19 Thread Sandra Loosemore
using Mentor's usual assortment of multilibs, specifically including one for microMIPS. -Sandra 2014-05-19 Iain Sandoe i...@codesourcery.com Catherine Moore c...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.c

Re: [patch ping] libstdc++ testsuite cxxflags

2014-05-19 Thread Sandra Loosemore
On 05/17/2014 04:07 AM, Jonathan Wakely wrote: On 17 May 2014 10:50, Jonathan Wakely wrote: On 17 May 2014 01:16, Sandra Loosemore wrote: It appears that this patch from last fall never got reviewed. https://gcc.gnu.org/ml/gcc-patches/2013-10/msg02340.html Can someone take a look? I'll

Re: Eliminate write-only variables

2014-05-19 Thread Sandra Loosemore
On 05/18/2014 08:45 PM, Sandra Loosemore wrote: On 05/18/2014 02:59 PM, Jan Hubicka wrote: For cases like local-statics-7 your approach can be saved by adding simple IPA analysis to look for static vars that are used only by one function and keeping your DSE code active for them, so we can

[patch, lto] add testcase for PR60179

2014-05-20 Thread Sandra Loosemore
. Is this OK, or is there a better way to do it? -Sandra 2014-05-20 Cesar Philippidis ce...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/testsuite/ * lib/scanasm.exp (scan-lto-assembler): New procedure. * gcc.target/nios2/custom-fp-lto.c: New test

Re: [patch, mips, tree] align microMIPS functions to 16 bits with -Os

2014-05-28 Thread Sandra Loosemore
On 05/19/2014 01:38 PM, Sandra Loosemore wrote: 2014-05-19 Iain Sandoe i...@codesourcery.com Catherine Moore c...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.c (mips_set_current_function): Choose function alignment once

[patch, arm] fix gcc.target/arm/pr45094.c options

2014-05-30 Thread Sandra Loosemore
results in BE8 code rather than BE32.) It seems simplest just to remove the specific -mcpu option and rely on the multilib options to supply appropriate test flags for the execution environment. OK to commit? -Sandra 2014-05-30 Julian Brown jul...@codesourcery.com Sandra

Re: [patch, mips, tree] align microMIPS functions to 16 bits with -Os

2014-05-30 Thread Sandra Loosemore
On 05/28/2014 01:09 PM, Richard Sandiford wrote: Sandra Loosemore san...@codesourcery.com writes: On 05/19/2014 01:38 PM, Sandra Loosemore wrote: 2014-05-19 Iain Sandoe i...@codesourcery.com Catherine Moore c...@codesourcery.com Sandra Loosemore san

Re: Eliminate write-only variables

2014-05-31 Thread Sandra Loosemore
On 05/20/2014 04:56 AM, Richard Biener wrote: On Tue, May 20, 2014 at 6:29 AM, Jan Hubicka hubi...@ucw.cz wrote: On 05/18/2014 08:45 PM, Sandra Loosemore wrote: On 05/18/2014 02:59 PM, Jan Hubicka wrote: For cases like local-statics-7 your approach can be saved by adding simple IPA analysis

Re: [patch, mips, tree] align microMIPS functions to 16 bits with -Os

2014-06-04 Thread Sandra Loosemore
On 06/04/2014 06:20 AM, Richard Biener wrote: On Tue, 3 Jun 2014, Richard Sandiford wrote: Richi suggested just changing the alignment at output time. I assume that would be a case of replacing the DECL_ALIGN in: /* Tell assembler to move to target machine's alignment for functions. */

Re: [patch, mips, tree] align microMIPS functions to 16 bits with -Os

2014-06-05 Thread Sandra Loosemore
On 06/05/2014 01:39 AM, Richard Biener wrote: [snip] Ok, we definitely need to preserve that (documented) behavior. I suppose it also sets DECL_USER_ALIGN. -falign-functions is probably another setter of DECL_ALIGN here. If we add a target hook that may adjust function alignment then it has

Re: [patch, mips, tree] align microMIPS functions to 16 bits with -Os

2014-06-05 Thread Sandra Loosemore
On 06/05/2014 03:50 PM, Richard Sandiford wrote: Sandra Loosemore san...@codesourcery.com writes: On 06/05/2014 01:39 AM, Richard Biener wrote: [snip] Ok, we definitely need to preserve that (documented) behavior. I suppose it also sets DECL_USER_ALIGN. -falign-functions is probably

Re: [patch, mips, tree] align microMIPS functions to 16 bits with -Os

2014-06-07 Thread Sandra Loosemore
On 06/06/2014 01:44 AM, Richard Biener wrote: On Fri, 6 Jun 2014, Richard Sandiford wrote: [snip] How about initialising the DECL_ALIGN to: (TARGET_PTRMEMFUNC_VBIT_LOCATION == ptrmemfunc_vbit_in_pfn ? 2 * BITS_PER_UNIT : BITS_PER_UNIT) instead of function_boundary (NULL)? That

[patch, committed] Nios II GOT pointer initialization

2014-02-02 Thread Sandra Loosemore
with old versions of the linker, but we'd rather have the first official release of GCC for Nios II implement this correctly. -Sandra 2014-02-02 Sandra Loosemore san...@codesourcery.com gcc/ * config/nios2/nios2.md (load_got_register): Initialize GOT pointer from _gp_got instead

Re: [PATCH] Fix libjava install with --enable-version-specific-runtime-libs

2014-02-19 Thread Sandra Loosemore
On 02/19/2014 02:43 AM, Andrew Haley wrote: On 02/19/2014 09:34 AM, Richard Biener wrote: Sandras patch was supposed to introduce support for --enable-version-specific-runtime-libs in libgcj (but obviously it failed, given the result above). Sandra? You're very quiet. What say you? I don't

[patch, ARM] Add support for crtfastmath.o

2014-11-08 Thread Sandra Loosemore
that the constructor is suppressed for soft-float targets, that it builds successfully for Thumb-2 with VFP, etc. I also regression-tested the gcc testsuite on an arm-none-linux-gnueabi build configured for -march=armv8-a with VFP floating point enabled. OK to commit? -Sandra 2014-11-08 Sandra

[RFC, C] add warning for unpromoted bit-field uses

2014-11-08 Thread Sandra Loosemore
testing on it yet, but I thought I'd put it out there for comment first to see if there is consensus that this is a reasonable thing to do. Maybe details like the warning option name and message could be improved, etc, too. Any other feedback? -Sandra 2014-11-08 Sandra Loosemore san

Re: [RFC, C] add warning for unpromoted bit-field uses

2014-11-09 Thread Sandra Loosemore
On 11/09/2014 02:28 AM, Manuel López-Ibáñez wrote: We had a request from a customer to add a warning to the C front end to diagnose cases where bit-fields larger than an int are used in shift expressions; confusingly, the operation is done in the precision of the bit-field size rather than its

Re: [doc] extend.texi copy-editing, 3/N (hyphenated phrases)

2012-12-19 Thread Sandra Loosemore
On 12/18/2012 10:42 PM, Gerald Pfeifer wrote: Hi Sandra, On Sat, 10 Nov 2012, Sandra Loosemore wrote: 2012-11-10 Sandra Loosemoresan...@codesourcery.com gcc/ * doc/extend.texi: Copy-edit to fix incorrect hyphenation phrases involving bit, byte, word, precision

[patch, committed] invoke.texi: clean up texinfo markup

2012-04-06 Thread Sandra Loosemore
the generated PDF manual. -Sandra 2012-04-06 Sandra Loosemore san...@codesourcery.com gcc/ * doc/invoke.texi: Clean up Texinfo markup throughout the file. Use @option markup on command-line options. Use @samp markup on literal keywords to options. Use @code markup

Re: [patch, committed] invoke.texi: clean up texinfo markup

2012-05-03 Thread Sandra Loosemore
On 05/02/2012 12:41 AM, Gerald Pfeifer wrote: Hi Sandra, On Fri, 6 Apr 2012, Sandra Loosemore wrote: This is another installment in my series of cleanups to invoke.texi. In this patch I have taken a break from nit-picking grammar and have nit-picked some Texinfo markup issues instead

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

2012-07-23 Thread Sandra Loosemore
for this problem in that backend. Nick, you're listed as mcore port maintainer; can you help? -Sandra 2012-07-23 Sandra Loosemore san...@codesourcery.com Paul Brook p...@codesourcery.com PR target/53633 gcc/ * target.def (warn_func_return): New hook. * doc

Re: [PATCH] Detect loops in find_comparison_args

2012-07-24 Thread Sandra Loosemore
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 circularity, rather than Andrew's solution of using a

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

2012-07-24 Thread Sandra Loosemore
as well as the ARM port. Something like this? The code part of the patch is unchanged from the last version I posted. OK to check in? -Sandra 2012-07-24 Sandra Loosemore san...@codesourcery.com Paul Brook p...@codesourcery.com PR target/53633 gcc

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

2012-07-25 Thread Sandra Loosemore
On 07/25/2012 09:57 AM, Richard Henderson wrote: I'll echo Nick's comments about arm asm in a common test. There's no need to have anything but __asm__(); there. Ok with that change. Thanks! Here's the version I committed. -Sandra 2012-07-25 Sandra Loosemore san...@codesourcery.com

Re: [PING] Re: [RFC, ivopts] fix bugs in ivopts address cost computation

2012-07-25 Thread Sandra Loosemore
On 07/17/2012 05:22 AM, Richard Guenther wrote: On Wed, Jul 4, 2012 at 6:35 PM, Sandra Loosemore san...@codesourcery.com wrote: Ping? Original post with patch is here: http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00319.html Can you update the patch and numbers based on what Bill did

Re: [PATCH] Detect loops in find_comparison_args

2012-07-25 Thread Sandra Loosemore
on x86_64. OK to check in? -Sandra 2012-07-25 Andrew Jenner and...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * cse.c (find_comparison_args): Check for cycles of any length. gcc/testsuite/ * gcc.c-torture/compile/pr50380.c: Add

Re: [PATCH] Detect loops in find_comparison_args

2012-07-26 Thread Sandra Loosemore
On 07/26/2012 01:28 AM, Paolo Bonzini wrote: Il 26/07/2012 04:25, Sandra Loosemore ha scritto: On 07/25/2012 01:27 AM, Paolo Bonzini wrote: What I'm worried about is the extra cost of malloc-ing and free-ing the pointer set. Perhaps you can skip the pointer set creation in the common case

[patch, libiberty] avoid closing files twice on Windows when exec fails

2012-07-26 Thread Sandra Loosemore
as it goes, and is at least an incremental improvement in robustness. OK for mainline? -Sandra 2012-07-26 Kazu Hirata k...@codesourcery.com Sandra Loosemore san...@codesourcery.com libiberty/ * pex-win32.c (pex_win32_exec_child): Only close original file

ping ancient MIPS: missed optimization patch

2012-07-27 Thread Sandra Loosemore
Richard, This ancient patch to tweak mips_legitimize_address http://gcc.gnu.org/ml/gcc/2008-11/msg00294.html seems to never have been applied. Do you have any idea whether this is still a useful change? The test case given in the first message in that thread no longer reproduces with a

[PATCH, MIPS] -mno-float odds and ends

2012-07-30 Thread Sandra Loosemore
override architecture defaults. This patch addresses both problems. OK to commit? -Sandra 2012-07-30 Sandra Loosemore san...@codesourcery.com Julian Brown jul...@codesourcery.com gcc/ * doc/invoke.texi (MIPS Options): Document -mno-float. * config/mips

Re: [PATCH, MIPS] -mno-float odds and ends

2012-07-30 Thread Sandra Loosemore
review! :-) -Sandra 2012-07-30 Sandra Loosemore san...@codesourcery.com Julian Brown jul...@codesourcery.com gcc/ * doc/invoke.texi (MIPS Options): Document -mno-float. * config/mips/mips.h (MIPS_ARCH_FLOAT_SPEC): Make it know about -mno-float. Index

Re: [PATCH, MIPS] -mno-float odds and ends

2012-07-30 Thread Sandra Loosemore
review! :-) -Sandra 2012-07-30 Sandra Loosemore san...@codesourcery.com Julian Brown jul...@codesourcery.com gcc/ * doc/invoke.texi (MIPS Options): Document -mno-float. * config/mips/mips.h (MIPS_ARCH_FLOAT_SPEC): Make it know about -mno-float. Index

[PATCH, MIPS] 74k madd scheduler tweaks

2012-08-01 Thread Sandra Loosemore
to contribute this to the FSF; ok for mainline? -Sandra 2012-08-01 Sandra Loosemore san...@codesourcery.com Maxim Kuvyrkov ma...@codesourcery.com Julian Brown jul...@codesourcery.com MIPS Technologies, Inc. * config/mips/74k.md (r74k_int_mult

[PATCH, MIPS] clean up 24k/74k store bypasses

2012-08-02 Thread Sandra Loosemore
This patch changes the 24k/74k scheduling descriptions to use the existing mips_store_data_bypass_p predicate instead of treating cprestore as a special case. OK for mainline? -Sandra 2012-08-02 Sandra Loosemore san...@codesourcery.com Maxim Kuvyrkov ma...@codesourcery.com

[PATCH, MIPS] DSP ALU scheduling

2012-08-02 Thread Sandra Loosemore
posted the other day: http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00062.html Assuming that one gets approved, is this patch OK for mainline? -Sandra 2012-08-02 Sandra Loosemore san...@codesourcery.com Maxim Kuvyrkov ma...@codesourcery.com Julian Brown jul

[PATCH, MIPS] diagnose -fpic/-fpie incompatibility with -mno-abicalls

2012-08-03 Thread Sandra Loosemore
that triggered the new error. OK for mainline? -Sandra 2012-08-03 Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.c (mips_option_override): Check -fpic/-fpie and -mabicalls options for compatibility with ABI. gcc/testsuite/ * g++.dg/opt

[PATCH, MIPS] fix clear cache test cases

2012-08-03 Thread Sandra Loosemore
Some MIPS configurations #define CACHE_FLUSH_FUNC to something other than _flush_cache, This fixes the two test cases that currently have that name wired in to know about the netbsd and sdemtk names too. OK for mainline? -Sandra 2012-08-03 Sandra Loosemore san...@codesourcery.com

[PATCH, MIPS] Netlogic XLR tuning tweaks

2012-08-03 Thread Sandra Loosemore
in our local source base for some time. OK for mainline? -Sandra 2012-08-03 Catherine Moore c...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/xlr.md (ir_xlr_alu_clz): New insn_reservation. (ir_xlr_alu): Remove clz

Re: [PATCH, MIPS] 74k madd scheduler tweaks

2012-08-04 Thread Sandra Loosemore
On 08/04/2012 07:48 AM, Richard Sandiford wrote: Sandra Loosemoresan...@codesourcery.com writes: The existing scheduler bypass information for madd on the 74k uses some bits copied from the 24k, and is not quite correct. This patch is based on one originally sent to us by MIPS and has been

Re: [PATCH, MIPS] Add 34Kn cpu

2012-08-04 Thread Sandra Loosemore
On 08/04/2012 03:01 PM, Gerald Pfeifer wrote: On Wed, 1 Aug 2012, Richard Sandiford wrote: 2012-08-01 Catherine Moorec...@codesourcery.com Sandra Loosemoresan...@codesourcery.com gcc/ * config/mips/mips-cpus.def (34kn): New. * config/mips/mips.h

Re: [PATCH, MIPS] diagnose -fpic/-fpie incompatibility with -mno-abicalls

2012-08-04 Thread Sandra Loosemore
should say anyway. I've committed the attached version. -Sandra 2012-08-04 Sandra Loosemore san...@codesourcery.com Richard Sandiford rdsandif...@googlemail.com gcc/ * config/mips/mips.c (mips_option_override): Check -fpic for compatibility

[PATCH] put exception tables for comdat functions in comdat, too

2012-08-05 Thread Sandra Loosemore
. This patch has also been present in our local source base for a while so it's been well-tested in that context. OK for mainline? -Sandra 2012-08-04 Paul Brook p...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * except.c (switch_to_exception_section

Re: [PATCH] put exception tables for comdat functions in comdat, too

2012-08-06 Thread Sandra Loosemore
On 08/06/2012 11:45 AM, Richard Henderson wrote: On 08/05/2012 08:40 PM, Sandra Loosemore wrote: 2012-08-04 Paul Brookp...@codesourcery.com Sandra Loosemoresan...@codesourcery.com gcc/ * except.c (switch_to_exception_section): Place tables for DECL_ONE_ONLY functions

[PATCH, MIPS] fix MIPS16 hard-float function stub bugs

2012-08-07 Thread Sandra Loosemore
This patch fixes a group of bugs that were causing link errors on hard-float MIPS16 code built with a mips-linux-gnu toolchain. This is Mark Mitchell's analysis of the original problem: The MIPS16 instruction set cannot directly access hard-float registers, so helper functions in libgcc are

Re: [PATCH, MIPS] fix MIPS16 hard-float function stub bugs

2012-08-09 Thread Sandra Loosemore
On 08/08/2012 03:07 AM, Richard Sandiford wrote: It looks like this patch might have been written before: http://gcc.gnu.org/ml/gcc-patches/2012-01/msg00756.html which added: /* If we're calling a locally-defined MIPS16 function, we know that it will return values in both the

Re: [PATCH, MIPS] DSP ALU scheduling

2012-08-15 Thread Sandra Loosemore
to remove the uses of mips_mult_madd_chain_bypass_p. -Sandra 2012-08-15 Sandra Loosemore san...@codesourcery.com Maxim Kuvyrkov ma...@codesourcery.com Julian Brown jul...@codesourcery.com MIPS Technologies, Inc. gcc/ * config/mips/mips.md

Re: [PATCH, MIPS] DSP ALU scheduling

2012-08-16 Thread Sandra Loosemore
)]) (define_insn mips_dpau_h_qbr [etc] I think all these want (set_attr accum_in 1) too. Eeek, you are right. I have checked in the obvious patch to correct this. -Sandra 2012-08-16 Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips-dsp.md (mips_dpau_h_qbl

[PATCH, MIPS] fix MIPS16 jump table overflow

2012-08-20 Thread Sandra Loosemore
. Is the rest of the patch OK to check in? -Sandra 2012-08-20 Julian Brown jul...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.md (MIPS16_T_REGNUM): New constant. (tablejump): Don't use for MIPS16_SHORT_JUMP_TABLES

[PATCH] fix wrong-code bug for -fstrict-volatile-bitfields

2012-08-21 Thread Sandra Loosemore
Brook p...@codesourcery.com Joseph Myers jos...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * expr.h (store_bit_field): Add packedp parameter to prototype. * expmed.c (store_bit_field, store_bit_field_1): Add packedp

Re: [PATCH, MIPS] add new peephole for 74k dspr2

2012-08-22 Thread Sandra Loosemore
. That would also allow us to use it for plain HI and LO. It wasn't obvious from the patch why it was restricted to the DSP extension registers. Please also add a scan-assembler test. How is this version of the fix? -Sandra 2012-08-22 Sandra Loosemore san...@codesourcery.com gcc

Re: [PATCH, MIPS] fix MIPS16 jump table overflow

2012-08-22 Thread Sandra Loosemore
. Is this part OK, at least? It passes regression testing. -Sandra 2012-08-22 Julian Brown jul...@codesourcery.com Sandra Loosemore san...@codesourcery.com gcc/ * config/mips/mips.md (UNSPEC_CASESI_DISPATCH): New. (MIPS16_T_REGNUM): New constant

Re: [PATCH] fix wrong-code bug for -fstrict-volatile-bitfields

2012-08-22 Thread Sandra Loosemore
On 08/22/2012 03:27 PM, Eric Botcazou wrote: + bool packedp = false; + + if (TREE_CODE(to) == COMPONENT_REF + (TYPE_PACKED (TREE_TYPE (TREE_OPERAND (to, 0))) + || (TREE_CODE (TREE_OPERAND (to, 1)) == FIELD_DECL + DECL_PACKED (TREE_OPERAND (to, 1)

Re: [PATCH] fix wrong-code bug for -fstrict-volatile-bitfields

2012-08-23 Thread Sandra Loosemore
On 08/23/2012 03:08 AM, Richard Guenther wrote: In fact, you should probably implement code-generation constraints from within the frontends by, for strict volatile bitfields, emitting loads/stores using DECL_BIT_FIELD_REPRESENTATIVE (doing read-modify-write explicitely). Or maybe you can

Re: [PATCH, MIPS] fix MIPS16 jump table overflow

2012-08-25 Thread Sandra Loosemore
On 08/24/2012 11:46 PM, Richard Sandiford wrote: Andrew Pinskipins...@gmail.com writes: On Fri, Aug 24, 2012 at 10:08 PM, Andrew Pinskipins...@gmail.com wrote: On Wed, Aug 22, 2012 at 7:15 PM, Sandra Loosemore san...@codesourcery.com wrote: On 08/21/2012 02:23 PM, Richard Sandiford wrote

another wrong-code problem with -fstrict-volatile-bitfields

2012-08-25 Thread Sandra Loosemore
While I was grovelling around trying to swap in more state on the bitfield store/extract code for the patch rewrite being discussed here: http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01546.html I found a reference to PR23623 and found that it is broken again, but in a different way. On ARM

Re: another wrong-code problem with -fstrict-volatile-bitfields

2012-08-26 Thread Sandra Loosemore
the previously-failing test case and regression tests OK on arm-none-eabi, but I haven't tried it on any other target yet. Is this a reasonable way to resolve this conflict, or should something farther up the call chain take care of it? -Sandra 2012-08-26 Sandra Loosemore san

PING^2 Re: PATCH: fix collect2 handling of --demangle and --no-demangle

2011-07-11 Thread Sandra Loosemore
(2) I fixed the COLLECT_NO_DEMANGLE environment variable setting recipe. (3) I simplified the argument processing for --demangle and --no-demangle to pass them straight through to the linker when HAVE_LD_DEMANGLE is defined. OK to commit? -Sandra 2011-06-17 Sandra Loosemore san

PATCH: fix think-o in genrecog.c

2011-07-18 Thread Sandra Loosemore
the generated insn-recog.c for a mips-sde-elf build against one I got using a pre-r174305 genrecog.c. -Sandra 2011-07-18 Sandra Loosemore san...@codesourcery.com gcc/ * genrecog.c (make_insn_sequence): Correct position numbering when filtering out match_scratch

[RFC, ivopts] fix bugs in ivopts address cost computation

2012-06-05 Thread Sandra Loosemore
me if other back ends could benefit from some tweaking as well, depending on the extent to which they're affected by the bugs I listed above. Comments, complaints, proposals for alternate fixes, etc? Or OK to commit? -Sandra 2012-06-05 Sandra Loosemore san...@codesourcery.com gcc

Re: [RFC, ivopts] fix bugs in ivopts address cost computation

2012-06-13 Thread Sandra Loosemore
On 06/06/2012 02:29 AM, Richard Guenther wrote: Pre-computing and caching things is to avoid creating RTXen over and over. As you have discarded this completely did you try to measure the cost of doing so in terms of produced garbage and compile-time cost? Did you consider changing the target

[PING] Re: [RFC, ivopts] fix bugs in ivopts address cost computation

2012-07-04 Thread Sandra Loosemore
On 06/05/2012 10:34 AM, Sandra Loosemore wrote: 2012-06-05 Sandra Loosemoresan...@codesourcery.com gcc/ * tree-ssa-loop-ivopts.c (comp_cost): Make complexity field signed. Update comments to indicate this is for addressing mode complexity. (new_cost): Make

[committed] avoid filename collision in libgomp html manual

2012-07-04 Thread Sandra Loosemore
. -Sandra 2012-07-04 Sandra Loosemore san...@codesourcery.com libgomp/ * libgomp.texi (Library Index): Renamed from Index to prevent conflict with index.html on case-insensitive file systems. Index: libgomp/libgomp.texi

PATCH: fix typos in SH_ASM_SPEC

2012-07-14 Thread Sandra Loosemore
Sandra Loosemore san...@codesourcery.com gcc/ * config/sh/sh.h (SH_ASM_SPEC): Add spaces between options. Index: gcc/config/sh/sh.h === --- gcc/config/sh/sh.h (revision 189485) +++ gcc/config/sh/sh.h (working copy

[committed] restrict gcc.target/m68k/pr36134.c to ColdFire

2012-07-17 Thread Sandra Loosemore
I've checked in this patch, which was conditionally approved 3+ years ago: http://gcc.gnu.org/ml/gcc-patches/2009-05/msg00688.html I did look at general usage for m68k and adding -mcpu is consistent with other existing tests; there aren't enough of them that are conditionalized in this way to

[PATCH, testsuite] Skip 20101011-1.c for bare-metal m68k

2012-07-17 Thread Sandra Loosemore
Like the subject line says; this is consistent with the existing test to bail out for MIPS bare-metal. OK for mainline? -Sandra 2012-07-17 Julian Brown jul...@codesourcery.com Sandra Loosemore san...@codesroucery.com gcc/testsuite/ * gcc.c-torture/execute

[patch, committed] invoke.texi: clean up n-bit/byte/word modifiers

2012-02-15 Thread Sandra Loosemore
similar issues with n-byte or n-word modifiers. I've checked this in as obvious. -Sandra 2012-02-15 Sandra Loosemore san...@codesourcery.com gcc/ * doc/invoke.texi: Clean up n-bit/byte/word modifiers. Index: gcc/doc/invoke.texi

[patch, committed] invoke.texi: big-endian, little-endian

2012-02-17 Thread Sandra Loosemore
did not have to deal with that case. ;-) -Sandra 2012-02-17 Sandra Loosemore san...@codesourcery.com gcc/ * doc/invoke.texi: Consistently hyphenate big-endian/little-endian when used as adjectives. Index: gcc/doc/invoke.texi

[patch, committed] invoke.texi: GCC coding conventions issues

2012-02-17 Thread Sandra Loosemore
/uppercase -Sandra 2012-02-17 Sandra Loosemore san...@codesourcery.com gcc/ * doc/invoke.texi: Minor copy-edits to bring into conformance with GCC coding conventions. Index: gcc/doc/invoke.texi === --- gcc/doc

[patch, committed] invoke.texi: copy-edit docs for IRA options and parameters

2012-02-18 Thread Sandra Loosemore
it's not strictly obvious, but it's intended to be a content-free change in terms of meaning, and it's at least an incremental improvement in readability compared to what's there now. -Sandra 2012-02-18 Sandra Loosemore san...@codesourcery.com gcc/ * doc/invoke.texi (-fira

[patch, committed] invoke.texi: copy-edit x86 sections

2012-03-04 Thread Sandra Loosemore
ways. -Sandra 2012-03-04 Sandra Loosemore san...@codesourcery.com gcc/ * doc/invoke.texi (C++ Dialect Options): Minor copy-edits to x86-specific text. (Debugging Options): Likewise. (Optimize Options): Likewise. (i386 and x86-64 Options

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

2012-03-09 Thread Sandra Loosemore
. Similarly, I've fixed up a couple instances of incorrect usage of stabs (official name seems to be all lowercase, like that) and ELF. Checked in to trunk as obvious. -Sandra 2012-03-09 Sandra Loosemore san...@codesourcery.com gcc/ * doc/invoke.texi: Use correct names

[patch, committed] invoke.texi: use correct name/markup for GCC and other program names

2012-03-09 Thread Sandra Loosemore
was at it, I checked references to GDB and ld. For the latter, the GNU ld manual seems to prefer @command markup in all contexts, so that's what I did here. Checked in to trunk as obvious. -Sandra 2012-03-09 Sandra Loosemore san...@codesourcery.com gcc/ * doc/invoke.texi: Use

[patch, RFA] delete obsolete -madjust-unroll option for SH

2012-03-09 Thread Sandra Loosemore
not delete this entirely, per the attached patch? I haven't built/tested this (except for the manual) but can do that if the patch isn't sufficiently obvious. -Sandra 2012-03-09 Sandra Loosemore san...@codesourcery.com gcc/ * config/sh/sh.opt (madjust-unroll): Delete

[patch, RFA] -no-integrated-cpp documentation

2012-03-11 Thread Sandra Loosemore
, and it's probably more trouble to remove than it's worth Anyway, I'd appreciate another pair of eyes looking at this, and suggestions on what better to do here if this rewrite isn't adequate. -Sandra 2012-03-11 Sandra Loosemore san...@codesourcery.com gcc/ * doc

PATCH: fix infinite loop in CSE

2011-12-12 Thread Sandra Loosemore
of iterations to be sure the outer loop always terminates. I regression-tested this in a GCC 4.5-based build for mips-linux-gnu and also did a full bootstrap and regression test for i686-pc-linux-gnu on mainline head. OK to check in on mainline? -Sandra 2011-12-12 Sandra Loosemore san

  1   2   3   4   5   6   7   8   9   10   >