RFA: RL78: Always select register bank 0 at the start of an ISR

2013-01-03 Thread Nick Clifton
happens. No regressions with a rl78-elf toolchain. OK to apply ? Cheers Nick gcc/ChangeLog 2013-01-03 Nick Clifton * config/rl78/rl78.c (rl78_expand_prologue): Always select register bank 0 at the start of an interrupt handler. Index: gcc/config/rl78/rl78.c

Re: [PATCH, libgcc, ARM] __gnu_f2h_internal inaccuracy

2012-12-13 Thread nick clifton
Hi John, John Tytgat * config/arm/fp16.c (__gnu_f2h_internal): Fix inaccuracy. Approved and applied. Cheers Nick

Commit: RX backend fixes

2012-11-20 Thread Nick Clifton
Hi Guys, I am applying this patch to fix up a few issues with the RX backend. With this patch applied there are 355 fewer gcc testsuite regressions. Cheers Nick gcc/ChangeLog 2012-11-20 Nick Clifton * config/rx/rx.c (rx_function_arg_boundary): When using the RX ABI

Commit: V850: Various backend fixes

2012-11-20 Thread Nick Clifton
Hi Guys, I am applying the patch below to fix various small problems with the V850 backend. With this patch applied there are 454 fewer unexpected failures in the gcc testsuite for a v850-elf toolchain. Cheers Nick gcc/ChangeLog 2012-11-20 Nick Clifton * config/v850/v850.c

Re: VEC re-write [patch 21/25]

2012-11-19 Thread nick clifton
Hi Diego, 2012-11-15 Diego Novillo Adjust for new vec API (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec) * config/rx/rx.c: Use new vec API in vec.h. This is fine. Cheers Nick

Re: Top Level GCC change questions

2012-10-26 Thread nick clifton
Hi Steve, It looks like there is another patch that has not been checked in to the GCC top level tree but not binutils and a patch in binutils but not GCC. Is there any automation for this or is it still up to each person checking in files to copy stuff over by hand? As far as I know it is st

Re: [patch] unbreak iq2000 build a bit

2012-10-17 Thread nick clifton
Hi Steven, iq2000 build is broken since at least r162089 (July 2010). I'm going to commit this patch to fix this part of the build problem. Thanks. It still fails later one, but I don't care about that FYI I have this patch installed in my local sources that allows the iq2000 port to buil

Commit: RX: Warn about multiple fast interrupt routines.

2012-10-03 Thread Nick Clifton
problem. Cheers Nick PS. I have even remembered to include a patch to the html documentation as well! gcc/ChangeLog 2012-10-03 Nick Clifton * config/rx/rx.c (struct decl_chain): New local structure. (warned_decls): New local variable. Contains a stack of decls for

Re: RFC: Using DECL_NO_LIMIT_STACK as a backend specific flag

2012-10-03 Thread nick clifton
Hi Ian, Can't you just keep a list of the decls for which you have issued the warning? Yes - that would work too. In fact I agree that this would be cleaner solution in my particular case. I'll create a new patch... How many decls do you expect to be on that list? Not very many. Mayb

RFC: Using DECL_NO_LIMIT_STACK as a backend specific flag

