Re: [C/C++ PATCH] Implement -Wshift-negative-value (PR c/65179)

2015-05-08 Thread Steve Ellcey
__time_t; typedef __time_t time_t; static time_t const time_t_min = time_t) -1) 0) ? (time_t) -1 (8 * sizeof (time_t) - 1) : 0) Steve Ellcey sell...@imgtec.com

Question about patch for PR bootstrap/65150 (identical functions)

2015-05-08 Thread Steve Ellcey
anything. Steve Ellcey sell...@imgtec.com

RFC: Patch to align spills beyond what the stack supports

2015-05-07 Thread Steve Ellcey
be approved for checkin with some further work? Steve Ellcey sell...@imgtec.com 2015-05-07 Steve Ellcey sell...@imgtec.com * Makefile.in (OBJS): Add tree-align-spills. * common.opt (Wunaligned-spills): New. (falign-spills): New. * params.def

[Patch, MIPS] Change mips4 default processor to r10K

2015-04-28 Thread Steve Ellcey
the r10k instruction scheduler instead of the generic MIPS scheduler (there is no r8000 specific scheduler) and that should be a good thing for most mips4 machines. Tested with the mips-linux-gnu toolchain. OK for checkin? Steve Ellcey sell...@imgtec.com 2015-04-28 Steve Ellcey sell

[Patch, MIPS] Minor cleanup in mips.md

2015-04-23 Thread Steve Ellcey
-gnu to make sure everything built and ran the GCC testsuite with the 74k scheduler to verify that there were no regressions but this change would have no affect on any MIPS target. OK for checkin? Steve Ellcey sell...@imgtec.com 2015-04-23 Steve Ellcey sell...@imgtec.com * config

Re: [RFC] Dynamically aligning the stack

2015-04-21 Thread Steve Ellcey
tell GCC what alignment you want for the function? Or does GCC figure that out for itself based on the instructions and data types it sees in the function? Steve Ellcey sell...@imgtec.com

Re: [PATCH] -Warray-bounds TLC

2015-04-17 Thread Steve Ellcey
. But I do not get this error in baz (where we don't check for -1. In reality, in glibc, we know that i can only be 0, -1, or -2. GCC of course doesn't know that. Does this error/warning seem right? The difference in behavior between bar and baz seems odd. Steve Ellcey sell...@imgtec.com

Re: [PATCH] -Warray-bounds TLC

2015-04-17 Thread Steve Ellcey
} (2 elements) Found new range for i_10: ~[-2, -2] It has to pick one of the 2 anti-ranges, which one it picks is pretty arbitrary. It probably warns if you swap the tests for 0 and -2. You are right, If I swap the tests then I do get the warning. Steve Ellcey sell...@imgtec.com

Re: [PATCH] -Warray-bounds TLC

2015-04-17 Thread Steve Ellcey
is declared as 'noreturn'. Steve Ellcey sell...@imgtec.com extern void bad (const char *__assertion) __attribute__ ((__noreturn__)); struct link_map { long int l_ns; }; extern struct link_namespaces { unsigned int _ns_nloaded; } _dl_ns[1]; void _dl_close_worker (struct link_map *map) { long int

Re: [PATCH] -Warray-bounds TLC

2015-04-17 Thread Steve Ellcey
that line, the line with the call to the noreturn function, and the ns-ns_loaded line (like there is in the real glibc), it is very hard to understand what the compiler is trying to tell me when it only points out the first line as where the error is. Steve Ellcey

Re: [RFC] Dynamically aligning the stack

2015-04-14 Thread Steve Ellcey
). That seems like to be fragile in the long run. jeff Yes, I am trying to look at how the x86 does dynamic stack alignment but it is difficult to untangle the generic concepts from the parts tied specifically to the x86 calling convention. No other platform appears to do dynamic stack alignment. Steve

[RFC] Dynamically aligning the stack

