[Patch, testsuite] Fix sra-13.c for 16 bit int

2013-04-01 Thread Pitchumani Sivanupandi
Fix test case sra-13.c that assumed int is always 4 bytes. Regards, Pitchumani 2013-04-01 Pitchumani Sivanupandi pitchuman...@atmel.com testsuite * gcc.dg/tree-ssa/sra-13.c: Fix for 16 bit int --- gcc/testsuite/gcc.dg/tree-ssa/sra-13.c (revision 197081) +++ gcc/testsuite/gcc.dg

Re: [Patch, testsuite] Fix sra-13.c for 16 bit int

2013-04-02 Thread Pitchumani Sivanupandi
could someone please review and commit if it is OK? I don't have commit access. Regards, Pitchumani On 4/1/2013 4:30 PM, Pitchumani Sivanupandi wrote: Fix test case sra-13.c that assumed int is always 4 bytes. Regards, Pitchumani 2013-04-01 Pitchumani Sivanupandi pitchuman...@atmel.com

[patch, avr] Fix PR67353

2016-06-10 Thread Pitchumani Sivanupandi
) {}   ^~~ $ avr-gcc test.c -Wno-misspelled-isr $ If OK, could someone commit please? I do not have commit access. Regards, Pitchumani gcc/ChangeLog 2016-06-10  Pitchumani Sivanupandi  <pitchuman...@atmel.com>     * config/avr/avr.c (avr_set_current_function): Warn misspelled     interrupt/

Re: [patch, avr] Fix PR67353

2016-06-13 Thread Pitchumani Sivanupandi
On Fri, 2016-06-10 at 20:08 +0200, Georg-Johann Lay wrote: > Pitchumani Sivanupandi schrieb: > > > > Hi, > > > > This patch introduces new flags for warning 'misspelled interrupt/ > > signal handler'. Flag -Wmisspelled-isr is enabled by default and it > >

Re: [patch, avr] Fix PR67353

2016-06-15 Thread Pitchumani Sivanupandi
On Mon, 2016-06-13 at 17:48 +0200, Georg-Johann Lay wrote: > Pitchumani Sivanupandi schrieb: > > > > $ avr-gcc test.c -Wno-misspelled-isr > > $ > What about -Werror=misspelled-isr? Updated patch. > > > > diff --git a/gcc/config/avr/avr.c b/gcc/config/avr/avr

Re: [patch, avr] Fix PR67353

2016-06-15 Thread Pitchumani Sivanupandi
On Mon, 2016-06-13 at 17:48 +0200, Georg-Johann Lay wrote: > Pitchumani Sivanupandi schrieb: > > > > $ avr-gcc test.c -Wno-misspelled-isr > > $ > What about -Werror=misspelled-isr? Updated patch. > > > > diff --git a/gcc/config/avr/avr.c b/gcc/config/avr/avr

[patch, avr] PR 58655

2016-06-21 Thread Pitchumani Sivanupandi
Attached patches add documentation for -mfract-convert-truncate option and add that info to release notes (gcc-4.9 changes). If OK, could someone commit please? I do not have commit access. Regards, Pitchumani gcc/ChangeLog 2016-06-21  Pitchumani Sivanupandi  <pitchuman...@atmel.com>

Re: [patch, avr,wwwdocs] PR 58655

2016-06-22 Thread Pitchumani Sivanupandi
On Tuesday 21 June 2016 09:39 PM, Georg-Johann Lay wrote: Pitchumani Sivanupandi schrieb: Attached patches add documentation for -mfract-convert-truncate option and add that info to release notes (gcc-4.9 changes). If OK, could someone commit please? I do not have commit access. Regards

Ping: [patch, avr] Fix unrecognizable insn ICE for avr (PR71103)

2016-05-20 Thread Pitchumani Sivanupandi
ds, Pitchumani   gcc/ChangeLog 2016-05-20  Pitchumani Sivanupandi  <pitchuman...@atmel.com> PR target/71103 * config/avr/avr.md (define_expand "mov"): If the source operand is subreg (symbol_ref) then move the symbol ref to register. gcc/testsuite/ChangeLog 2016-05-20

[patch, avr] Fix unrecognizable insn ICE for avr (PR71103)

2016-05-13 Thread Pitchumani Sivanupandi
avr-gcc crashes for following test as it couldn't recognize the instruction pattern. struct st {   unsigned char uc1;   unsigned int *ui1; }; unsigned int ui1; struct st foo () {   struct st ret;   ret.uc1 = 6;   ret.ui1 =   return ret; } $ avr-gcc -mmcu=atmega328p -O1 test.c (-- snip --) 

