[Patch, avr] Add device name to cpp_builtins

device information themselves. If ok, could someone apply please? I don't have commit access. Regards Senthil 2014-07-23 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * config/avr/avr-c.c (avr_cpu_cpp_builtins): Add __AVR_DEVICE_NAME__. diff --git gcc/config/avr/avr-c.c gcc

Re: [Patch, avr] Add device name to cpp_builtins

On Mon, Jul 28, 2014 at 08:09:28PM +0200, Georg-Johann Lay wrote: Senthil Kumar Selvaraj schrieb: The below patch adds a new preprocessor define for the device name (__AVR_DEVICE_NAME__) that was passed to the compiler. While the device name macro (say __AVR_ATmega128__) can be used to check

[Patch, avr] Propagate -mrelax gcc driver flag to assembler

is already committed in the binutils repo. I'm not sure how to manage a running a newer gcc with an older version of binutils though - how is this generally handled? If ok, could someone commit please? I don't have commit access. Regards Senthil 2014-04-11 Senthil Kumar Selvaraj

Re: [Patch, avr] Propagate -mrelax gcc driver flag to assembler

On Sat, Apr 12, 2014 at 06:36:01PM +0200, Georg-Johann Lay wrote: Senthil Kumar Selvaraj schrieb: This patch modifies AVR target's ASM spec to pass -mlink-relax to the assembler if -mrelax is passed to the compiler driver. This was already being passed on to the linker, this patch merely

[Patch, avr] Fix PR60991

be great if this was also committed to 4.9 and 4.8 branches as well. Regards Senthil gcc/ChangeLog 2014-05-12 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com PR target/60991 * config/avr/avr.c (avr_out_store_psi): Use correct constant to restore Y. gcc/testsuite

Re: [Patch, avr] Propagate -mrelax gcc driver flag to assembler

Ping! Regards Senthil On Fri, Apr 18, 2014 at 03:22:46PM +0530, Senthil Kumar Selvaraj wrote: On Sat, Apr 12, 2014 at 06:36:01PM +0200, Georg-Johann Lay wrote: Senthil Kumar Selvaraj schrieb: This patch modifies AVR target's ASM spec to pass -mlink-relax to the assembler if -mrelax

Re: [Patch, avr] Propagate -mrelax gcc driver flag to assembler

On Mon, May 12, 2014 at 01:19:37PM +0200, Georg-Johann Lay wrote: Am 04/18/2014 11:52 AM, schrieb Senthil Kumar Selvaraj: On Sat, Apr 12, 2014 at 06:36:01PM +0200, Georg-Johann Lay wrote: Senthil Kumar Selvaraj schrieb: This patch modifies AVR target's ASM spec to pass -mlink-relax

Re: [Patch, avr] Propagate -mrelax gcc driver flag to assembler

ASM_SPEC %:device_to_as(%{mmcu=*:%*}) LINK_RELAX_SPEC Does this look ok? I don't have commit access, so could someone commit this please? Regards Senthil 2014-05-15 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * config/avr/avr.h: Pass on mlink-relax to assembler

Re: [Patch, avr] Propagate -mrelax gcc driver flag to assembler

On Fri, May 16, 2014 at 12:02:12PM +0200, Georg-Johann Lay wrote: Am 05/15/2014 09:55 AM, schrieb Senthil Kumar Selvaraj: On Wed, May 14, 2014 at 12:56:54PM +0200, Rainer Orth wrote: Georg-Johann Lay a...@gjlay.de writes: Or what about simply that, which works for me: Index: config/avr

Fix address space computation in expand_debug_expr

For the AVR target, assertions in convert_debug_memory_address cause a couple of ICEs (see PR 52472). Jakub suggested returning a NULL rtx, which works, but on debugging further, I found that expand_debug_expr appears to incorrectly compute the address space for ADDR_EXPR and MEM_REFs. For

Re: Fix address space computation in expand_debug_expr

On Thu, Jun 05, 2014 at 09:46:25AM +0200, Richard Biener wrote: On Wed, Jun 4, 2014 at 10:06 PM, Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com wrote: For the AVR target, assertions in convert_debug_memory_address cause a couple of ICEs (see PR 52472). Jakub suggested returning

Fix broken build for AVR and SPU targets

. Regards Senthil gcc/ChangeLog 2014-02-12 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * config/avr/avr-c.c (avr_resolve_overloaded_builtin): Pass vNULL for arg_loc. * config/spu/spu-c.c (spu_resolve_overloaded_builtin): Likewise. diff --git gcc/config/avr/avr-c.c gcc

[Testsuite] Skip torture/pr60183.c for AVR target