2012-10-03 Thread Nick Clifton
Hi Guys, Is there a target specific way to add a flag bit to a function decl ? For the RX port I want to be able to mark a function decl for which I have issued a warning message. Unfortunately I could find no easy way to do this other than stealing one of the generic bits (no_limit_stac

Commit: V850 tidy ups

2012-09-07 Thread Nick Clifton
Hi Guys, I am applying the patch below to tidy up the v850 backend a little and fix a few minor bugs. Cheers Nick gcc/ChangeLog 2012-09-07 Nick Clifton * config/v850/v850.h (DBX_DEBUGGING_INFO): Define. (ASM_GENERATE_INTERNAL_LABEL): Define if not already provided

Re: [middle-end] Add machine_mode to address_cost target hook

2012-09-04 Thread nick clifton
Hi Oleg, And so I did. Attached is an updated patch that adds the address space parameter to the address_cost function. I hope that this change does not reset the ACKs so far: [x] target-independent bits [ ] alpha [ ] arm [ ] avr [ ] bfin [ ] cr16 [ ] cris [ ] epip

Re: [wwwdocs] PATCH for Re: Commit: XStormy16: Add support for -fstack-usage

2012-09-03 Thread nick clifton
Hi Gerald, Anything you'd like to add or tweak? No, it is fine thanks. (Is "stormy" fine as an anchor, or should we go for "xstormy" or the full "xstormy16"? I used what we have for the port itself...) I like the full "xstormy16" as well. I think that the fact that the gcc backend sourc

RFA: libiberty: cope with integer overflow in _objalloc_alloc

2012-08-31 Thread Nick Clifton
geLog 2012-08-31 Jan Lieskovsky Nick Clifton * objalloc.c (_objalloc_alloc): Align length before testing for zero length. Refuse to allocate when length is so big that integer overflow would occur during internal calculations. Index: libi

Re: RFA: LM32: Fix building libgcc

2012-08-22 Thread nick clifton
Next there is seg-fault building ffssi2 in newlib. I do not have a fix for this one yet, but at least libgcc builds now. OK to apply ? Cheers Nick gcc/ChangeLog 2012-08-21 Nick Clifton * config/lm32/lm32.md (return_internal): Place the (return) as the first element in th

Re: Commit: BFIN: Fix use of VEC_last macro in bfin.c

2012-08-17 Thread nick clifton
Hi Diego, Would it be too much imposition for you to put this on the wiki? I would be happy to. Where do you think would be appropriate ? I would guess that somewhere off the Getting Started page (http://gcc.gnu.org/wiki/GettingStarted) would be best. Maybe a new item in the Tutorials se

RFA: LM32: Fix building libgcc

2012-08-17 Thread Nick Clifton
built. OK to apply ? Cheers Nick gcc/ChangeLog 2012-08-17 Nick Clifton * config/lm32/lm32.md (return): Rename to simple_return. (epilogue): Rename to return. Index: gcc/config/lm32/lm32.md === --- gcc/config

Commit: FR30: Fix building libgcc

2012-08-17 Thread Nick Clifton
Hi Guys, I am applying the patch below to fix building libgcc for the fr30-elf target. The problem was that the comparison patterns were applying a mode to the operator which prevents them from matching rtl generated by the middle-end. Cheers Nick gcc/ChangeLog 2012-08-17 Nick

Commit: MCore: Fix building libgcc

2012-08-17 Thread Nick Clifton
Hi Guys, I am applying the patch below to fix a problem building libgcc for the mcore-elf target. The cbranchsi4 pattern was applying a mode to the comparison operator which was preventing it from matching rtl generated by the middle end. Cheers Nick gcc/ChangeLog 2012-08-17 Nick

Commit: IQ2000: Remove modes from

2012-08-17 Thread Nick Clifton
that the two extend-and-compare patterns were running into reload problems when building libstdc++-v3. I have disabled them until I have time to investigate further. Cheers Nick gcc/ChangeLog 2012-08-17 Nick Clifton * config/iq2000/iq2000.md (cbranchsi4): Remove mode from

Commit: MEP: Use C++ to compile mep-pragma.

2012-08-17 Thread Nick Clifton
Hi DJ, I am applying the following patch as an obvious patch fix for building the MEP targeted port of gcc. It fixes the build rule for mep-pragma.o so that is uses the C++ compiler instead of the C compiler. Cheers Nick gcc/ChangeLog 2012-08-17 Nick Clifton * config/mep

Re: Commit: BFIN: Fix use of VEC_last macro in bfin.c

2012-08-17 Thread nick clifton
Hi Diego, Thanks Nick. I made the wrong fix here, sorry about that. I will be making more changes to VEC_ shortly. What's a good way for me to test them? All I was doing was building a variety of targets, just to make sure that a local, generic patch of my own did not break anything. If yo

Commit: Fix i386 ASM_OUTPUT_ASCII macro for C++ compilation.

2012-08-16 Thread Nick Clifton
r: invalid conversion from 'const unsigned char*' to 'const char*' [-fpermissive] With the patch applied the compiler now builds correctly. Cheers Nick gcc/ChangeLog 2012-08-16 Nick Clifton * config/i386/i386elf.h (ASM_OUTPUT_ASCII): Cast _ascii_byte

Commit: BFIN: Fix use of VEC_last macro in bfin.c

2012-08-16 Thread Nick Clifton
se '->' ?) With this patch applied the bfin port now compiles correctly. Cheers Nick gcc/ChangeLog 2012-08-16 Nick Clifton * config/bfin/bfin.c (hwloop_optimize): Fix use of VEC_last macro. Index: gcc/config/bfin/bfin.c =

Commit: Fixes for building avr-elf with g++.

2012-08-16 Thread Nick Clifton
++ style headers (eg vec.h). * avr.c was using integer values in places where a specific enum was required. The patch below fixes these problems and allows avr-elf-gcc to be built. Cheers Nick gcc/ChangeLog 2012-08-16 Nick Clifton * config/avr/t-avr: Replace

Re: Commit: RL78: Include tree-pass.h

2012-08-08 Thread nick clifton
Hi Richard, Err - you are inside the compiler and should not use plugin stuff to register your machine dependent pass. Umm, OK, what is the correct method for registering target specific passes ? (Ones that need to run at times other than TARGET_MACHINE_DEPENDENT_REORG). Cheers Nick

Commit: RL78: Include tree-pass.h

2012-08-08 Thread Nick Clifton
Hi DJ, I am applying the following patch to the gcc mainline as an obvious fix for the following problem building the RL78 backend: gcc/config/rl78/rl78.c:151:3: error: 'PASS_POS_INSERT_BEFORE' undeclared here (not in a function) Cheers Nick gcc/ChangeLog 2012-08-08 Ni

Re: Huge change on 4.7 branch

2012-08-06 Thread nick clifton
Hi H.J.., Hi Richard, Change reverted. Sorry about the mess-up. I was trying to perform a local merge but I had my cached svn control directories the wrong way around. Doh. Cheers Nick

RFA: MN10300: Add support for -fstack-usage

2012-08-06 Thread Nick Clifton
Hi Jeff, Hi Alex, Please may I apply the small patch below to add support for reporting stack usage to the MN10300 port ? Cheers Nick gcc/ChangeLog 2012-08-06 Nick Clifton * config/mn10300/mn10300.c (mn10300_expand_prologue): Report stack usage if requested. Index

Re: Commit: XStormy16: Add __clrsbhi2() to libgcc

2012-08-01 Thread nick clifton
Hi Johann, Hi, just out of curiosity: Is there a special reason to add a stormy16-specific implementation instead of using the code in libgcc2.c? t-avr for example uses the C implementation in libgcc2.c for HI. Are there disadvantages? Apart from not using some slightly frightening makefile mag

Commit: XStormy16: Add support for -fstack-usage

2012-07-31 Thread Nick Clifton
Hi Guys, I am checking in the patch below to add support for reporting stack usage by the XStormy16 backend. Cheers Nick gcc/ChangeLog Index: gcc/ChangeLog 2012-07-31 Nick Clifton * config/stormy16/stormy16.c (xstormy16_expand_prologue): Add support for reporting

RFA: M32C: Add __clrsbhi2 to libgcc

2012-07-31 Thread Nick Clifton
Hi DJ, The patch below adds an implementation of the __clrsbhi2 function to libgcc for the M32C port. Implementing this function resolves several gcc testsuite failures that rely upon the function being present. Ok to apply ? Cheers Nick libgcc/ChangeLog 2012-07-31 Nick Clifton

Commit: FRV: Add support for -fstack-usage

2012-07-31 Thread Nick Clifton
Hi Guys, I am applying the patch below to add support for the -fstack-usage option to the FRV backend. Cheers Nick gcc/ChangeLog 2012-07-31 Nick Clifton * config/frv/frv.c (frv_expand_prologue): Report stack usage. gcc/testsuite/ChangeLog 2012-07-31 Nick Clifton

Commit: FRV: Fix typo in definition of QUAD macro

2012-07-31 Thread Nick Clifton
Hi Guys, I am checking in the patch below as an obvious fix for the definition of the QUAD macro in the FRV backend. Cheers Nick gcc/ChangeLog 2012-07-31 Nick Clifton * config/frv/frv.c (QUAD): Fix typo. Index: gcc/config/frv/frv.c

Commit: XStormy16: Add __clrsbhi2() to libgcc

2012-07-31 Thread Nick Clifton
Hi Guys, I am checking in the patch below to add a __clrsbhi2 function to libgcc for the XStormy16 port. This fixes several gcc testsuite failures that need this particular function. Cheers Nick libgcc/ChangeLog 2012-07-31 Nick Clifton * config/stormy16/lib2funcs.c

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

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

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

2012-07-24 Thread nick clifton
Hi Sandra, I've updated the patch One suggestion - rather than having architecture specific test files, why not just have a single generic test case with a new dg-require-naked-attribute qualifier. That way the mcore port would be tested as well as the ARM port. I'm not set up to test o

Re: [PATCH, frv-linux] Silence warning

2012-07-23 Thread nick clifton
rameter to frv_ifcvt_machdep_init is not used. So I just made it a void * and updated frv.c. Cheers Nick gcc/ChangeLog 2012-07-23 Jan-Benedict Glaw Nick Clifton * config/frv/frv-protos.h (frv_ifcvt_machdep_init): Prototype. * config/frv/frv.c (frv_ifcvt_machdep_init): Chan

Re: [PATCH, frv-linux] Silence warning

2012-07-23 Thread nick clifton
Hi JBG, warning: implicit declaration of function ‘frv_ifcvt_machdep_init’ warning: no previous prototype for ‘frv_ifcvt_machdep_init’ oops! gcc/ 2012-07-21 Jan-Benedict Glaw * config/frv/frv-protos.h: Update copyright year, include basic-block.h (for ce_if

Re: Commit: ARM: Document -munaligned-access

2012-07-20 Thread nick clifton
Hi H-P, *cough* Sounds like a call for a note in changes.html, to warn people that they have to turn on the alignment feature in their startup code (for whatever OS) for their ARMv6-or-later targets, if they have not already done so. What a good idea :-) Maybe something like: Index: chan

Fwd: Re: Commit: ARM: Document -munaligned-access

2012-07-18 Thread nick clifton
Hi Richard, I have a documentation update for the 4.7 branch. Is it OK to apply this ? Cheers Nick gcc/ChangeLog 2012-07-18 Nick Clifton * doc/invoke.texi (ARM Options): Document -munaligned-access. Index: gcc/doc/invoke.texi

Commit: ARM: Document -munaligned-access

2012-07-18 Thread Nick Clifton
Hi Guys, I am checking in this patch to the mainline to document the ARM port's -munaligned-access command line option. Cheers Nick gcc/ChangeLog 2012-07-18 Nick Clifton * doc/invoke.texi (ARM Options): Document -munaligned-access. Index: gcc/doc/invoke

Re: RFA: MN10300: Call default_encode_section_info

2012-07-06 Thread nick clifton
Hi Jeff, Thanks - patch committed. I decided that it would be best to call default_encode_section_info first, just in case it turns out to be important one day. Cheers Nick

RFA: MN10300: Call default_encode_section_info

2012-07-05 Thread Nick Clifton
SYMBOL_FLAG_FUNCTION, SYMBOL_FLAG_LOCAL or the TLS encoding. Fixed by the following patch, and tested on an mn10300-elf toolchain. OK for mainline and 4.7/4.6 branches ? Cheers Nick gcc/ChangeLog 2012-07-05 Nick Clifton * config/mn10300/mn10300.c (mn10300_encode_section_info): Call

Re: RFA: MEP: Fix use of delete_insn.

2012-07-03 Thread nick clifton
not any others that follow it. Whilst doing this I noticed that follow might be set incorrectly (to a debug insn), so I fixed that as well. Cheers Nick gcc/ChangeLog 2012-07-03 Nick Clifton * config/mep/mep.c (mep_reorg_regmove): Use next_nonnote_non_debug_insn to advance t

Re: RFA: MEP: Fix use of delete_insn.

2012-07-02 Thread nick clifton
Hi DJ, We have this at the top of the loop, so I don't think it matters: next = NEXT_INSN (insn); if (GET_CODE (insn) != INSN) continue; Good point. However, I think an insn will be skipped if we use NEXT. Perhaps we want PREV? Or the loop might need to be

RFA: MN10300: Replace REG_SAVE_BYTES macro with calls to mn10300_get_live_callee_saved_regs

2012-06-29 Thread Nick Clifton
does this. Tested with no regressions on an mn10300-elf and an am33_2.0-linux-gnu toolchain. OK to apply ? Cheers Nick gcc/ChangeLog 2012-06-29 Nick Clifton * config/mn10300/mn10300.c (REG_SAVE_BYTES): Delete. (mn10300_get_live_callee_saved_regs): If requested return a

RFA: MEP: Fix use of delete_insn.

2012-06-29 Thread Nick Clifton
next_nonnote_nondebug_insn() instead. What do you think ? Cheers Nick gcc/ChangeLog 2012-06-29 Nick Clifton * config/mep/mep.c (mep_reorg_regmove): Get next insn before calling delete_insn. Index: gcc/config/mep/mep.c === --- gcc

Commit: LM32: Fix typo in

2012-06-29 Thread Nick Clifton
Hi Sebastien, Hi Steven I am applying the patch below as obvious fix for a typo in lm32.c. Cheers Nick gcc/ChangeLog 2012-06-29 Nick Clifton * config/lm32/lm32.c (lm32_compute_frame_size): Fix typo. Index: lm32.c

Commit: RX: Fix comparesi3_extend pattern

2012-06-27 Thread Nick Clifton
the gcc testsuite. Cheers Nick gcc/ChangeLog 2012-06-27 Nick Clifton * config/rx/rx.md (comparesi3_extend): Remove = modifier from input operand. Index: gcc/config/rx/rx.md === --- gcc/config/rx/rx.md (rev

Re: Commit: RX: Fix simple_return pattern

2012-06-27 Thread nick clifton
Hi Mike, I plan on applying a similar patch to the mainline sources once I have finished regression testing them. Really, trunk should always go in first... Could you hold 4.7 until trunk goes in? Sorry, I had already checked the patch in. I have now checked in the trunk patch, with o

Re: [PATCH ARM iWMMXt 0/5] Improve iWMMXt support

2012-06-26 Thread nick clifton
Hi Matt, There's also a trivial documentation fix: [PATCH 1/2] doc: Correct __builtin_arm_tinsr prototype documentation and a test to exercise the intrinsics: [PATCH 2/2] arm: add iwMMXt mmx-2.c test These have both been checked in. It turns out that both needed minor updates as some of th

Commit: RX: Fix simple_return pattern

2012-06-26 Thread Nick Clifton
I have finished regression testing them. Cheers Nick gcc/ChangeLog 2012-06-26 Nick Clifton * config/rx/rx.md (simple_return): Use the simple_return rtl. Index: gcc/config/rx/rx.md === --- gcc/config/rx/rx.md

Re: RFC/RFA: Allow targets to override the definition of FLOAT_BIT_ORDER_MISMATCH

2012-06-26 Thread nick clifton
Ping ? http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00806.html libgcc/ChangeLog 2012-06-13 Nick Clifton * fp-bit.h (FLOAT_BIT_ORDER_MISMATCH): If LIBGCC2_FLOAT_BIT_ORDER_MISMATCH is defined then use this to determine if FLOAT_BIT_ORDER_MISMATCH should be defined

RFC/RFA: Allow targets to override the definition of FLOAT_BIT_ORDER_MISMATCH

2012-06-13 Thread Nick Clifton
is correct however, then please may I apply the patch ? Cheers Nick libgcc/ChangeLog 2012-06-13 Nick Clifton * fp-bit.h (FLOAT_BIT_ORDER_MISMATCH): If LIBGCC2_FLOAT_BIT_ORDER_MISMATCH is defined then use this to determine if FLOAT_BIT_ORDER_MISMATCH should be defined

Re: [PATCH ARM iWMMXt 0/5] Improve iWMMXt support

2012-06-13 Thread nick clifton
Hi Matt, Hi Xinyu, This series was written by Marvell and sent by Xinyu Qi a number of times in the last year. Sorry for the long delay in reviewing these patches. Overall they were fine, with only a few, very minor, formatting issues. I have committed the entire series of patches to the m

Re: [PATCH ARM iWMMXt 0/5] Improve iWMMXt support

2012-06-11 Thread nick clifton
Hi Matt, This is just to let you know that I am currently reviewing these patches. I do have a problem however. With the patches applied I am seeing very bad results from the gcc testsuite when run with -mcpu=iwmmxt. (Bad as in the testsuite takes days to run and most tests fail). I am

Re: [patch] Deal with #ident without

2012-06-07 Thread nick clifton
Hi Steven, This is the patch to deal with #ident. It removes the ASM_OUTPUT_IDENT and IDENT_ASM_OP target macros, and replaces them with a single target hook. The RX parts of this patch are approved. Cheers Nick

Re: RFA: Fix detecting of in-tree MPFR 3.1.0 sources

2012-05-09 Thread nick clifton
Hi Guys, Ok. I think it's nicer for the users if we enable both builds to work. Thanks. I have applied the patch, along with a reference to PR 50461 and a credit to Paul Smith in the ChangeLog entry since he came up with basically the same patch as mine. Cheers Nick

RFA: Fix detecting of in-tree MPFR 3.1.0 sources

2012-05-09 Thread Nick Clifton
th an in-tree copy of the mpfr 2.4.2 sources, just to make sure that the old paths still worked. Both builds worked. OK to apply ? Cheers Nick gcc/ChangeLog 2012-05-09 Nick Clifton * configure.ac (mpfr-dir): When using in-tree MPFR sources allow for the fact that from re

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

2012-05-09 Thread nick clifton
Hi DJ, Thanks for the test program. With that I can confirm that the match_dup does work and that the bset instruction is still generated even with my patch applied. OK to apply ? Cheers Nick

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

2012-05-08 Thread nick clifton
Hi DJ, Make sure a match_dup will still match the generated pattern later, I've had problems with match_dup not matching two rtx that rtx_equals() says are "the same" but not physically the same. I have tried, but failed, to find a way to trigger the use of the bset_qi pattern. :-( I tried re

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

2012-05-02 Thread nick clifton
int with a match_dup. Tested with no regressions on an m32c-elf toolchain. OK to apply ? Cheers Nick gcc/ChangeLog 2012-05-02 Nick Clifton PR target/53120 * config/m32c/bitops.md (bset_qi): Change operand 2 from having a "0" constraint to being a (match_d

Re: [rx] add initial rtx_costs() function

2012-05-01 Thread nick clifton
Hi DJ, Initial implementation of RTX_COSTS target function for rx-elf. Minor increase in coremark scores, and enables division by multiplication of reciprocals, tested on trunk and 4.7. Ok for trunk and/or 4.7 branch? * config/rx/rx.c (TARGET_RTX_COSTS): Define. (rx_rtx_costs)

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

2012-04-19 Thread nick clifton
Hi Doug, 2012-04-18 Doug Evans * dwarf2.h (dwarf_location_atom): Add DW_OP_GNU_addr_index. Approved - please apply. Cheers Nick

Commit: RL78: Remove use of TODO_dump_func

2012-04-13 Thread Nick Clifton
Hi DJ, The optimization pass flag "TODO_dump_flag" has been removed (see patch committed 2012-04-11) which was causing the RL78 backend to fail to build. I am applying the following patch as an obvious fix. Cheers Nick gcc/ChangeLog 2012-04-13 Nick Clifton * c

Re: RFA: consolidate DWARF strings into libiberty

2012-04-07 Thread nick clifton
Hi Tom, Built and regtested on x86-64 Fedora 16. Ok? Tom 2012-03-15 Tom Tromey * dwarf2out.c (dwarf_stack_op_name): Use get_DW_OP_name. (dwarf_tag_name): Use get_DW_TAG_name. (dwarf_attr_name): Use get_DW_AT_name. (dwarf_form_name): Use get_DW_FORM_name.

Re: RFA: RL78: Add -fstack-usage support

2012-04-04 Thread nick clifton
Hi Eric, > On 04/04/12 12:24, Eric Botcazou wrote: You probably need to adjust gcc.dg/stack-usage-1.c too. s/flag_stack_usage/flag_stack_usage_info/ Thanks for the corrections. Revised patch attached. OK for mainline/4.7 branch ? Cheers Nick gcc/ChangeLog 2012-04-04 Nick Clif

RFA: RL78: Add -fstack-usage support

2012-04-04 Thread Nick Clifton
Hi DJ, Please could I apply the patch below to the RL78 port ? It adds support for generating stack use info with the -fstack-usage option. Also - may I apply this patch to the 4.7 branch as well please ? Cheers Nick gcc/ChangeLog 2012-04-04 Nick Clifton * config/rl78/rl78

Re: [patch] Poison SMALL_REGISTER_CLASSES

2012-02-16 Thread nick clifton
Hi Mike, Hi Steve, * config/rx/rx.h: Remove SMALL_REGISTER_CLASSES. I wasn't sure if these are obvious, because I already removed the definition from rx.h before, see ChangeLog-2010. Maybe someone put it back intentionally. Nick did it (intentionally) with: Sorry about that - sanfu

Re: PR middle-end/24306 revisited: va_arg and zero-sized objects

2012-02-01 Thread nick clifton
Hi Richard, [ Nick, you might remember mentioning this bug a few months back. I think I've finally got a proper fix, rather than the failed attempt I originally sent. ] Thanks for fixing this. I am going to copy your patch into our internal sources and mark it as a local fix on the ass

Commit: RX: Predicate return pattern

2012-01-17 Thread Nick Clifton
Hi Guys, As pointed out by Richard Henderson the new return pattern in the RX backend should only be used when it is possible to return with just a simple RTS instruction, so I am checking in the patch below to implement this. Cheers Nick gcc/ChangeLog 2012-01-17 Nick Clifton

Re: Commit: RX: Add return pattern

2012-01-12 Thread nick clifton
Hi Richard, Not an ideal solution, since the availability of this pattern implies it's extremely cheap, and we'll replace jumps to the epilogue with this pattern. Or to define an availability predicate similar to i386, testing if the epilogue is trivial, and only a return insn is needed. Somet

Commit: RX: Add return pattern

2012-01-11 Thread Nick Clifton
Hi Guys, I am checking in the patch below to fix a problem building the RX port. Targets that define the "simple_return" pattern must also define a "return" pattern. Otherwise gcc/function.c will fail to build. Cheers Nick gcc/ChangeLog 2012-01-11 Nick Clifton

RFA: AVR: Provide local version of c_addr_space_name

2011-11-30 Thread Nick Clifton
? Cheers Nick gcc/ChangeLog 2011-11-30 Nick Clifton * config/avr/avr.c (avr_c_addr_space_name): New function. (avr_out_lpm): Use it. (avr_pgm_check_var_decl): Use it. (avr_insert_attributes): Use it. Index: gcc/config/avr/avr.c

Commit: ARM: Add comments to emitted .eabi_attribute directives

2011-10-12 Thread Nick Clifton
Hi Guys, I am checking in the patch below to add comments to the .eabi_attribute assembler directives emitted by the ARM backend, when commented assembler output is enabled. Cheers Nick gcc/ChangeLog 2011-10-12 Nick Clifton * config/arm/arm.h (EMIT_EABI_ATTRIBUTE): New macro

RFA: Compare Elimination Pass: Fix use of dataflow info

2011-10-07 Thread Nick Clifton
ranch ? Cheers Nick gcc/ChangeLog 2011-10-07 Nick Clifton PR middle-end/49801 * compare-elim.c (find_comparisons_in_bb): Use df_get_live_in and df_get_live_out instead of accessing the bitmaps directly. (execute_compare_elim_after_reload): Remove calls to df_set_

Re: Commit: RX: Codegen bug fixes

2011-10-06 Thread Nick Clifton
Hi Richard, The SMIN pattern has the same problem. *sigh* Fixed. Cheers Nick

RFC: ARM: Add comments to emitted .eabi_attribute directives

2011-10-05 Thread Nick Clifton
in effect) emitted by GCC. It does not have any dependencies upon header files, nor does it attempt to make use of the symbolic names in the actual .eabi_directives. Any objections to this version of the patch ? Cheers Nick gcc/ChangeLog 2011-10-05 Nick Clifton * config/arm

Commit: RX: Add PID support

2011-10-05 Thread Nick Clifton
-time, rather than link-time, and without the overhead of storing relocations in the executable image. The code was written by DJ Delorie for Renesas and it is now being contributed back to the FSF. Cheers Nick gcc/ChangeLog 2011-10-05 DJ Delorie Nick Clifton

Commit: RX: Codegen bug fixes

2011-10-05 Thread Nick Clifton
e min and max instructions did not work for the SMAX insn, as the RX does not have narrow mode versions of this insn. Cheers Nick gcc/ChangeLog 2011-10-05 Nick Clifton * config/rx/rx.md (tablejump): Add missing label. (adddi3_internal): Mark operand 0 as early-clob

Re: ifcvt cond_exec support rewrite

2011-09-30 Thread Nick Clifton
Hi Bernd, Experiments show that the existing multi-if-block support isn't terribly effective on FRV; before-after comparisons show that by turning it off, there are three spots in gcc that are meaningfully changed, and below 20 in the C benchmarks of SPEC2k. FRV also doesn't build in mainline,

Commit: FRV: Fix problems building toolchain.

2011-09-29 Thread Nick Clifton
information for function prologues when sometimes CALL_ARG_LOCATION notes can be separated from their CALL insns. Cheers Nick gcc/ChangeLog 2011-09-29 Nick Clifton Bernd Schmidt * config/frv/frvbegin.c: Fix location of unwind-dw2-fde.h header file

Commit: RX: Add support for MIN and MAX instructions in QI and HI modes

2011-09-28 Thread Nick Clifton
Hi Guys, I am going to apply the patch below to the RX backend to add support for generating MIN and MAX instructions for HI and QI modes. Cheers Nick gcc/ChangeLog 2011-09-28 Nick Clifton * config/rx/predicates.md (rx_minmax_operand): New predicate. Accepts immediates

Commit: RX: Fix problems building libgcc

2011-09-28 Thread Nick Clifton
libgcc/ChangeLog 2011-09-28 Nick Clifton * config/rx/rx-lib.h: Always restrict doubles to the SF type when 64-bit doubles are not enabled. * config/rx/rx-abi.h: Fix extraneous renaming of the floatsisf and floatunsisf functions. Index: libgcc/config/rx/rx-lib.h

Re: RFC: ARM: Add comment enumerating emitted .eabi_attribute tags

2011-09-27 Thread Nick Clifton
Hi Richard, You should only emit the comment for -dA, I think. Really ? I thought that it was kind of nice to have the comments in the default output. If I go with -dA as the trigger then I assume that it is also OK to add the comments if --verbose-asm is enabled, yes ? Cheers Nick

Re: RFC: ARM: Add comment enumerating emitted .eabi_attribute tags

2011-09-27 Thread Nick Clifton
Hi Ramana, +/* Get the definitions of the ARM EABI Attribute tag values. */ +#define BFD_ARCH_SIZE +#include "elf/arm.h" Defining BFD_ARCH_SIZE appears to be a bit of a hack. I would also ifdef this inclusion on TARGET_AAPCS since we shouldn't really be caring about object attributes for non

RFC: ARM: Add comment enumerating emitted .eabi_attribute tags

2011-09-27 Thread Nick Clifton
week. Cheers Nick gcc/ChangeLog 2011-09-27 Nick Clifton * config/arm/arm.c (EMIT_EABI_ATTRIBUTE): New macro. (arm_file_start): Use it to generate .eabi_attribute assembler pseudo-ops. Index: gcc/config/arm/arm.c

Re: [M32R] Use match_test rather than eq/ne symbol_ref

2011-09-26 Thread Nick Clifton
Hi Richard, gcc/ * config/m32r/m32r.md: Use match_test rather than eq/ne symbol_ref throughout file. Approved - please apply. Cheers Nick

Re: [IQ2000] Use match_test rather than eq/ne symbol_ref

2011-09-26 Thread Nick Clifton
Hi Richard, gcc/ * config/iq2000/iq2000.md: Use match_test rather than eq/ne symbol_ref throughout file. Approved - please apply. Cheers Nick

Re: RFA: MN10300: Fix splitting AND insns

2011-09-09 Thread Nick Clifton
Hi Jeff, This is fine, Thanks - committed. after adding a function comment for mn10300_split_and_operand_count; I added this comment: /* This function is used to help split: (set (reg) (and (reg) (int))) into: (set (reg) (shift (reg) (int)) (set (reg) (shift (reg) (in

RFA: MN10300: Fix splitting AND insns

2011-09-07 Thread Nick Clifton
28 fewer GCC testsuite failures, 7 fewer G++ testsuite failures and no regressions. OK to apply ? Cheers Nick gcc/ChangeLog 2011-09-07 Nick Clifton * config/mn10300/mn10300.c (mn10300_split_and_operand_count): Return a positive value to indicate that the bits at the

RFA: MN10300: Add jump label to Lcc insn

2011-09-06 Thread Nick Clifton
Nick gcc/ChangeLog 2011-09-06 Nick Clifton * config/mn10300/mn10300.c (mn10300_insert_setlb_lcc): Set the jump label on the parallel part of the insn. Index: gcc/config/mn10300/mn10300.c === --- gcc/config/mn10300

Re: RFA: Do not use cmpstrnsi to implement builtin memcmp

2011-08-12 Thread Nick Clifton
Hi Jeff, The point of the patch is that the cmpstrnsi machine pattern should not be used to implement the memcmp builtin function. This is because a string comparison will terminate if two zero bytes are read whereas a memory comparison should continue. OK. Kindof surprised this wasn't deal

Commit: RX: Add support for conditional register moves

2011-08-11 Thread Nick Clifton
Hi Guys, I am applying the patch below on behalf of Renesas. It adds support to the RX backend for conditional register moves. Tested without any regressions on an rx-elf toolchain. Cheers Nick gcc/ChangeLog 2011-08-11 Kazuhiro Inaoka * config/rx/rx.md (movsicc): Allow reg

RFA: Do not use cmpstrnsi to implement builtin memcmp

2011-08-10 Thread Nick Clifton
2011-08-17 Nick Clifton * builtins.c (expand_builtin_memcmp): Do not use cmpstrnsi pattern. * doc/md.texi (cmpstrn): Note that the comparison stops if both fetched bytes are zero. (cmpstr): Likewise. (cmpmem): Note that the comparison does not stop if bo

Re: Commit: RX: Disable extender peepholes at -O3

2011-08-09 Thread Nick Clifton
Hi Paul, It disables the peephole optimizations in the rx.md file that combine a load followed by a zero- or sign- extend operation. The disabling only happens at -O3 (or higher) as although the peepholes reduce the number of instructions they can introduce pipeline stalls that actual

Commit: RX: Disable extender peepholes at -O3

2011-08-09 Thread Nick Clifton
number of instructions they can introduce pipeline stalls that actually make the program slower. (This is certainly true for the coremark test). Cheers Nick gcc/ChangeLog 2011-08-09 Nick Clifton * config/rx/rx.md: Disable extender peepholes at -O3. Index: gcc/config/rx/rx.md

Ping: Re: [backport] arm,rx: don't ICE on naked functions with local vars

2011-08-09 Thread Nick Clifton
Hi Richard, DJ is on vacation at the moment, but he asked me to ping you about backporting this patch: http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02321.html to the 4.5 and 4.6 branches: http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02683.html May I apply the patch there please ? Che

Re: [build] Move unwinder to toplevel libgcc (v2)

2011-08-03 Thread Nick Clifton
Hi Rainer, How should we proceed with this patch, especially given the quite moderate comments from most affected target maintainers? I have no objections to the patch from an ARM or FRV point of view. Cheers Nick

<    1   2   3   4   5   >