Re: [patch, avr] Fix mmcu to specs issues

2016-07-28 Thread Pitchumani Sivanupandi
On Tuesday 26 July 2016 06:00 PM, Georg-Johann Lay wrote: On 26.07.2016 12:20, Pitchumani Sivanupandi wrote: avr-gcc expected to find the device specs in the search paths specified. But it doesn't work as expected when device specs in different place than the installed location. Example-1

Re: [patch, avr] Fix mmcu to specs issues

2016-07-29 Thread Pitchumani Sivanupandi
On Friday 29 July 2016 02:06 PM, Georg-Johann Lay wrote: On 28.07.2016 13:50, Pitchumani Sivanupandi wrote: On Tuesday 26 July 2016 06:00 PM, Georg-Johann Lay wrote: On 26.07.2016 12:20, Pitchumani Sivanupandi wrote: avr-gcc expected to find the device specs in the search paths specified

[patch, avr] Fix mmcu to specs issues

2016-07-26 Thread Pitchumani Sivanupandi
spec string "device-specs-file (device-specs/specs-%s)". New spec function for device-specs-file, that will check the access for device specs file and issue diagnostics. If Ok, could someone commit please? I do not have commit access. Regards, Pitchumani gcc/ChangeLog 2016-07-26

Re: [patch, avr,wwwdocs] PR 58655

2016-06-27 Thread Pitchumani Sivanupandi
Ping! On Wednesday 22 June 2016 12:05 PM, Pitchumani Sivanupandi wrote: On Tuesday 21 June 2016 09:39 PM, Georg-Johann Lay wrote: Pitchumani Sivanupandi schrieb: Attached patches add documentation for -mfract-convert-truncate option and add that info to release notes (gcc-4.9 changes). If OK

Re: [patch, avr] Fix mmcu to specs issues

2016-08-08 Thread Pitchumani Sivanupandi
Ping! On Friday 29 July 2016 05:14 PM, Pitchumani Sivanupandi wrote: On Friday 29 July 2016 02:06 PM, Georg-Johann Lay wrote: On 28.07.2016 13:50, Pitchumani Sivanupandi wrote: On Tuesday 26 July 2016 06:00 PM, Georg-Johann Lay wrote: On 26.07.2016 12:20, Pitchumani Sivanupandi wrote: avr

[patch, avr, pr71676 and pr71678] Issues with casesi expand

2016-09-26 Thread Pitchumani Sivanupandi
) (reg:HI 45))) Fix of PR71676 avoids the above pattern as it changes the comparison to SImode. Regtested using avrtest. No regression found. If OK, could someone commit please? Is this OK for gcc-5-branch? Regards, Pitchumani gcc/ChangeLog 2016-09-26 Pitchumani Sivanupandi

[patch, avr] Add flash size to device info and make wrap around default (was: Re: [patch, avr] Make pmem-wrap-around option as default)

2016-11-07 Thread Pitchumani Sivanupandi
On Thursday 03 November 2016 06:19 PM, Georg-Johann Lay wrote: On 03.11.2016 08:58, Pitchumani Sivanupandi wrote: Most of the AVR's 8k memorydevices have only rjmp instruction, not jmp. So, it is important to wrap around jump destination to check if it can reach backwards. Currently link

Re: [patch, avr] Add flash size to device info and make wrap around default

2016-11-10 Thread Pitchumani Sivanupandi
On Wednesday 09 November 2016 08:05 PM, Georg-Johann Lay wrote: On 09.11.2016 10:14, Pitchumani Sivanupandi wrote: On Tuesday 08 November 2016 02:57 PM, Georg-Johann Lay wrote: On 08.11.2016 08:08, Pitchumani Sivanupandi wrote: I have updated patch to include the flash size as well. Took

Re: [patch, avr, pr71676 and pr71678] Issues with casesi expand

2016-10-19 Thread Pitchumani Sivanupandi
On Wednesday 19 October 2016 07:51 PM, Georg-Johann Lay wrote: On 17.10.2016 09:27, Pitchumani Sivanupandi wrote: On Thursday 13 October 2016 08:42 PM, Georg-Johann Lay wrote: On 13.10.2016 13:44, Pitchumani Sivanupandi wrote: On Monday 26 September 2016 08:19 PM, Georg-Johann Lay wrote

Re: [patch, avr, pr71676 and pr71678] Issues with casesi expand