2015-03-26 Thread Steve Ellcey
the stack? It seems a lot simpler and more target independent than what x86 is doing. Steve Ellcey sell...@imgtec.com A pass that inserts __builtin_alloca(8) at front of all routines: unsigned int pass_realign_stack::execute (function *fun) { basic_block bb; gimple g; tree size

Re: [Patch] PR 65315 - Fix alignment of local variables

2015-03-05 Thread Steve Ellcey
alignment is used to align the dynamically allocated memory instead of 'c' which has a greater alignment requirement. Steve Ellcey sell...@imgtec.com int foo(int *x) { int i,y; int a[40]; int b[50] __attribute__ ((aligned(32))); int c[40] __attribute__ ((aligned(128

[Patch] PR 65315 - Fix alignment of local variables

2015-03-04 Thread Steve Ellcey
how to create a generic test case, I was checking the alignment on MIPS by hand by looking for the shift-right/shift-left instructions that create an aligned pointer but that obviously doesn't work on other architectures. Steve Ellcey sell...@imgtec.com 2015-03-04 Steve Ellcey sell...@imgtec.com

[Patch, MIPS] Fix PR 58158 - ICE on MIPS Loongson

2015-03-02 Thread Steve Ellcey
to do a complete test run on. The change seems very safe though and it should be OK for R6 as well as Loongson since '!ISA_HAS_FP_CONDMOVE' is true for R6 so using it instead of 'ISA_HAS_SEL' should be OK and more descriptive of why we want to fail in this case. OK to checkin? Steve Ellcey sell

RE: [Patch, MIPS] Fix PR 58158 - ICE on MIPS Loongson

2015-03-02 Thread Steve Ellcey
conditional move based on an FP condition then that's fine. 2015-03-02 Steve Ellcey sell...@imgtec.com PR target/58158 * config/mips/mips.md (movmodecc): Change ISA_HAS_SEL check to !ISA_HAS_FP_CONDMOVE. OK. Are you planning on doing any backports of this patch

Re: ipa-icf::merge TLC

2015-02-27 Thread Steve Ellcey
building a toolchain for mips-mti-linux-gnu and when compiling sysdeps/gnu/siglist.c from glibc for mips64r2 (N32 ABI) I get the following ICE. I will try to create a preprocessed source file for this but I wanted to report it first to see if anyone else is seeing it on other platforms. Steve Ellcey sell

Re: ipa-icf::merge TLC

2015-02-27 Thread Steve Ellcey
On Fri, 2015-02-27 at 09:33 -0800, Steve Ellcey wrote: On Fri, 2015-02-27 at 03:10 +0100, Jan Hubicka wrote: Bootstrapped/regtested x86_64-linux, comitted. Honza * ipa-icf.c (symbol_compare_collection::symbol_compare_colleciton): Use address_matters_p. I think this patch

Re: [Patch] Missing plugin header files

2015-01-15 Thread Steve Ellcey
2015-01-14 Steve Ellcey sell...@mips.com * Makefile.in (PLUGIN_HEADERS): Add dominance.h, cfg.h, cfgrtl.h, cfganal.h, cfgbuild.h, cfgcleanup.h, lcm.h, builtins.def, chkp-builtins.def, and pass-instances.def Should pass-instances.def be removed from

[Patch] Missing plugin header files

2015-01-14 Thread Steve Ellcey
I tried compiling an empty plugin that just included gcc-plugin.h and plugin-version.h and found that these header files were included from gcc-plugin.h but not in the list of header files to be copied to the plugin include directory. OK to checkin? Steve Ellcey sell...@imgtec.com 2015-01-14

RE: [Patch, MIPS] Remove some multilibs from mips-mti-linux-gnu target

2014-12-29 Thread Steve Ellcey
From: Matthew Fortune 2014-12-24 Steve Ellcey sell...@mips.com * config/mips/t-mti-linux (MULTILIB_EXCEPTIONS): Add exceptions for mips32[r1] and mips64[r1] with -mnan=2008. This is OK, but I think it may be best to fix t-mti-elf at the same time even though

[Patch, MIPS] Remove some multilibs from mips-mti-linux-gnu target

2014-12-24 Thread Steve Ellcey
-Werror mean that the warning GCC gives is treated as an error and that glibc will not build with these flag combinations so I would like to remove them from the GCC mips-mti-linux-gnu multilib list. This does not affect any GCC target other then mips-mti-linux.gnu. OK to checkin? Steve Ellcey sell

Re: [Patch] Improving jump-thread pass for PR 54742

2014-12-08 Thread Steve Ellcey
of the jump threading, also great. But I was surprised that if I just used '-O3 -fno-thread-jumps' then I still see this optimization. Is that expected? Should this test also check flag_thread_jumps? Or should that be getting checked somewhere else? Steve Ellcey

[Patch,MIPS] Add default SYS_futex definition in libgomp

2014-11-21 Thread Steve Ellcey
if it is not already defined and the same type of define is already in the x86 and alpha futex.h header files in libgomp. Tested on mips-mti-linux-gnu and mips android builds. OK for checkin? Since this is bug fix (inability to build on android) I think it qualifies under the stage 3 rules. Steve Ellcey sell

[PING][PATCH] Change contrib/test_installed for testing cross compilers

2014-11-21 Thread Steve Ellcey
I noticed I never got a reply to this patch proposal I sent out back in March. Does anyone have an opinion on this? I think it would make testing of installed cross compilers easier. Steve Ellcey sell...@imgtec.com --- Original Email --- I was doing some testing

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-18 Thread Steve Ellcey
and you ought to run contrib/check_GNU_style.sh on the patch and clean up the coding style issues it highlights. Thanks, James Greenhalgh I tested the patch on MIPS and things looked good there too. I got the desired speedup and did not see any regressions. Steve Ellcey

RE: [Patch] MIPS configuration patch to enable --with-[arch,endian,abi]

2014-11-06 Thread Steve Ellcey
Catherine thinks. Thanks, Matthew Here is the patch with the endian support removed, I will go ahead and check it in. 2014-11-06 Steve Ellcey sell...@imgtec.com * config.gcc (mips*-mti-linux*): Remove gnu_ld and gas assignments. Set default_mips_arch and default_mips_abi instead

[Patch] MIPS configuration patch to enable --with-[arch,endian,abi]

2014-10-31 Thread Steve Ellcey
, but they are needed if building a non-multilib GCC with a default ABI other than the old 32 bit ABI. Tested with many builds of mips*-*-linux-gnu targets and various combinations of --with-arch, --with-abi, --with-endian, --disable-multilib, and --enable-targets=all. OK to checkin? Steve Ellcey

RE: [Patch] Add MIPS flag to avoid use of ldc1/sdc1/ldxc1/sdxc1

2014-10-28 Thread Steve Ellcey
change their allocator and included a reference to section 7.20.3 of the C standard. Section 7.20.3 of C99 states: The pointer returned if the allocation succeeds is suitably aligned so that it may be assigned to a pointer to any type of object. Steve Ellcey

[Patch] Add MIPS flag to avoid use of ldc1/sdc1/ldxc1/sdxc1

2014-10-27 Thread Steve Ellcey
on mips-mti-linux-gnu. OK to checkin? Steve Ellcey sell...@imgtec.com 2014-10-27 Steve Ellcey sell...@imgtec.com * config/mips/mips.h (ISA_HAS_LDC1_SDC1): Check TARGET_LDC1_SDC1. * config/mips/mips.md (*ANYF:loadx_P:mode): Ditto. * config/mips/mips.md

Re: [Patch] Add MIPS flag to avoid use of ldc1/sdc1/ldxc1/sdxc1

2014-10-27 Thread Steve Ellcey
On Mon, 2014-10-27 at 15:32 -0700, Andrew Pinski wrote: On Mon, Oct 27, 2014 at 3:23 PM, Steve Ellcey sell...@imgtec.com wrote: There are some MIPS patches that have been applied to the Google Android GCC tree but not been submitted to FSF GCC. I would like to get those patches checked

RE: [Patch, MIPS] Cleanup mips header files.

2014-10-08 Thread Steve Ellcey
of GNU_USER_TARGET_LINK_SPEC and double checked that we still pass -EL or -EB to the linker in all cases. Otherwise OK (assuming the link specs behave as described above). Thanks, Matthew Thanks for the review, I have gone ahead and checked in the patch with those changes. Steve Ellcey sell

RE: [Patch, MIPS] Cleanup mips header files.

2014-10-08 Thread Steve Ellcey
for checkin? Steve Ellcey sell...@mips.com 2014-10-08 Steve Ellcey sell...@mips.com * config/mips/mti-linux.h (DRIVER_SELF_SPECS): Change LINUX64_DRIVER_SELF_SPECS to LINUX_DRIVER_SELF_SPECS diff --git a/gcc/config/mips/mti-linux.h b/gcc/config/mips/mti-linux.h index 318e981

Re: [PATCH] Redesign jump threading profile updates

2014-10-02 Thread Steve Ellcey
applying it. It looks good on MIPS now. I haven't done a complete build and test but the ICE went away when I re-applied the patch. Steve Ellcey sell...@mips.com

Re: [PATCH] Redesign jump threading profile updates

2014-10-01 Thread Steve Ellcey
On Wed, 2014-10-01 at 13:04 -0700, Teresa Johnson wrote: 2014-10-01 Teresa Johnson tejohn...@google.com * tree-ssa-threadupdate.c (freqs_to_counts_path): Scale frequencies up when synthesizing counts to avoid rounding errors. I tried this patch on my MIPS toolchain build

Re: [Patch, MIPS] Add .note.GNU-stack section

2014-09-26 Thread Steve Ellcey
(for any platform) so I wasn't planning on creating a glibc to add them to mips glibc assembly language files. OK to check in this patch? Steve Ellcey sell...@mips.com 2014-09-26 Steve Ellcey sell...@mips.com * config/mips/mips.c (TARGET_ASM_FILE_END): Define. * libgcc

[Patch, MIPS] Cleanup mips header files.

2014-09-26 Thread Steve Ellcey
all of them. OK for checkin? Steve Ellcey sell...@mips.com 2014-09-26 Steve Ellcey sell...@mips.com * config/mips/linux64.h: Remove. * config/mips/gnu-user64.h: Remove. * gcc.config (mips*-*-*): Remove references to linux64.h and gnu-user64.h * config

[Patch, MIPS] Add .note.GNU-stack section

2014-09-10 Thread Steve Ellcey
Someone noticed that the MIPS GCC compiler was not putting out the .note.GNU-stack section. This simple patch fixes that problem by calling the standard file_end_indicate_exec_stack function. Tested on mips-mti-linux-gnu, OK to checkin? Steve Ellcey sell...@mips.com 2014-09-10 Steve Ellcey

[Patch] Switch elimination pass for PR 54742

2014-08-19 Thread Steve Ellcey
affect the performance of those benchmarks. The biggest impact I could find was on the perl benchmark in SPEC where I saw around a 0.4% improvement on a MIPS 74k. Not huge, but not nothing. So, OK to checkin? Steve Ellcey sell...@mips.com 2014-08-12 Steve Ellcey sell...@mips.com PR tree

[Patch, documentation] Add documentation for check_effective_target_strict_align

2014-08-19 Thread Steve Ellcey
I was pinged about the lack of documentation for the check_effective_target_non_strict_align procedure I added to target-supports.exp. This patch fixes that. If I don't get any objections in the next day or two I will check this in as an obvious patch. Steve Ellcey sell...@mips.com 2014-08-19

Re: RFC: Patch for switch elimination (PR 54742)

2014-08-14 Thread Steve Ellcey
...@redhat.com wrote: On 08/12/14 11:46, Steve Ellcey wrote: Try setting the header latch fields for the loop structure to NULL, then call loops_set_state (LOOPS_NEED_FIXUP). But that is _not_ the appropriate way of keeping loops preserved! I think that's done when we've scrogged the loop

Re: RFC: Patch for switch elimination (PR 54742)

2014-08-14 Thread Steve Ellcey
people on the CC list for this bug certainly shows interest in having it even if it is just for a benchmark. Does 'competing against other compilers' sound better then 'optimizing for a benchmark'? Steve Ellcey sell...@mips.com

[Patch] Fix crtstuff.c when compiling with mingw tools.

2014-08-13 Thread Steve Ellcey
coming from the sys/types.h system header file (part of glibc in my case). Since crtstuff.c doesn't actually need or use caddr_t my patch undef's it after including auto-host.h in the same way that pid_t, rlim_t, ssize_t, and vfork are already undef'ed. OK to checkin? Steve Ellcey sell...@mips.com

Re: [Patch] Fix crtstuff.c when compiling with mingw tools.

2014-08-13 Thread Steve Ellcey
On Wed, 2014-08-13 at 20:42 +, Joseph S. Myers wrote: On Wed, 13 Aug 2014, Steve Ellcey wrote: This is a ping on a patch I sent out a while ago to fix the GCC build when building with the mingw toolset. This is not a review. When pinging, please give *the gcc.gnu.org URL

Re: [PATCH mips] Do not compile mips16.S in soft-float mode

2014-08-12 Thread Steve Ellcey
against mips16 hard-float objects so we do not need these routines when building libgcc for those cases. */ Steve Ellcey sell...@mips.com

RFC: Patch for switch elimination (PR 54742)

2014-08-12 Thread Steve Ellcey
'loop_optimizer_init (LOOPS_NORMAL)' before the fix_loop_structure but that did not seem to have any affect. Steve Ellcey sell...@mips.com 2014-08-12 Steve Ellcey sell...@mips.com PR tree-opt/54742 * Makefile.in (OBJS): Add tree-switch-shortcut.o. * common.opt (ftree-switch

Re: RFC: Patch for switch elimination (PR 54742)

2014-08-12 Thread Steve Ellcey
patch that seems to be working and that also addresses the comments that Jakub Jelinek had. I want to do more testing before I officially submit it but I thought I would put it out here for others to look over again while I do that. Steve Ellcey sell...@mips.com 2014-08-12 Steve Ellcey sell

[PATCH mips] Pass -msoft-float/-mhard-float flags to GAS

2014-08-08 Thread Steve Ellcey
-linux-gnu toolchain. OK to checkin? Steve Ellcey sell...@mips.com 2014-08-08 Steve Ellcey sell...@mips.com * config/mips/mips.h (ASM_SPEC): Pass float options to assembler. diff --git a/gcc/config/mips/mips.h b/gcc/config/mips/mips.h index 8d7a09f..c1b57b1 100644 --- a/gcc/config

[PATCH mips] Do not compile mips16.S in soft-float mode

2014-08-08 Thread Steve Ellcey
assembly (due to using floating point registers in soft-float mode). Tested with the mips-mti-linux-gnu toolchain. Steve Ellcey sell...@mips.com 2014-08-08 Steve Ellcey sell...@mips.com * config/mips/mips16.S: Skip when __mips_soft_float is defined. diff --git a/libgcc/config/mips/mips16

[PATCH mips] Remove fp64 multilibs from mips-mti-* targets.

2014-08-08 Thread Steve Ellcey
and mips-mti-elf targets. OK to checkin? Steve Ellcey sell...@mips.com 2014-08-08 Steve Ellcey sell...@mips.com * config/mips/t-mti-elf (MULTILIB_OPTIONS): Remove fp64 multilib. (MULTILIB_DIRNAMES): Ditto. * config/mips/t-mti-elf (MULTILIB_OPTIONS): Ditto

RE: [PATCH,MIPS] MIPS64r6 support

2014-06-23 Thread Steve Ellcey
if that change could be easily added, I don't see a clean way to implement it off hand. Steve Ellcey sell...@mips.com 2014-06-23 Steve Ellcey sell...@mips.com * gcc.target/mips/mips.exp (mips-dg-options): Check for range of values on isa or isa_rev pseudo-options. diff --git a/gcc

Re: [PATCH 8/8] Add a common .md file and define standard constraints there

2014-06-16 Thread Steve Ellcey
fixed my build and the testsuite results looked good too. Steve Ellcey sell...@mips.com

Re: [PATCH 8/8] Add a common .md file and define standard constraints there

2014-06-13 Thread Steve Ellcey
)' Steve Ellcey sell...@mips.com extern long long __mips16_syscall4(long, long, long, long, long); typedef unsigned int size_t; union __mips16_syscall_return { long long val; }; typedef struct { int lock; } _IO_lock_t; struct _IO_FILE { _IO_lock_t *_lock; }; typedef struct

[patch, mips, commited] Check in obvious patch to fix mips build.

2014-05-27 Thread Steve Ellcey
The recent comdat group changes broke the mips build because mips.c was not including cgraph.h. I am checking in the following patch as obvious to fix the mips build. 2014-05-27 Steve Ellcey sell...@mips.com * config/mips/mips.c: Add include of cgraph.h. diff --git a/gcc/config

Re: [patch, mips, commited] Check in obvious patch to fix mips build.

2014-05-27 Thread Steve Ellcey
On Tue, 2014-05-27 at 17:12 +0100, Richard Sandiford wrote: Steve Ellcey sell...@mips.com writes: The recent comdat group changes broke the mips build because mips.c was not including cgraph.h. I am checking in the following patch as obvious to fix the mips build. Wasn't that fixed

Re: [patch, mips, commited] Check in obvious patch to fix mips build.

2014-05-27 Thread Steve Ellcey
/src/gcc/gcc/config/mips/mips.c:6278:27: error: 'cgraph_create_node' was not declared in this scope make[2]: *** [mips.o] Error 1 Ah, sorry, hadn't realised we had a direct caller there. Richard No problem, I probably should have included the error message in my original email. Steve

Re: [Patch, testsuite, mips] Fix two failing MIPS tests.

2014-05-05 Thread Steve Ellcey
PUSH_ARGS_REVERSED as 1, simplify code accordingly. (expand_call): Likewise. (emit_library_call_calue_1): Likewise. * expr.c (PUSH_ARGS_REVERSED): Do not define. (emit_push_insn): Always treat PUSH_ARGS_REVERSED as 1, simplify code accordingly. Steve Ellcey sell

Re: [Patch, testsuite, mips] Fix two failing MIPS tests.

2014-05-05 Thread Steve Ellcey
code. But if I run an older GCC on that new swapped code then I get something like the new code. So while the old code may have been better for this example, the new code is better if I swap the arguments around. Steve Ellcey sell...@mips.com

[Patch, testsuite, mips] Fix two failing MIPS tests.

2014-05-02 Thread Steve Ellcey
ABIs). OK to checkin? Steve Ellcey sell...@mips.com 2014-05-02 Steve Ellcey sell...@mips.com * gcc.target/mips/const-anchor-1.c: Modify asm scan. * gcc.target/mips/const-anchor-2.c: Ditto. diff --git a/gcc/testsuite/gcc.target/mips/const-anchor-1.c b/gcc/testsuite/gcc.target

Re: [Patch] Fix crtstuff.c compilation with mingw build.

2014-05-01 Thread Steve Ellcey
On Thu, 2014-04-24 at 19:27 +0200, Rainer Orth wrote: Steve Ellcey sell...@mips.com writes: diff --git a/libgcc/crtstuff.c b/libgcc/crtstuff.c index 12bed4b..d09455f 100644 --- a/libgcc/crtstuff.c +++ b/libgcc/crtstuff.c @@ -54,6 +54,7 @@ see the files COPYING3 and COPYING.RUNTIME

[Patch] Simple change to include/longlong.h to quiet warnings.

2014-04-30 Thread Steve Ellcey
compiling with -Wundef (like glibc is now doing). Ok to checkin? I will submit it for checkin to the binutils and glibc groups as well once it is approved here. Steve Ellcey sell...@mips.com 2014-04-30 Steve Ellcey sell...@mips.com * include/longlong.h: Use 'defined()' to check

[Patch] Fix crtstuff.c compilation with mingw build.

2014-04-24 Thread Steve Ellcey
toolchain. OK to checkin? Steve Ellcey sell...@mips.com 2014-04-24 Steve Ellcey sell...@mips.com * crtstuff.c: Undef caddr_t. diff --git a/libgcc/crtstuff.c b/libgcc/crtstuff.c index 12bed4b..d09455f 100644 --- a/libgcc/crtstuff.c +++ b/libgcc/crtstuff.c @@ -54,6 +54,7 @@ see the files

Re: [Patch] Fix obsolete autoconf macros in configure.ac

2014-04-24 Thread Steve Ellcey
On Thu, 2014-04-24 at 10:23 +0200, Richard Biener wrote: +AC_CHECK_TYPES([ssize_t]) +AC_CHECK_TYPES([caddr_t]) But I am not sure what header file this code would go in. In system.h. Steve Ellcey FYI: I ran into problems defining ssize_t and caddr_t in system.h because

[Patch] Fix obsolete autoconf macros in configure.ac

2014-04-23 Thread Steve Ellcey
a windows GCC using the mingw toolset. I would like to replace the obsolete autoconf macros with a 'proper' one. Tested with my mingw build and a MIPS targetted linux build. OK to checkin? Steve Ellcey sell...@mips.com 2014-04-23 Steve Ellcey sell...@mips.com * configure.ac (caddr_t

Re: [Patch] Fix obsolete autoconf macros in configure.ac

2014-04-23 Thread Steve Ellcey
On Wed, 2014-04-23 at 18:40 +0200, Andreas Schwab wrote: Steve Ellcey sell...@mips.com writes: diff --git a/gcc/configure.ac b/gcc/configure.ac index d789557..98acb1b 100644 --- a/gcc/configure.ac +++ b/gcc/configure.ac @@ -1083,8 +1083,8 @@ int main() fi fi

Ping: [PATCH, PR 60556] Fix ICE on platforms with signed pointer extension.

2014-04-10 Thread Steve Ellcey
Patch ping. This is not a regression as I can reproduce it on GCC 4.8.* but it is an ICE on legal C code. Any chance of having it approved for 4.9? Steve Ellcey sell...@mips.com On Thu, 2014-03-20 at 09:48 -0700, Steve Ellcey wrote: This patch fixes pr60556, a GCC ICE. The problem

Re: RFA: Testsuite PATCH to add support for dlopen tests

2014-04-08 Thread Steve Ellcey
. This toolchain does not support shared libraries and before the change it was not using '-fPIC -shared' during C++ compilations. Now it is and that causes GCC to generate an error message. Steve Ellcey

[Patch, testsuite, mips] Fix gcc.dg/tree-ssa/ssa-ifcombine-13.c

2014-04-08 Thread Steve Ellcey
on mips-mti-elf. OK for checkin? Steve Ellcey sell...@mips.com 2014-04-01 Steve Ellcey sell...@mips.com * gcc.dg/tree-ssa/ssa-ifcombine-13.c: Remove mips*-*-* from option and scan lists. diff --git a/gcc/testsuite/gcc.dg/tree-ssa/ssa-ifcombine-13.c b/gcc/testsuite/gcc.dg/tree

RE: RFA: Testsuite PATCH to add support for dlopen tests

2014-04-08 Thread Steve Ellcey
regressions. Steve Ellcey sell...@mips.com patch Description: patch

Test failure: gcc.dg/tree-ssa/ssa-ifcombine-13.c

2014-04-07 Thread Steve Ellcey
the options and dg-final lists? Steve Ellcey sell...@mips.com

[PATCH, PR 60556] Fix ICE on platforms with signed pointer extension.

2014-03-20 Thread Steve Ellcey
reference into a register. Tested on mips-mti-linux-gnu with no regressions. OK for checkin? Steve Ellcey sell...@mips.com 2014-03-20 Steve Ellcey sell...@mips.com PR middle-end/60556 * expr.c (convert_move): Force symbol references into register. diff --git a/gcc/expr.c b/gcc

[PATCH] Change contrib/test_installed for testing cross compilers

2014-03-06 Thread Steve Ellcey
in site.exp before running the testsuite, thus making it more useful when testing cross-compilers. If you don't use the option nothing is changed. OK to checkin? Steve Ellcey sell...@mips.com 2014-03-06 Steve Ellcey sell...@mips.com * test_installed (--target=): New option. diff --git

Re: [PATCH, testsuite] Fix profile test failures

2014-02-14 Thread Steve Ellcey
/sim.exp patch to the dejagnu mailing list later today. Steve Ellcey sell...@mips.com

[PATCH, testsuite] Fix profile test failures

2014-02-13 Thread Steve Ellcey
. OK for checkin? Steve Ellcey sell...@mips.com 2014-02-13 Steve Ellcey sell...@mips.com * lib/profopt.exp (profopt-execute): Use host instead of target in remote_file and remote_upload calls. diff --git a/gcc/testsuite/lib/profopt.exp b/gcc/testsuite/lib/profopt.exp index

Re: [PATCH, testsuite] Fix profile test failures

2014-02-13 Thread Steve Ellcey
On Thu, 2014-02-13 at 23:09 +, Joseph S. Myers wrote: On Thu, 13 Feb 2014, Steve Ellcey wrote: While testing the C++ profiling tests in g++.dg/bprob and using the qemu simulator we discovered that these tests were passing when we ran the testsuite with no extra options but that if we

[Patch, MIPS, testsuite] Checking in obvious testsuite patch

2014-01-27 Thread Steve Ellcey
I am checking in this testsuite patch as an obvious fix after checking with Richard Sandiford. Steve Ellcey sell...@mips.com 2014-01-27 Steve Ellcey sell...@mips.com * gcc.target/mips/pr52125.c: Add -mno-optgp option. diff --git a/gcc/testsuite/gcc.target/mips/pr52125.c b/gcc

Re: [Patch, cilk, C++] Fix cilk testsuite failure

2014-01-17 Thread Steve Ellcey
; If no issues come up over the weekend with this patch I will backport it to the 4.8 branch next week. Steve Ellcey sell...@mips.com

Re: [Patch, cilk, C++] Fix cilk testsuite failure

2014-01-16 Thread Steve Ellcey
is also looking at the node to see if it is a NE or a EQ. Given that we have a CC reg type is it reasonable/correct that the ne comparision is SI mode and not CC mode? Steve Ellcey sell...@mips.com

Re: [Patch, cilk, C++] Fix cilk testsuite failure

2014-01-16 Thread Steve Ellcey
become a ne:SI in instruction 76 after the combine. Steve Ellcey sell...@mips.com

Re: [Patch, cilk, C++] Fix cilk testsuite failure

2014-01-16 Thread Steve Ellcey
like me to do that? Steve Ellcey sell...@mips.com

Re: [Patch, cilk, C++] Fix cilk testsuite failure

2014-01-16 Thread Steve Ellcey
operand instead of the operator. It was tested with my mips-mti-linux-gnu target and I verified it fixes the bug and causes no regressions. OK to checkin? Steve Ellcey sell...@mips.com 2014-01-15 Andrew Pinski apin...@cavium.com Steve Ellcey sell...@mips.com PR target

[Patch, cilk, C++] Fix cilk testsuite failure

2014-01-15 Thread Steve Ellcey
target predicate. Since no other tests are failing with this problem I would rather tweak the cilk code instead. Tested on mips-mti-linux-gnu with no regressions. OK to checkin? Steve Ellcey sell...@mips.com 2014-01-15 Steve Ellcey sell...@mips.com PR target/59462 * cp-array

[Patch] Remove references to non-existent tree-flow.h file

2014-01-09 Thread Steve Ellcey
the references to these now-nonexistent files. OK to checkin? Steve Ellcey sell...@mips.com 2014-01-09 Steve Ellcey sell...@mips.com * Makefile.in (TREE_FLOW_H): Remove. (TREE_SSA_H): Add files names from tree-flow.h. * doc/tree-ssa.texi (Annotations): Remove reference

[Patch, testsuite, mips] Fix test gcc.dg/delay-slot-1.c for MIPS

2014-01-09 Thread Steve Ellcey
The gcc.dg/delay-slot-1.c test is failing for MIPS targets that do not support the 64 bit ABI because it didn't check to see if that support existed before using the -mabi=64 flag. This patch fixes the problem by using the mips64 check. OK to checkin? Steve Ellcey sell...@mips.com 2014-01-09

[Patch] Fix PR plugin/59335, plugins not compiling

2014-01-09 Thread Steve Ellcey
platform issues that are mentioned in the defect. Tested with my switch-shortcut plugin on MIPS. OK to checkin? Steve Ellcey sell...@mips.com 2014-01-09 Steve Ellcey sell...@mips.com PR plugins/59335 * Makefile.in (PLUGIN_HEADERS): Add gimplify.h, gimple-iterator.h, gimple

Re: [PATCH, ia64] [PR target/52731] internal compiler error: in ia64_st_address_bypass_p, at config/ia64/ia64.c:9357

2013-12-03 Thread Steve Ellcey
On Fri, 2013-11-29 at 15:01 +0300, Kirill Yukhin wrote: Hello, On 20 Nov 18:37, Kirill Yukhin wrote: Hello, Patch in the bottom fixes PR52731. Is it ok for trunk? Ping? -- Thanks, K OK. Steve Ellcey sell...@mips.com

[patch, fortran] Fix Fortran configure for aarch64

2013-11-15 Thread Steve Ellcey
on aarch64 and I tested it on mips, OK for checkin? Steve Ellcey sell...@mips.com 2013-11-15 Steve Ellcey sell...@mips.com * configure.ac: Do not define HAVE_STRTOLD. * configure: Regenerate. diff --git a/libgfortran/configure.ac b/libgfortran/configure.ac index 6417373..6acc0f2

[Patch, mips] MIPS performance patch for PR 56552

2013-11-15 Thread Steve Ellcey
Andrew Pinski created a patch for PR 56552 some months ago but has not checked it in. I would like to go ahead and check it in so that it gets into GCC 4.9. I have tested this on MIPS with no regressions. OK to checkin? Steve Ellcey sell...@mips.com 2013-11-15 Andrew Pinski apin

Re: [PATCH, ia64] [PR target/57491] internal compiler error: in ia64_split_tmode -O2, quadmath

2013-11-13 Thread Steve Ellcey
On Tue, 2013-11-12 at 11:38 +0300, Kirill Yukhin wrote: Hello, On 07 Nov 15:42, Kirill Yukhin wrote: Could you pls take a look? Ping? -- Thanks, K Looks OK to me, go ahead and check it in. Steve Ellcey sell...@mips.com

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

2013-11-12 Thread Steve Ellcey
to reverting my patch. Steve Ellcey sell...@mips.com

[Patch, MIPS] Add new multilibs to mips-mti-* targets.

2013-11-06 Thread Steve Ellcey
point registers; none, 32 bit, or 64 bit. I then made mnan=2008 a seperate entry and added a MULTILIB_EXCEPTIONS to avoid building multilibs with soft-float and nan2008. OK to checkin? 2013-11-06 Steve Ellcey sell...@mips.com * config/mips/mti-linux.h (SYSROOT_SUFFIX_SPEC): Add fp64

[Patch] Fix type of clock_t in timevar.c

2013-11-01 Thread Steve Ellcey
build and a native x86 linux build. OK to checkin? Steve Ellcey sell...@mips.com 2013-11-01 Steve Ellcey sell...@mips.com * timevar.c: Fix type of clock_t. diff --git a/gcc/timevar.c b/gcc/timevar.c index 23b7118..b66f94a 100644 --- a/gcc/timevar.c +++ b/gcc/timevar.c @@ -23,7 +23,7

[Patch] configure patch for caddr_t and ssize_t types

2013-11-01 Thread Steve Ellcey
a problem in my build because caddr_t was also getting set via a typedef by the mingw compilers. Here is my fix, tested with a canadian cross build and with a native x86 linux build. OK to checkin? 2013-11-01 Steve Ellcey sell...@mips.com * configure.ac: Add header checks for fenv.h

Re: [Patch] configure patch for caddr_t and ssize_t types

2013-11-01 Thread Steve Ellcey
On Fri, 2013-11-01 at 19:22 +0100, Marek Polacek wrote: On Fri, Nov 01, 2013 at 11:15:02AM -0700, Steve Ellcey wrote: --- a/gcc/system.h +++ b/gcc/system.h @@ -1060,6 +1060,14 @@ helper_const_non_const_cast (const char *p) #define DEBUG_VARIABLE #endif +#ifndef HAVE_CADDR_T

Re: [Patch] Fix type of clock_t in timevar.c

2013-11-01 Thread Steve Ellcey
On Fri, 2013-11-01 at 12:43 -0700, Mike Stump wrote: On Nov 1, 2013, at 10:47 AM, Steve Ellcey sell...@mips.com wrote: While working on a canadian cross build I ran into a problem with the type of clock_t. If HAVE_CLOCK_T is not defined timevar.c defines it to be int. I think the type

Re: [Patch] Fix type of clock_t in timevar.c

2013-11-01 Thread Steve Ellcey
On Fri, 2013-11-01 at 13:45 -0700, Mike Stump wrote: On Nov 1, 2013, at 12:56 PM, Steve Ellcey sell...@mips.com wrote: You should report a bug to them and have them define clock_t. They are defining clock_t, but for some reason the GCC configure is not seeing it (perhaps because of what

[Patch] Fix canadian cross build on systems with no fenv.h

2013-10-30 Thread Steve Ellcey
'AC_LANG_CPLUSPLUS'. Tested with both my canadian cross build and a standard cross build targetting mips-mti-elf. OK for checkin? Steve Ellcey sell...@mips.com 2013-10-30 Steve Ellcey sell...@mips.com * configure.ac: Add header checks for fenv.h and complex.h. * configure: Regenerate

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

2013-10-24 Thread Steve Ellcey
the libraries and not just for libgfortran. Steve Ellcey sell...@mips.com

[patch, mips] Fix optimization bug involving nor instruction

2013-10-22 Thread Steve Ellcey
the cost calculation of a nor to be the cost of one operation (plus the cost of getting the operands into registers) instead of the cost of three operations. Tested with no regressions, OK to checkin? Steve Ellcey sell...@mips.com 2013-10-22 Steve Ellcey sell...@mips.com * config/mips

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

2013-10-22 Thread Steve Ellcey
expression. I.e. *total = (); Steve Ellcey

<    1   2   3   4   5   6   7   >