gcc/testsuite/ChangeLog 2014-02-17 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * gcc.dg/torture/pr60183.c: Skip for avr target. diff --git gcc/testsuite/gcc.dg/torture/pr60183.c gcc/testsuite/gcc.dg/torture/pr60183.c index d37b4b8..8e519dc 100644 --- gcc/testsuite/gcc.dg

[Patch, avr] Remove atxmega16x1 from device list

The atxmega16x1 AVR variant doesn't exist, but the device name got into the device list in gcc/config/avr/avr-mcus.def. This patch removes the non-existent device. If ok, could someone commit please? I don't have commit access. Regards Senthil 2014-03-03 Senthil Kumar Selvaraj

Re: [patch,avr] Fix PR59396: Ignore leading '*' in warning generation for ISR names

On Thu, Mar 13, 2014 at 02:24:06PM +0100, Georg-Johann Lay wrote: Problem is that the assembler name might or might not be prefixed by '*' depending on when TARGET_SET_CURRENT_FUNCTION is called. The change is just to fix wrong warning because the current implementation of

Emitting correct DWARF location descriptor for multi-reg frame pointer

As a follow-up to this discussion (http://gcc.gnu.org/ml/gcc/2012-11/msg00307.html), I have a patch that makes dwarf2out.c emit correct location information for a FP register spanning multiple hard regs. Before the patch, the DW_AT_location for a local variable on the AVR target (whose FP spans

[PATCH] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

This patch allows cflags set in board config files using set_board_info cflags to be used in the selectors of dg-skip-if and other dejagnu commands that use the check-flags proc. The code merely adds cflags to compiler_flags in the check-flags proc, exactly the same way as multilib_flags is

Re: [PING][PATCH,dejagnu] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

Hello On Sat, Aug 11, 2012 at 11:09:03PM +0530, Senthil Kumar Selvaraj wrote: This patch allows cflags set in board config files using set_board_info cflags to be used in the selectors of dg-skip-if and other dejagnu commands that use the check-flags proc. The code merely adds cflags

Re: [PATCH] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

On Tue, Aug 21, 2012 at 03:08:43PM -0700, Mike Stump wrote: On Aug 11, 2012, at 10:39 AM, Senthil Kumar Selvaraj wrote: This patch allows cflags set in board config files using set_board_info cflags to be used in the selectors of dg-skip-if and other dejagnu commands that use the check

Re: [Patch] Fix infinite loop/crash if array initializer index equals max value

On Fri, Aug 23, 2013 at 09:49:55PM +, Joseph S. Myers wrote: On Thu, 22 Aug 2013, Selvaraj, Senthil_Kumar wrote: 2013-08-23 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * c-typeck.c (output_pending_init_elements): Handle overflow of constructor_unfilled_index

[Patch, avr] Emit diagnostics if -f{pic,PIC,pie,PIE} or -shared is passed

is provided ( config/mep/mep.c and config/s390/tpf.h already do something very similar). Regression tested with no new failures.Tests which exercise PIC now report as unsupported. If ok, could someone commit please? I don't have commit access. Regards Senthil gcc/ChangeLog 2013-11-04 Senthil Kumar

[Patch, testsuite] Require scheduling support in gcc.dg/superblock.c

do not have commit access. Regards Senthil gcc/testsuite 2013-11-04 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * gcc.dg/superblock.c: Require scheduling support diff --git gcc/testsuite/gcc.dg/superblock.c gcc/testsuite/gcc.dg/superblock.c index 2b9aedf..272d161 100644

[Ping, avr] Emit diagnostics if -f{pic,PIC,pie,PIE} or -shared is passed

Ping! Regards Senthil On Mon, Nov 04, 2013 at 06:45:19PM +0530, Senthil Kumar Selvaraj wrote: The AVR backend does not generate position independent code, yet it happily accepts -fpic, -fPIC, -fpie and -fPIE. The generated code doesn't change at all. Also, it accepts the -shared option

[Ping] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

Hello, Could one of the maintainers please check in the patch below? It was already approved at http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00689.html Regards Senthil On Tue, Aug 21, 2012 at 03:08:43PM -0700, Mike Stump wrote: On Aug 11, 2012, at 10:39 AM, Senthil Kumar Selvaraj wrote

Re: [Patch] Emit error for negative _Alignas alignment values

On Wed, Apr 24, 2013 at 03:18:51PM +, Joseph S. Myers wrote: On Wed, 3 Apr 2013, Senthil Kumar Selvaraj wrote: 2013-04-03 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * c-common.c (check_user_alignment): Emit error for negative values * gcc.dg/c1x-align

[Patch, testsuite] Add -gdwarf to debug/dwarf2 testcases (Take 2)

. Regards Senthil gcc/testsuite/ChangeLog 2013-04-29 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * gcc.dg/debug/dwarf2/dwarf2.exp: Replace -gdwarf-2 with -gdwarf and force -gdwarf when invoking dg-runtest. diff --git gcc/testsuite/gcc.dg/debug/dwarf2/dwarf2.exp gcc

[Patch, testsuite] Require scheduling support for test that uses -fschedule-insns

/ChangeLog 2013-05-01 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * gcc.dg/torture/alias-1.c: Add dg-require-effective-target scheduling. diff --git gcc/testsuite/gcc.dg/torture/alias-1.c gcc/testsuite/gcc.dg/torture/alias-1.c index 1e60341..92b8809 100644 --- gcc

[Ping, Patch, testsuite] Require scheduling support for test that uses -fschedule-insns

Could someone commit this patch please? I don't have commit access. Regards Senthil On Wed, May 01, 2013 at 05:04:18PM -0700, Mike Stump wrote: On May 1, 2013, at 2:40 AM, Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com wrote: gcc.dg/torture/alias-1.c specifies -fschedule-insns

[Patch] Emit error for negative _Alignas alignment values

for a negative power of 2. Bootstrapped and regression tested with x86_64-unknown-linux-gnu with no new failures. If ok, could someone commit please? I don't have commit access. Regards Senthil ChangeLog 2013-04-03 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * c

[Patch] Add -gdwarf option to make gcc generate DWARF with the default version

tests with the -gdwarf option added to dg-options. Regards Senthil gcc/ChangeLog 2013-04-10 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com Jason Merrill ja...@redhat.com diff --git gcc/common.opt gcc/common.opt index e02e7ed..e3645c3 100644 --- gcc/common.opt +++ gcc

Re: [Patch] Add -gdwarf option to make gcc generate DWARF with the default version

On Thu, Apr 11, 2013 at 09:08:08AM +0200, Jakub Jelinek wrote: On Wed, Apr 10, 2013 at 06:55:10PM +0530, Senthil Kumar Selvaraj wrote: --- gcc/opts.c +++ gcc/opts.c @@ -1699,6 +1699,18 @@ common_handle_option (struct gcc_options *opts, set_debug_level (SDB_DEBUG, false, arg, opts

[Patch, testsuite] Add -gdwarf to dg-options in debug/dwarf2 testcases

is not the default. If ok, could someone commit please? I don't have commit access. Regards Senthil gcc/testsuite/ChangeLog 2013-04-11 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * gcc.dg/debug/dwarf2/global-used-types.c: Add -gdwarf to dg-options * gcc.dg/debug/dwarf2/inline2.c

[Ping, Patch, c-common] Emit error for negative _Alignas alignment values

Ping again! Regards Senthil On Wed, Apr 03, 2013 at 07:18:20PM +0530, Senthil Kumar Selvaraj wrote: This patch detects and emits an error if the value provided in _Alignas is negative. The fix was approved pending full regression testing in a previous discussion (http://gcc.gnu.org/ml/gcc

[Ping, Patch, testsuite] Add -gdwarf to dg-options in debug/dwarf2 testcases

Ping. http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00665.html Regards Senthil On Thu, Apr 11, 2013 at 03:36:03PM +0530, Senthil Kumar Selvaraj wrote: Hi, This patch adds the new -gdwarf option to dg-options for testcases in gcc.dg/debug/dwarf2 that don't already explicitly request DWARF

Re: [Ping, Patch, testsuite] Add -gdwarf to dg-options in debug/dwarf2 testcases

On Thu, Apr 18, 2013 at 04:18:31PM +0200, Jakub Jelinek wrote: On Thu, Apr 18, 2013 at 07:46:30PM +0530, Senthil Kumar Selvaraj wrote: Ping. http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00665.html Shouldn't dwarf2.exp just prepend that to options instead? It does it if the testcase

[Patch, regcprop] Tentative fix for PR 64331

approach? What do you guys think? Regards Senthil ChangeLog 2014-12-16 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com PR rtl-optimization/64331 * regcprop.c (copyprop_hardreg_forward_1): Kill regs marked REG_DEAD. diff --git gcc/regcprop.c gcc/regcprop.c index daeb980

Re: [patch,avr]: Fix various problems with specs and specs file generation.

On Mon, Mar 02, 2015 at 08:40:17PM +0100, Georg-Johann Lay wrote: The new specs file generation introduces several problems. This patch - Fix build warnings - Clean up unused code and the old, now dead specs definitions. - Removes unused files and adjust build scripts / rules. -

Re: [patch, avr] Fix PR64331: insn output and insn length computation rely on REG_DEAD notes.

On Mon, Feb 23, 2015 at 11:53:52AM +0100, Georg-Johann Lay wrote: This patch fixes PR64331 which produced wrong code because of outdated (too many) REG_DEAD notes. These notes are not (re)computed per default, hence do the computation by hand each time avr.c:reg_unused_after is called in a

Re: [patch,avr] Fix PR 65657 - read from __memx address space tramples arguments to function call

On Fri, Apr 17, 2015 at 01:16:58PM +0530, Senthil Kumar Selvaraj wrote: On Thu, Apr 16, 2015 at 06:47:26PM +0200, Georg-Johann Lay wrote: Am 04/16/2015 um 11:28 AM schrieb Senthil Kumar Selvaraj: On Thu, Apr 16, 2015 at 11:02:05AM +0200, Georg-Johann Lay wrote: Am 04/16/2015 um 08:43 AM

[patch,avr] Fix PR 65657 - read from __memx address space tramples arguments to function call

this? If not, could someone commit this please? I don't have commit access. Regards Senthil gcc/ChangeLog 2015-04-16 Senthil Kumar Selvaraj senthil_kumar.selva...@atmel.com * config/avr/avr.c (avr_rtx_costs_1): Increase cost for MEM rtx in non-default address space. gcc

Re: [patch,avr] Fix PR 65657 - read from __memx address space tramples arguments to function call

On Thu, Apr 16, 2015 at 11:02:05AM +0200, Georg-Johann Lay wrote: Am 04/16/2015 um 08:43 AM schrieb Senthil Kumar Selvaraj: This patch fixes PR 65657. The following artifact appears to be PR63633. I did see that one - unfortunately, that fix won't help here. IIUC, you check if input/output

Re: [patch,avr] Fix PR 65657 - read from __memx address space tramples arguments to function call

On Thu, Apr 16, 2015 at 06:47:26PM +0200, Georg-Johann Lay wrote: Am 04/16/2015 um 11:28 AM schrieb Senthil Kumar Selvaraj: On Thu, Apr 16, 2015 at 11:02:05AM +0200, Georg-Johann Lay wrote: Am 04/16/2015 um 08:43 AM schrieb Senthil Kumar Selvaraj: This patch fixes PR 65657. The following

Re: [dwarf, RFC] Emitting per-function dwarf info

Ping! Regards Senthil On Fri, Apr 10, 2015 at 12:19:36PM +0530, Senthil Kumar Selvaraj wrote: Hi, This (rather big) patch is an attempt to generate per function DWARF information for functions that go into their own sections (through -ffunction-section or otherwise). This is so

Re: [Patch, avr] Fix PR 67839 - bit addressable instructions generated for out of range addresses

Ping! Regards Senthil On Fri, Oct 16, 2015 at 03:17:17PM +0530, Senthil Kumar Selvaraj wrote: > Ping! > > Regards > Senthil > > On Mon, Oct 05, 2015 at 02:30:58PM +0530, Senthil Kumar Selvaraj wrote: > > Hi, > > > > As part of support for io an

Re: [Patch, vrp] Allow VRP type conversion folding only for widenings upto word mode

On Sat, Nov 14, 2015 at 09:13:41AM +0100, Marc Glisse wrote: > On Sat, 14 Nov 2015, Senthil Kumar Selvaraj wrote: > > >This patch came out of a discussion held in the gcc mailing list > >(https://gcc.gnu.org/ml/gcc/2015-11/msg00067.html). > > > >The patch restrict

[Patch, vrp] Allow VRP type conversion folding only for widenings upto word mode

. Bootstrapped and reg tested on x86_64 (with --enable-languages=c,c++). If ok, could you commit please? I don't have commit access. Regards Senthil gcc/ChangeLog 2015-11-11 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * tree-vrp.c (simplify_cond_using_ranges): Fol

Re: [Patch, avr] Fix PR 67839 - bit addressable instructions generated for out of range addresses

Ping! Regards Senthil On Mon, Oct 05, 2015 at 02:30:58PM +0530, Senthil Kumar Selvaraj wrote: > Hi, > > As part of support for io and io_low attributes, the upper bound of > the range check for low IO and IO addresses was changed from hardcoded > values to hardcode

Re: [Patch, vrp] Allow VRP type conversion folding only for widenings upto word mode

On Mon, Nov 16, 2015 at 10:02:15AM +0100, Richard Biener wrote: > On Sat, 14 Nov 2015, Senthil Kumar Selvaraj wrote: > > > On Sat, Nov 14, 2015 at 09:57:40AM +0100, Richard Biener wrote: > > > On November 14, 2015 9:49:28 AM GMT+01:00, Senthil Kumar Selvaraj >

Re: [Patch, vrp] Allow VRP type conversion folding only for widenings upto word mode

On Sat, Nov 14, 2015 at 09:57:40AM +0100, Richard Biener wrote: > On November 14, 2015 9:49:28 AM GMT+01:00, Senthil Kumar Selvaraj > <senthil_kumar.selva...@atmel.com> wrote: > >On Sat, Nov 14, 2015 at 09:13:41AM +0100, Marc Glisse wrote: > >> On Sat, 14 Nov 2015, S

[Patch, avr] Fix PR65210

Hi, This (rather trivial) patch fixes PR65210. The ICE happens because code wasn't handling io_low attribute where it is supposed to. If this is ok, could someone commit please? I don't have commit access. Regards Senthil gcc/ChangeLog 2015-09-02 Senthil Kumar Selvaraj

Re: [Patch, avr] Fix PR65210

On Tue, Sep 08, 2015 at 09:16:35AM +0200, Georg-Johann Lay wrote: > Senthil Kumar Selvaraj schrieb: > > This (rather trivial) patch fixes PR65210. The ICE happens because code > > wasn't handling io_low attribute where it is supposed to. > > Hi Senthil, could you line o

Re: [Patch, testsuite] Skip addr_equal-1 if target keeps null pointer checks

On Mon, Sep 28, 2015 at 01:38:18PM -0600, Jeff Law wrote: > On 09/28/2015 02:15 AM, Senthil Kumar Selvaraj wrote: > >Hi, > > > > The below patch skips gcc.dg/addr_equal-1.c if the target keeps null > > pointer checks. > > > > The test fails for

[Patch, avr] Fix PR 67839 - bit addressable instructions generated for out of range addresses

Selvaraj <senthil_kumar.selva...@atmel.com> PR target/67839 * config/avr/predicates.md (low_io_address_operand): Don't consider MODE when computing upper bound. (io_address_operand): Likewise. gcc/testsuite/ChangeLog 2015-10-05 Senthil Kumar Se

[Patch, testsuite] Skip addr_equal-1 if target keeps null pointer checks

the builtin_constant_p to a false value, resulting in the test failure. If this is ok, could someone commit please? I don't have commit access. Regards Senthil gcc/testsuite/ChangeLog 2015-09-28 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.dg/addr_equal-1.c: Skip test if

[Patch, avr] Provide correct memory move costs

gcc/ChangeLog 2015-12-16 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * config/avr/avr.h (MOVE_MAX): Set value to 1. (MOVE_MAX_PIECES): Define. (MOVE_RATIO): Define. * config/avr/avr.c (TARGET_USE_BY_PIECES_INFRASTRUCTURE_P): Provide

Re: [Patch, vrp] Allow VRP type conversion folding only for widenings upto word mode

On Wed, Nov 18, 2015 at 09:36:21AM +0100, Richard Biener wrote: > On Wed, 18 Nov 2015, Senthil Kumar Selvaraj wrote: > > > On Mon, Nov 16, 2015 at 10:02:15AM +0100, Richard Biener wrote: > > > On Sat, 14 Nov 2015, Senthil Kumar Selvaraj wrote: > > > > > &g

Re: [Patch, vrp] Allow VRP type conversion folding only for widenings upto word mode

On Thu, Nov 19, 2015 at 10:31:41AM -0700, Jeff Law wrote: > On 11/18/2015 11:20 PM, Senthil Kumar Selvaraj wrote: > >On Wed, Nov 18, 2015 at 09:36:21AM +0100, Richard Biener wrote: > >> > >>Otherwise ok. > > > >See modified patch below. If you think vrp98

[Patch, testsuite] Skip some more tests for targets with int size < 32

? I don't have commit access. Regards Senthil gcc/testsuite/ChangeLog 2016-06-08 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.c-torture/execute/bswap-2.c: Require int32plus. * gcc.dg/torture/pr68067-1.c: Likewise. * gcc.dg/torture/pr680

[Patch, avr] Fix broken stack-usage-1.c test

for making gcc.dg/stack-usage-1.c pass again for avr. If this is ok, could someone commit please? I don't have commit access. Regards Senthil gcc/testsuite/ChangeLog 2016-06-08 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.dg/stack-usage-1.c (SIZE): Consider

Re: [Patch, avr] Fix PR 71151

Georg-Johann Lay writes: > Senthil Kumar Selvaraj schrieb: >> Hi, >> >> This patch fixes PR 71151 by eliminating the >> TARGET_ASM_FUNCTION_RODATA_SECTION hook and setting >> JUMP_TABLES_IN_TEXT_SECTION to 1. >> >> As described in the

[Patch, avr] Fix PR 71151

with that some more. If ok, could someone commit please? Could you also backport to gcc-6-branch? Regards Senthil gcc/ChangeLog 2016-06-03 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * config/avr/avr.c (avr_asm_function_rodata_section): Remove. * config/avr

Re: [Patch, avr] Fix PR 71151

Georg-Johann Lay writes: > Senthil Kumar Selvaraj schrieb: >> Hi, >> >> This patch fixes PR 71151 by eliminating the >> TARGET_ASM_FUNCTION_RODATA_SECTION hook and setting >> JUMP_TABLES_IN_TEXT_SECTION to 1. >> >> As described in the

MAINTAINERS (Write After Approval): Add myself

Added myself to MAINTAINERS (Write After Approval). Regards Senthil Index: ChangeLog === --- ChangeLog (revision 237571) +++ ChangeLog (working copy) @@ -1,3 +1,7 @@ +2016-06-18 Senthil Kumar Selvaraj <senthil_kumar.se

Re: [Patch, avr] Fix PR 71151

Georg-Johann Lay writes: > Senthil Kumar Selvaraj schrieb: >> Senthil Kumar Selvaraj writes: >> >>> Georg-Johann Lay writes: >>> >>>> Senthil Kumar Selvaraj schrieb: >>>>> Hi, >>>>> >>>>> [set JUMP_TA

Re: [patch,avr]: ad PR71151: Make test cases pass on smaller targets.

Georg-Johann Lay writes: > On 22.06.2016 19:06, Mike Stump wrote: >> On Jun 22, 2016, at 7:21 AM, Georg-Johann Lay wrote: >>> >>> Some tests for PR71151 assume that the target MCU has a 3-byte PC. The >>> tests are failing because the simulator (avrtest) rejects to load the >>> respective

Re: [Patch, avr] Fix PR 71151

Denis Chertykov writes: > 2016-06-16 10:27 GMT+03:00 Senthil Kumar Selvaraj > <senthil_kumar.selva...@atmel.com>: >> >> Senthil Kumar Selvaraj writes: >> >>> Georg-Johann Lay writes: >>> >>>> Senthil Kumar Selvaraj schrieb: >

Re: [RFC][PATCH, ARM 7/8] ARMv8-M Security Extension's cmse_nonsecure_call: use __gnu_cmse_nonsecure_call]

User-agent: mu4e 0.9.13; emacs 24.5.1 Hi, Apologies for the bad posting style (I don't have the original email handy), but shouldn't _gnu_cmse_nonsecure_call be defined with the .global directive in the below hunk (to make it visible when linking)? diff --git

Re: [Patch, avr] Fix PR 71151

Senthil Kumar Selvaraj writes: > Georg-Johann Lay writes: > >> Senthil Kumar Selvaraj schrieb: >>> Hi, >>> >>> This patch fixes PR 71151 by eliminating the >>> TARGET_ASM_FUNCTION_RODATA_SECTION hook and setting >>> JUMP_TABLES_IN_T

Re: PATCH] Fix PR 31531: A microoptimization of isnegative of signed integer

Hurugalawadi, Naveen writes: > Hi, > > Please find attached the patch that performs optimization on unsigned values. > > Original fold-const part implemented in match.pd. > > Please review the patch and let us know if it's OK? > > Regression Tested on X86_64 with no regressions. > > Thanks, >

[Patch, testsuite] Require int32 target support in sso tests

machine. If this patch is ok, could someone commit please? I don't have commit access. Regards Senthil gcc/testsuite/ChangeLog 2016-02-04 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc/testsuite/gcc.dg/sso/p1.c: Add dg-require-effective-target int32. * gcc/tes

[Patch, avr] Restore default value of PARAM_ALLOW_STORE_DATA_RACES to 1

with interrupts will need special handling for atomic access anyway, so I thought we should revert the default back to allow store data races. If this is ok, could someone commit please? I don't have commit access. Regards Senthil gcc/ChangeLog 2016-02-01 Senthil Kumar Selvaraj <senthil_kumar

Re: [Patch, testsuite] Require int32 target support in sso tests

Mike Stump writes: > On Feb 22, 2016, at 12:41 AM, Senthil Kumar Selvaraj > <senthil_kumar.selva...@atmel.com> wrote: >> Could someone commit it for me please? I don't have commit access. > > Could you test

Re: [Patch, testsuite] Require int32 target support in sso tests

Mike Stump okayed the patch (https://gcc.gnu.org/ml/gcc-patches/2016-02/msg00321.html). Could someone commit it for me please? I don't have commit access. Regards Senthil Senthil Kumar Selvaraj writes: > Hi, > > When running the regression testsuite for the AVR target, I noticed a

[Patch, avr]Fix multiple ICE fallout of PR 69764

2016-03-10 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * config/avr/avr.md (rotl3): Set mode for operand 2. diff --git gcc/config/avr/avr.md gcc/config/avr/avr.md index ff26f2e..c988446 100644 --- gcc/config/avr/avr.md +++ gcc/config/avr/avr.md @@ -3351,7 +

[Patch, testsuite] Skip testcase for avr

don't have commit access. Regards Senthil gcc/testsuite/ChangeLog 2016-03-16 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.c-torture/compile/20151204.c: Skip for avr. diff --git a/gcc/testsuite/gcc.c-torture/compile/20151204.c b/gcc/testsuite/gcc.c-torture/compile/

Re: [Patch, testsuite] Skip testcase for avr

ite/gcc.c-torture/compile/20151204.c >> @@ -1,3 +1,5 @@ >> +/* { dg-skip-if "Array too big" { "avr-*-*" } { "*" } { "" } } */ > > just one nit: please omit the default args to dg-skip-if, they are > unnecessary. > Done. gcc/tes

[Patch] Fix PR 60040

. If ok, could someone commit please? I don't have commit access. Regards Senthil gcc/ChangeLog 2016-04-07 Joern Rennecke <g...@amylaar.uk> Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> PR target/60040 * reload1.c (find_reload_regs): A

Re: [Patch, testsuite] Require int32plus and scheduling support for some tests

Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Hi, > > This patch add dg-require-effective-target directives to a few tests > that were failing unnecessarily for the AVR target. > > One of them invokes the compiler with -fschedule-insns2 - I've > requir

[Patch, testsuite] Require int32plus and scheduling support for some tests

- they either use bit shifts wider than 16 bits (AVR's int size), or use int constants that are too big for a 16 bit int. If ok, could someone commit please? I don't have commit access. Regards Senthil 2016-04-04 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.c-t

Re: [Patch] Fix PR 60040

Bernd Schmidt writes: > On 04/15/2016 02:52 PM, Senthil Kumar Selvaraj wrote: >> >> For both testcases in the PR, reload fails to take into account that >> FP-SP elimination can no longer be performed, and tries to find reload >> regs for an rtx generated when

Re: [Patch] Fix PR 60040

Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Bernd Schmidt writes: > >> On 04/07/2016 01:52 PM, Senthil Kumar Selvaraj wrote: >>>The below patch fixes PR 60040 by not halting with a hard error on >>>a spill failure, if reload knows that it has t

Re: [Patch, avr] Include INCOMING_FRAME_SP_OFFSET when printing stack usage

Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Hi, > > This trivial patch adds INCOMING_FRAME_SP_OFFSET to > current_function_static_stack_size, thus fixing the 2 (or 3, for > 3 byte PC devices) byte difference between reported and actual > values when u

[Patch, avr] Include INCOMING_FRAME_SP_OFFSET when printing stack usage

when setting current_function_static_stack_size. If this is ok, could someone commit please? I don't have commit access. Regards Senthil gcc/ChangeLog 2016-05-13 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * config/avr/avr.c (avr_expand_prologue

Re: [Patch] Fix PR 60040

Bernd Schmidt writes: > On 04/07/2016 01:52 PM, Senthil Kumar Selvaraj wrote: >>The below patch fixes PR 60040 by not halting with a hard error on >>a spill failure, if reload knows that it has to run again anyway. > > Some additional information as to how this situ

Re: [Patch] Fix PR 60040

Joern Wolfgang Rennecke writes: > On 28/04/16 07:57, Senthil Kumar Selvaraj wrote: >> diff --git libcilkrts/ChangeLog libcilkrts/ChangeLog >> index 8fada8a..ed26a3a 100644 >> --- libcilkrts/ChangeLog >> +++ libcilkrts/ChangeLog >> @@ -1,9 +1,3 @@ >> -201

[Patch, testsuite] Make some more tests xfail/pass/unsupported for avr

have commit access. Regards Senthil 2016-05-25 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * c-c++-common/Wduplicated-cond-1.c: Use smaller const literal. * c-c++-common/pr60226.c: Require int32plus. * gcc.c-torture/execute/pr70602.c: Li

Re: [Patch, testuite, committed] Fix some more tests that fail for non 32-bit int targets

Mike Stump writes: > On Jul 25, 2016, at 5:00 AM, Senthil Kumar Selvaraj > <senthil_kumar.selva...@atmel.com> wrote: >> >> The below patch fixes tests that fail for the avr target, because they >> assume ints are atleast 32 bits wide and pointers an

[Patch, testuite, committed] Fix some more tests that fail for non 32-bit int targets

<-> int size difference warning. Reg tested on avr and x86_64 with no regressions. Committed as obvious. Regards Senthil gcc/testsuite/ChangeLog 2016-07-25 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.dg/torture/pr69352.c (foo): Cast to intptr_t ins

[Patch, tentative, reload] Make push_reload work with more types of subregs?

Hi, When analyzing PR 71873 (ICE in push_reload), I found that that code in push_reload that recursively calls push_reload for subreg expressions doesn't correctly set subreg_in_class for a few cases. Specifically, reload_inner_reg_of_subreg returns true if SUBREG_REG(x) is CONSTANT_P

Re: [patch,avr] PR70677: Use -fno-caller-saves for avr

Georg-Johann Lay writes: > On 02.08.2016 06:50, Senthil Kumar Selvaraj wrote: >> >> Denis Chertykov writes: >> >>> 2016-08-01 15:17 GMT+03:00 Georg-Johann Lay <a...@gjlay.de>: >>>> Problem with -fcaller-saves is that there are situations where i

Re: [patch,avr] PR70677: Use -fno-caller-saves for avr

Denis Chertykov writes: > 2016-08-01 15:17 GMT+03:00 Georg-Johann Lay : >> Problem with -fcaller-saves is that there are situations where it triggers >> an expensive frame just to store a variable around a function call even >> though there are plenty of call-saved registers. >>

[Patch, testsuite] Fix some more bogus failures for avr

Hi, Committed below patch to trunk as obvious. Regards Senthil 2016-08-03 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.dg/init-excess-2.c: Require int32plus. * gcc.dg/pr44024.c: Skip if target keeps null pointer checks. * gcc.dg/pr599

Re: [Patch, tentative, reload] Make push_reload work with more types of subregs?

Bernd Schmidt writes: > On 08/08/2016 05:26 PM, Senthil Kumar Selvaraj wrote: > >> I picked out the commit where you'd added SYMBOL_REF handling (rev >> #190252), and patched that with the below code. Bootstrapped and >> regtested on x86_64-pc-linux - th

[Patch, testsuite] Skip tests that expect 4 byte alignment for avr

? Regards Senthil gcc/testsuite/ChangeLog: 2016-08-11 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.dg/ipa/propalign-1.c: Skip for targets with !natural_alignment_32 and !natural_alignment_64. * gcc.dg/ipa/propalign-2.c: Likewise. * gcc.

Re: [patch,avr] minor tweaks for 8-bit operations

Georg-Johann Lay writes: > This patch contains some unrelated tweaks > > - Supplying no-ldregs variant for andqi3, iorqi3 where a const_int mask > affects > only 1 bit > > - Some patterns that match situations with zero_extend that can be performed > with less instructions / register

[Patch, testsuite, committed] Fix gcc.dg/params/blocksort-part.c for non 32-bit int targets

Hi, The below patch conditionally defines Int32 and UInt32 to accomodate targets with sizeof(int) != 4. Regtested with x86_64 and avr. Committed as obvious. Regards Senthil 2016-07-19 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> * gcc.dg/params/blocksort-

[Patch, testsuite, tentative] Explicitly disable pointer <-> int cast warnings for avr?

Hi, The patch fixes a couple of testsuite failures that show up for the avr target because it has different sizes for longs and pointers (4 bytes versus 2), by explicitly disabling the warning for avr. Does this make sense? Skipping the test by requiring ptr32plus would have worked,

[Patch, testsuite] Fix some bogus testsuite failures for avr

Hi, This patch requires int32plus and ptr32plus for a couple of tests, tweaks Wduplicated-cond-3.c to use a smaller constant that fits in 16 bits, and marks one test as too big for avr. Committed to trunk. Regards Senthil 2016-07-13 Senthil Kumar Selvaraj <senthil_kumar.se

Re: [Patch, avr] Fix PR 50739 - nameless error with -fmerge-all-constants

Senthil Kumar Selvaraj writes: > Hi, > > This patch fixes a problem with fmerge-all-constants and the progmem > attribute - on trunk, the below testcase errors out with a section > conflict error. > > When avr_asm_select_section renames .rodata.xyz section to >

[Patch, avr] Fix PR 50739 - nameless error with -fmerge-all-constants

by switch_section anyway. Reg testing showed no new regressions. Ok for trunk and backport to 6.x? Regards Senthil gcc/testsuite/ChangeLog: 2016-07-05 Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> PR target/50739 * gcc.target/avr/pr50739.c: New test.

  1   2   >