2016-10-17 Thread Pitchumani Sivanupandi
On Thursday 13 October 2016 08:42 PM, Georg-Johann Lay wrote: On 13.10.2016 13:44, Pitchumani Sivanupandi wrote: On Monday 26 September 2016 08:19 PM, Georg-Johann Lay wrote: On 26.09.2016 15:19, Pitchumani Sivanupandi wrote: Attached patch for PR71676 and PR71678. PR71676 is for AVR target

Re: [patch, avr] Add flash size to device info and make wrap around default

2016-11-14 Thread Pitchumani Sivanupandi
Ping! On Thursday 10 November 2016 01:53 PM, Pitchumani Sivanupandi wrote: On Wednesday 09 November 2016 08:05 PM, Georg-Johann Lay wrote: On 09.11.2016 10:14, Pitchumani Sivanupandi wrote: On Tuesday 08 November 2016 02:57 PM, Georg-Johann Lay wrote: On 08.11.2016 08:08, Pitchumani

[RFC, tentative patch] Adjust cost for conversion expression

2016-11-24 Thread Pitchumani Sivanupandi
GCC inlines small functions if the code size after expansion is not excedded. For test case (inline.c, avr-gcc -Os -S inline.c) code size become higher if 'func2' is inlined. It happens because the CONVERT_EXPR/ NOP_EXPR are considered as zero cost expression. Few conversions will cost

Re: [RFC, tentative patch] Adjust cost for conversion expression

2016-11-28 Thread Pitchumani Sivanupandi
On Thursday 24 November 2016 04:54 PM, Richard Biener wrote: On Thu, 24 Nov 2016, Pitchumani Sivanupandi wrote: GCC inlines small functions if the code size after expansion is not excedded. For test case (inline.c, avr-gcc -Os -S inline.c) code size become higher if 'func2' is inlined

Re: Ping: Re: [patch, avr] Add flash size to device info and make wrap around default

2016-11-27 Thread Pitchumani Sivanupandi
the resolved patch. Regards, Pitchumani gcc/ChangeLog 2016-11-09 Pitchumani Sivanupandi <pitchuman...@atmel.com> * config/avr/avr-arch.h (avr_mcu_t): Add flash_size member. * config/avr/avr-devices.c(avr_mcu_types): Add flash size info. * config/avr/avr-mcu.def: Likewise. *

Ping: Re: [patch, avr] Add flash size to device info and make wrap around default

2016-11-21 Thread Pitchumani Sivanupandi
Ping! On Monday 14 November 2016 07:03 PM, Pitchumani Sivanupandi wrote: Ping! On Thursday 10 November 2016 01:53 PM, Pitchumani Sivanupandi wrote: On Wednesday 09 November 2016 08:05 PM, Georg-Johann Lay wrote: On 09.11.2016 10:14, Pitchumani Sivanupandi wrote: On Tuesday 08 November 2016

[patch, avr] Make pmem-wrap-around option as default

2016-11-03 Thread Pitchumani Sivanupandi
, Pitchumani gcc/ChangeLog 2016-11-03 Pitchumani Sivanupandi <pitchuman...@atmel.com> * config/avr/gen-avr-mmcu-specs.c (print_mcu): Update link_pmem_wrap spec string to add linker option --pmem-wrap-around=8k. * config/avr/specs.h: Add link_pmem_wrap to linker specs (LIN

Re: [patch, avr, pr71676 and pr71678] Issues with casesi expand

2016-10-13 Thread Pitchumani Sivanupandi
On Monday 26 September 2016 08:19 PM, Georg-Johann Lay wrote: On 26.09.2016 15:19, Pitchumani Sivanupandi wrote: Attached patch for PR71676 and PR71678. PR71676 is for AVR target that generates wrong code when switch case index is more than 16 bits. Switch case index of larger than SImode

Re: Ping: Re: [patch, avr] Add flash size to device info and make wrap around default

2016-11-29 Thread Pitchumani Sivanupandi
On Tuesday 29 November 2016 10:06 PM, Denis Chertykov wrote: 2016-11-28 10:17 GMT+03:00 Pitchumani Sivanupandi <pitchumani.sivanupa...@microchip.com>: On Saturday 26 November 2016 12:11 AM, Denis Chertykov wrote: I'm sorry for delay. I have a problem with the patch: (Stripping traili

Re: [patch,avr] Add support for devices with flash accessible by LD.

2017-06-12 Thread Pitchumani Sivanupandi
On Friday 09 June 2017 03:59 PM, Georg-Johann Lay wrote: Hi, This patch adds support for devices that can access flash memory by LD* instructions, hence there is no need to put .rodata in RAM. The default linker script for the new multilib versions already supports this feature, it's similar