Re: [committed] Get avr building again

2021-05-06 Thread Senthil Kumar Selvaraj via Gcc-patches
Jeff Law via Gcc-patches writes: > Removes references to CC_STATUS_INIT from the avr port, which should get > it to the point of building again. > > > Committed to the trunk. Thanks, I was about to send a patch for that. Regards Senthil > > > Jeff

Re: [Ping, PATCH 2/n] AVR CC0 conversion - adjust peepholes

2021-04-29 Thread Senthil Kumar Selvaraj via Gcc-patches
mentioned in the base patch submission. Regards Senthil Senthil Kumar Selvaraj writes: > Applies cleanly on rebased version of previous patch > (https://gcc.gnu.org/pipermail/gcc-patches/2021-April/568658.html). > > Senthil Kumar Selvaraj writes: > >> Hi, >> >>

Re: [Ping] AVR CC0 conversion

2021-04-28 Thread Senthil Kumar Selvaraj via Gcc-patches
John Paul Adrian Glaubitz writes: > On 4/28/21 7:59 PM, Senthil Kumar Selvaraj wrote: >>>> OK for trunk. >>> >>> Anything else that keeps us from merging the changes? Would be great to >>> have the last backend besides CR-16 finally converted to M

Re: [Ping] AVR CC0 conversion

2021-04-28 Thread Senthil Kumar Selvaraj via Gcc-patches
John Paul Adrian Glaubitz writes: > Hi Senthil! > >> On Mon, Apr 26, 2021 at 9:20 AM Senthil Kumar Selvaraj via Gcc-patches >> wrote: >>> >>> Hi, >>> >>> This is >>> https://gcc.gnu.org/pipermail/gcc-patches/2021-January/563638.

[Ping, PATCH 2/n] AVR CC0 conversion - adjust peepholes

2021-04-25 Thread Senthil Kumar Selvaraj via Gcc-patches
Applies cleanly on rebased version of previous patch (https://gcc.gnu.org/pipermail/gcc-patches/2021-April/568658.html). Senthil Kumar Selvaraj writes: > Hi, > > This patch, to be applied on top of > https://gcc.gnu.org/pipermail/gcc-patches/2021-January/563638.html, > adjus

[PATCH 2/n] AVR CC0 conversion - adjust peepholes

2021-01-18 Thread Senthil Kumar Selvaraj via Gcc-patches
Hi, This patch, to be applied on top of https://gcc.gnu.org/pipermail/gcc-patches/2021-January/563638.html, adjusts peepholes to match and generate parallels with a clobber of REG_CC. It also sets mov_insn as the name of the pattern for the split insn (rather than the define_insn_and_split), so t

Re: [PATCH] avr: cc0 to mode_cc conversion

2021-01-05 Thread Senthil Kumar Selvaraj via Gcc-patches
Senthil Kumar Selvaraj writes: > Georg-Johann Lay writes: > >> >> Finally, some general remarks: > > The work on my github branch was not complete - I'd blindly followed > whatever the CC0 Transition wiki mentioned (the first three steps of > case #2), an

Re: [PATCH] emit-rtl.c: Allow splitting of RTX_FRAME_RELATED_P insns?

2020-08-13 Thread Senthil Kumar Selvaraj via Gcc-patches
sted on x86_64-linux. > > BTW, just to check: is your email address in MAINTAINERS still correct? It was out-of-date, yes - updated now. Regards Senthil 2020-08-13 Senthil Kumar Selvaraj gcc/ChangeLog: * emit-rtl.c (try_split): Call copy_frame_info_to_split_ins

[Committed] Update email address

2020-08-12 Thread Senthil Kumar Selvaraj via Gcc-patches
This patch updates my email address in the MAINTAINERS file 2020-08-12 Senthil Kumar Selvaraj * MAINTAINERS: Update my email address. diff --git MAINTAINERS MAINTAINERS index 0b825c7ea6d..217ec9c9eca 100644 --- MAINTAINERS +++ MAINTAINERS @@ -588,7 +588,7 @@ Stefan Schulze

Re: [Patch, avr, PR85624] - Fix ICE when initializing 128-byte aligned array

2018-08-06 Thread Senthil Kumar Selvaraj
Jeff Law writes: > On 08/06/2018 04:53 AM, Senthil Kumar Selvaraj wrote: >> Ping! >> >> Regards >> Senthil >> >> Senthil Kumar Selvaraj writes: >> >>> Hi, >>> >>> The below patch fixes an ICE for the avr target w

Re: [Patch, avr, PR85624] - Fix ICE when initializing 128-byte aligned array

2018-08-06 Thread Senthil Kumar Selvaraj
Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Hi, > > The below patch fixes an ICE for the avr target when the setmemhi > expander is involved. > > The setmemhi expander generated RTL ends up as an unrecognized insn > if the alignment of the destination exceed

Re: [Patch, avr, PR86635] Fix miscompilation with __memx and libgcc float function __gtsf2

2018-07-26 Thread Senthil Kumar Selvaraj
Senthil Kumar Selvaraj writes: > The below patch fixes a miscompilation of function calls with__memx address > space > arguments. > > For code like > > extern const __memx float a; > extern const float b; > > int diff () { return a > b; } > > when c

[Patch, avr, PR86635] Fix miscompilation with __memx and libgcc float function __gtsf2

2018-07-25 Thread Senthil Kumar Selvaraj
with clobbered regs and substitutes pseudos in their place. The patch also adds a testcase to verify a is actually read. Reg testing passed. Ok to commit to trunk? Regards Senthil gcc/ChangeLog: 2018-07-25 Senthil Kumar Selvaraj * config/avr/avr-protos.h (avr_emit2_fix_outputs): New prot

[Patch, avr, PR85624] - Fix ICE when initializing 128-byte aligned array

2018-07-18 Thread Senthil Kumar Selvaraj
also adds a testcase. Regression test showed no new failures, ok to commit to trunk? Regards Senthil gcc/ChangeLog: 2018-07-18 Senthil Kumar Selvaraj PR target/85624 * config/avr/avr.md (*clrmemqi): Change mode of operands[2] from QI to HI. gcc/testsuite/ChangeLog

[Patch, testsuite, committed] Fix bogus builtin-snprintf-warn-3.c failure for avr

2017-05-23 Thread Senthil Kumar Selvaraj
to (16 bit) int size. Fixed by marking the test as unsupported for targets with ptr size < 32. Committed as obvious. Regards Senthil gcc/testsuite/ 2017-05-23 Senthil Kumar Selvaraj * gcc.dg/tree-ssa/builtin-snprintf-warn-3.c: Require ptr32plus. Index: gcc/testsuite/gcc.dg/tre

[Patch, testsuite, committed] Fix bogus pr78886.c for avr

2017-05-16 Thread Senthil Kumar Selvaraj
gcc/testsuite/ChangeLog 2017-05-17 Senthil Kumar Selvaraj * gcc.dg/tree-ssa/pr78886.c: Use __SIZE_TYPE__ instead of unsigned long. Index: gcc/testsuite/gcc.dg/tree-ssa/pr78886.c === --- gcc/testsuite/gcc.dg/tre

[Patch, testsuite, committed] Fix cunroll-13.c failure for avr

2017-05-09 Thread Senthil Kumar Selvaraj
hil gcc/testsuite/ 2017-05-09 Senthil Kumar Selvaraj * gcc.dg/tree-ssa/cunroll-13.c: Use __INT32_TYPE__ for for targets with __SIZEOF_INT__ < 4. diff --git gcc/testsuite/gcc.dg/tree-ssa/cunroll-13.c gcc/testsuite/gcc.dg/tree-ssa/cunroll-13.c index f3fe8b51468..904e6dc075b 100

[Patch, testsuite] Fix bogus pr78138.c failure for avr

2017-05-02 Thread Senthil Kumar Selvaraj
in a couple of FAILs. The patch fixes that by typedef'ing __SIZE_TYPE__ to size_t and using size_t as the type for memcpy's third parameter. Committed to trunk as obvious. Regards Senthil gcc/testsuite/ChangeLog 2017-05-02 Senthil Kumar Selvaraj * gcc.dg/pr78

[Patch, testsuite] Fix failing attr-alloc_size-10.c for avr

2017-04-25 Thread Senthil Kumar Selvaraj
-04-25 Senthil Kumar Selvaraj * gcc.dg/attr-alloc_size-10.c: Ignore overflow warnings for targets with int size less than 32 bits. Index: gcc/testsuite/gcc.dg/attr-alloc_size-10.c === --- gcc/testsuite/gcc.dg/attr

[Patch, testsuite] Skip pr80170.c for non-ptr32plus targets

2017-04-19 Thread Senthil Kumar Selvaraj
gcc/testsuite/ChangeLog 2017-04-19 Senthil Kumar Selvaraj * gcc.dg/pr80170.c: Require ptr32plus. Index: gcc.dg/pr80170.c === --- gcc.dg/pr80170.c(revision 247010) +++ gcc.dg/pr80170.c(working copy) @@ -1,5 +1,6

[Patch, testsuite] Fix broken pr80341.c for avr

2017-04-19 Thread Senthil Kumar Selvaraj
onversion" of the unsigned short to signed int doesn't occur, and this causes the test to fail. Committed as obvious. Regards Senthil 2017-04-19 Senthil Kumar Selvaraj * gcc.dg/torture/pr80341.c: Require int32plus. Index: g

[Patch, testsuite] Fix failing builtin-sprintf-warn-{3,10}.c for avr

2017-04-06 Thread Senthil Kumar Selvaraj
est appears to assume all non lp64 targets to be ilp32. For the avr, pointer size and int size are equal, but both are 16 bits, not 32. The patch fixes this by explicitly adding avr to the dejagnu selector. Ok for trunk? Regards Senthil gcc/testsuite/ChangeLog: 2017-04-06 Senthil Ku

[Patch, testsuite] Fix more failing tests for avr

2017-03-28 Thread Senthil Kumar Selvaraj
Kumar Selvaraj * gcc.c-torture/execute/pr79121.c:Use __{U}INT32_TYPE__ for targets with sizeof(int) < 4. * gcc.c-torture/execute/pr79737-1.c (struct S): Likewise. * gcc.c-torture/execute/pr79737-2.c: Likewise. * gcc.dg/torture/pr79777.c: Likew

[Patch, testsuite] Fix failing overflow-1.c for avr

2017-03-21 Thread Senthil Kumar Selvaraj
Hi, The test assumes 32 bit ints, and expects a constant in the dump that is only valid for 32 bit ints. This trivial patch fixes that by explicitly specifying __UINT32_TYPE__ as the type. Committed as obvious. Regards Senthil gcc/testsuite/ChangeLog 2017-03-21 Senthil Kumar Selvaraj

[Patch, testsuite] Fix bogus pr31096-1.c failure for avr

2016-11-29 Thread Senthil Kumar Selvaraj
trunk as obvious. Regards Senthil gcc/testsuite/ChangeLog 2016-11-29 Senthil Kumar Selvaraj * testsuite/gcc.dg/pr31096-1.c: Use __{U,}INT32_TYPE__ for targets with sizeof(int) < 4. Index: gcc/testsuite/gcc.dg/

[Patch, testsuite] Fix bogus pr64427.c failure for avr

2016-11-25 Thread Senthil Kumar Selvaraj
int32_t everywhere. Regards Senthil 2016-11-25 Senthil Kumar Selvaraj * gcc.dg/pr64277.c: Use 32 bit int for targets with sizeof(int) < 4. Index: gcc/testsuite/gcc.dg/pr64277.c === --- gcc/testsuite/gcc.d

Re: [Patch, tentative] Use MOVE_MAX_PIECES instead of MOVE_MAX in gimple_fold_builtin_memory_op

2016-11-24 Thread Senthil Kumar Selvaraj
Richard Biener writes: > On Thu, 24 Nov 2016, Richard Biener wrote: > >> On Thu, 24 Nov 2016, Senthil Kumar Selvaraj wrote: >> >> > Hi, >> > >> > I've been analyzing a failing regtest (gcc.dg/strlenopt-8.c) for the avr >> > tar

[Patch, tentative] Use MOVE_MAX_PIECES instead of MOVE_MAX in gimple_fold_builtin_memory_op

2016-11-24 Thread Senthil Kumar Selvaraj
Hi, I've been analyzing a failing regtest (gcc.dg/strlenopt-8.c) for the avr target. I found that the (dump) failure is because there are 4 instances of memcpy, while the testcase expects only 2 for a non-strict align target like the avr. Comparing that with a dump generated by x64_64-p

Re: [Patch, testsuite] Fix bogus uninit-19.c failure for avr

2016-11-23 Thread Senthil Kumar Selvaraj
Jeff Law writes: > On 11/23/2016 02:54 AM, Senthil Kumar Selvaraj wrote: >> Hi, >> >> The below patch fixes uninit-19.c for avr by adding >> -finline-small-functions for avr. >> >> The test fails for avr because fn1 does not get inlined into fn2. Inli

[Patch, testsuite] Fix bogus uninit-19.c failure for avr

2016-11-23 Thread Senthil Kumar Selvaraj
, and b[3] = p10[a] results in a bigger size for fn1, resulting in estimate_growth > 0 and no inlining. Adding -finline-small-functions forces early inliner to inline fn1, making the test pass. Committed to trunk. Regards Senthil gcc/testsuite/ChangeLog 2016-11-23 Senthil Kumar

Re: [patch,testsuite] Support dg-require-effective-target label_offsets.

2016-11-17 Thread Senthil Kumar Selvaraj
Georg-Johann Lay writes: > On 04.11.2016 06:18, Senthil Kumar Selvaraj wrote: >> >> Georg-Johann Lay writes: >>> State of matters is that Binutils support is missing, and if I understand >>> you >>> correctly, dg-require is not appropriate to fa

[Patch, testsuite] Fix bogus Wlogical-op-1.c test failure for avr

2016-11-16 Thread Senthil Kumar Selvaraj
g when the logical operands are of type short and int. This isn't true for the avr - shorts and ints are of the same size, and therefore the warning triggers for the above case also. Committed to trunk as obvious. Regards Senthil gcc/testsuite/ChangeLog 2016-11-16 Senthil

Re: [patch,testsuite] Support dg-require-effective-target label_offsets.

2016-11-03 Thread Senthil Kumar Selvaraj
Georg-Johann Lay writes: > On 28.10.2016 17:58, Mike Stump wrote: >> On Oct 27, 2016, at 3:16 AM, Georg-Johann Lay wrote: >>> >>> Now imagine some arithmetic like &&LAB2 - &&LAB1. This might result in >>> one or two stub addresses, and difference between such addresses is a >>> complete differe

[Patch, testsuite] Add new effective-target_store_merge

2016-11-03 Thread Senthil Kumar Selvaraj
gcc/testsuite/ChangeLog 2016-11-03 Senthil Kumar Selvaraj * gcc.dg/store_merging_1.c: Require store_merge. * gcc.dg/store_merging_2.c: Likewise. * gcc.dg/store_merging_4.c: Likewise. * gcc.dg/store_merging_5.c: Likewise. * gcc.dg/store_merging_6.c: Lik

[Patch, testsuite] Fix bogus PR 78170 failure for avr

2016-11-03 Thread Senthil Kumar Selvaraj
Hi, The below patch requires int32plus for gcc.c-torture/execute/pr78170.c, as it has int bitfields that are more than 16 bits wide. Committed to trunk as obvious. Regards Senthil gcc/testsuite/ChangeLog 2016-11-03 Senthil Kumar Selvaraj * gcc.c-torture/execute/pr78170.c

[Patch, testsuite] Skip gcc.dg/lto/pr60449_0.c for avr

2016-10-31 Thread Senthil Kumar Selvaraj
Hi, gcc.dg/lto/pr60449_0.c fails to link for the avr target because it doesn't have an implementation for gettimeofday. This patch therefore skips the test for avr. Committed to trunk as obvious. Regards Senthil gcc/testsuite/ChangeLog 2016-10-31 Senthil Kumar Sel

Re: [Patch, reload, tentative, PR 71627] Tweak conditions in find_valid_class_1

2016-10-21 Thread Senthil Kumar Selvaraj
Bernd Schmidt writes: > On 10/18/2016 02:15 PM, Senthil Kumar Selvaraj wrote: >> Will do both the changes and re-run the reg tests. Ok for trunk if the >> tests pass for x86_64-pc-linux and avr? >> > Probably but let's see the patch first. How does this look? Boots

Re: [Patch, reload, tentative, PR 71627] Tweak conditions in find_valid_class_1

2016-10-18 Thread Senthil Kumar Selvaraj
Bernd Schmidt writes: > On 10/13/2016 08:57 AM, Senthil Kumar Selvaraj wrote: >> >> 2016-10-13 Senthil Kumar Selvaraj >> >> * reload.c (find_valid_class_1): Allow regclass if atleast one >> regno in class is ok. Compute and use rclass size based on

[Patch,testsuite] Fix sso.exp not calling torture-finish for avr

2016-10-18 Thread Senthil Kumar Selvaraj
s like the avr for which the effective target condition is true, and this leaves torture-options set, which causes the errors and differing options. The below patch makes the return occur earlier - before calling the init functions. Committed to trunk. Regards Senthil 2016-10-18 Sen

Re: [Patch] Backport fix for PR 52085 to gcc-5-branch?

2016-10-18 Thread Senthil Kumar Selvaraj
Jakub Jelinek writes: > On Tue, Oct 18, 2016 at 02:46:29PM +0530, Senthil Kumar Selvaraj wrote: >> > I'm not convinced it is desirable to backport such changes, it affects ABI, >> > people are used to deal with minor ABI changes in between major GCC >> > rele

Re: [Patch, reload, tentative, PR 71627] Tweak conditions in find_valid_class_1

2016-10-18 Thread Senthil Kumar Selvaraj
Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Bernd Schmidt writes: > >> On 09/16/2016 09:02 PM, Senthil Kumar Selvaraj wrote: >>> Does this make sense? I ran a reg test for the avr target with a >>> slightly older version of this patch, it did not

Re: [Patch] Backport fix for PR 52085 to gcc-5-branch?

2016-10-18 Thread Senthil Kumar Selvaraj
Jakub Jelinek writes: > On Tue, Oct 18, 2016 at 10:12:24AM +0200, Richard Biener wrote: >> On Mon, Oct 17, 2016 at 6:57 PM, Senthil Kumar Selvaraj >> wrote: >> > >> > Richard Biener writes: >> > >> >> On Mon, Oct 17, 2016 at 12

Re: [Patch] Backport fix for PR 52085 to gcc-5-branch?

2016-10-17 Thread Senthil Kumar Selvaraj
Richard Biener writes: > On Mon, Oct 17, 2016 at 12:21 PM, Senthil Kumar Selvaraj > wrote: >> Hi, >> >> The fix for PR 52085 went into trunk when trunk was 6.0. I ran into the >> same issue on a gcc 5.x and found that the fix didn't get backported. >&g

[Patch] Backport fix for PR 52085 to gcc-5-branch?

2016-10-17 Thread Senthil Kumar Selvaraj
10-17 Senthil Kumar Selvaraj Backport from mainline 2015-04-25 Marek Polacek PR c/52085 * c-decl.c (finish_enum): Copy over TYPE_ALIGN. Also check for "mode" attribute. gcc/testsuite/ChangeLog 2016-10-17 Senthil Kumar Selvaraj Backpor

Re: [Patch, reload, tentative, PR 71627] Tweak conditions in find_valid_class_1

2016-10-12 Thread Senthil Kumar Selvaraj
Bernd Schmidt writes: > On 09/16/2016 09:02 PM, Senthil Kumar Selvaraj wrote: >> Does this make sense? I ran a reg test for the avr target with a >> slightly older version of this patch, it did not show any regressions. >> If this is the right fix, I'll make sure

[Patch, testsuite] Fix gcc.g/tree-ssa/pr59597.c failure for avr

2016-10-11 Thread Senthil Kumar Selvaraj
reg testing with avr and x86_64-pc-linux Regards Senthil gcc/testsuite/ChangeLog 2016-10-11 Senthil Kumar Selvaraj * gcc.dg/tree-ssa/pr59597.c: Typedef __INT32_TYPE__ to i32. (main): Declare j as i32 instead of int. Index: gcc/testsuite/gcc.dg/tree-ssa/pr5959

[Patch, testsuite] Fix pr69941.c test failure for avr

2016-10-05 Thread Senthil Kumar Selvaraj
/ChangeLog 2016-10-05 Senthil Kumar Selvaraj * gcc.dg/torture/pr69941.c: Use __INT32_TYPE__ instead of int if __SIZEOF_INT__ is less than 4 bytes. Index: gcc/testsuite/gcc.dg/torture/pr69941.c === --- gcc/testsuite

[Patch, testsuite] Add ffat-lto-objects to gcc.target/avr/torture/builtins_error.c

2016-10-03 Thread Senthil Kumar Selvaraj
consistently at compile, as expected. Committed to trunk. Regards Senthil gcc/testsuite/ChangeLog: 2016-10-03 Senthil Kumar Selvaraj * gcc.target/avr/torture/builtins-error.c: Add -ffat-lto-objects option. Index: gcc/testsuite/gcc.target/avr/torture/builtins-error.c

Re: [Patch, testsuite] Require int32plus for builtin-sprintf-warn-1.c

2016-09-27 Thread Senthil Kumar Selvaraj
James Greenhalgh writes: > On Tue, Sep 27, 2016 at 04:40:22PM +0530, Senthil Kumar Selvaraj wrote: >> Hi, >> >> This patch requires int32plus for >> gcc.dg/tree-ssa/builtin-sprintf-warn-1.c, as it reports a bunch of >> failures for a 16 bit int tar

[Patch, testsuite] Require int32plus for builtin-sprintf-warn-1.c

2016-09-27 Thread Senthil Kumar Selvaraj
unexpected warnings. Comitted to trunk. Regards Senthil 2016-09-27 Senthil Kumar Selvaraj * gcc.dg/tree-ssa/builtin-sprintf-warn-1.c: Require int32plus. PR fortran/77666 Index: gcc.dg/tree-ssa/builtin-sprint

Re: [Patch, reload, tentative, PR 71627] Tweak conditions in find_valid_class_1

2016-09-26 Thread Senthil Kumar Selvaraj
Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Hi, > > The below patch fixes what I think are a couple of problems with > reload.c:find_valid_class_1. > > First, even if no regno is in_hard_reg_set_p, it goes ahead and > considers rclass as valid. bad is s

Re: [Patch, avr] Backport fix for PR 65210 to gcc-5-branch

2016-09-26 Thread Senthil Kumar Selvaraj
Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Hi, > > Is it ok to backport PR 65210 to gcc-5-branch? The patch is already in > 6.x and trunk. > > Regards > Senthil > > gcc/ChangeLog > > 2016-09-22 Senthil Kumar Selvaraj > > Backport

Backport fix for PR 65210 to gcc-5-branch

2016-09-22 Thread Senthil Kumar Selvaraj
Hi, Is it ok to backport PR 65210 to gcc-5-branch? The patch is already in 6.x and trunk. Regards Senthil gcc/ChangeLog 2016-09-22 Senthil Kumar Selvaraj Backport from trunk r227496 PR target/65210 * config/avr/avr.c (avr_eval_addr_attrib): Look for io_low

[Patch, avr] Provide correct LD offset bound in avr_address_cost

2016-09-21 Thread Senthil Kumar Selvaraj
aling SImode costs by 2 for DImode muls. With both changes in, there are no regressions, and the builtin-arith-overflow-* tests now PASS and don't timeout. Ok to commit to trunk and backport to 5.x? Regards Senthil gcc/ChangeLog: 2016-09-21 Senthil Kumar Selvaraj * config

[Patch, testsuite] Make pr64130.c pass for avr

2016-09-20 Thread Senthil Kumar Selvaraj
nthil Kumar Selvaraj * gcc.dg/tree-ssa/pr64130.c: Use __UINT32_TYPE__ instead of int. Index: gcc.dg/tree-ssa/pr64130.c === --- gcc.dg/tree-ssa/pr64130.c (revision 240299) +++ gcc.dg/tree-ssa/pr64130.c (revision 240300) @@

[Patch, reload, tentative, PR 71627] Tweak conditions in find_valid_class_1

2016-09-16 Thread Senthil Kumar Selvaraj
Hi, The below patch fixes what I think are a couple of problems with reload.c:find_valid_class_1. First, even if no regno is in_hard_reg_set_p, it goes ahead and considers rclass as valid. bad is set only if a regno is in the reg class *and* HARD_REGNO_MODE_OK is false - if both are fal

[Patch, testsuite] Require int32plus for pr70421.c

2016-09-16 Thread Senthil Kumar Selvaraj
Hi, This patch fixes a bogus testsuite failure for the avr target. The test has integer literals that only fit on targets with int size >= 32. Committed to trunk. Regards Senthil gcc/testsuite/ChangeLog 2016-09-16 Senthil Kumar Selvaraj * gcc.dg/torture/pr70421.c: Requ

[Patch, testsuite, avr] Skip gcc.dg/Wno-frame-address.c for avr

2016-09-06 Thread Senthil Kumar Selvaraj
Hi, The below patch adds avr to the list of targets excluded for Wno-frame-address.c. Like the other excluded targets, the avr backend only supports the builtin for the current stack frame. Committed to trunk. Regards Senthil gcc/testsuite/ChangeLog: 2016-09-06 Senthil Kumar Selvaraj

[Patch, testsuite] Fix more bogus failures for avr

2016-09-01 Thread Senthil Kumar Selvaraj
-01 Senthil Kumar Selvaraj * gcc.dg/pr64252.c: Require int32plus. * gcc.dg/pr66299-1.c: Likewise. * gcc.dg/pr66299-2.c: Likewise. * gcc.dg/torture/20131115-1.c: Skip for avr. Index: gcc/testsuite/gcc.dg/pr64252.c

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

2016-08-24 Thread Senthil Kumar Selvaraj
gcc/testsuite/ChangeLog 2016-08-24 Senthil Kumar Selvaraj * gcc.c-torture/execute/pr71083.c: Use UINT32_TYPE instead of unsigned int. * gcc.dg/zero_sign_ext_test.c: Require int32plus. Index: gcc/testsuite/gcc.c-torture/execute/pr71083.c

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

2016-08-16 Thread Senthil Kumar Selvaraj
Jeff Law writes: > On 08/11/2016 01:40 AM, Senthil Kumar Selvaraj wrote: >> Hi, >> >> The below patch adds the AVR target to the list of targets that don't >> have natural_alignment_32. It also skips ipa/propalign-*.c >> tests (w

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

2016-08-11 Thread Senthil Kumar Selvaraj
ommit? Regards Senthil gcc/testsuite/ChangeLog: 2016-08-11 Senthil Kumar Selvaraj * 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.dg/ipa/propalign-3.c: Lik

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

2016-08-10 Thread Senthil Kumar Selvaraj
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 - the re

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

2016-08-08 Thread Senthil Kumar Selvaraj
Bernd Schmidt writes: > On 07/28/2016 09:33 AM, Senthil Kumar Selvaraj wrote: >> >> Is there a reason why only REG and SYMBOL_REFs get valid >> subreg_in_class? I tried extending it handle constants and PLUS >> expressions, and it fixes PR 71873. It also fixes a

Re: [Patch, wwwdocs] Add caveat for AVR port

2016-08-08 Thread Senthil Kumar Selvaraj
Georg-Johann Lay writes: > On 08.08.2016 10:24, Senthil Kumar Selvaraj wrote: >> Hi, >> >> This doc patch informs the user that a specific (or higher) version of >> binutils is a prerequisite for the fix for a rather vexing bug (PR >> 71151) that was fixed

[Patch, wwwdocs] Add caveat for AVR port

2016-08-08 Thread Senthil Kumar Selvaraj
Hi, This doc patch informs the user that a specific (or higher) version of binutils is a prerequisite for the fix for a rather vexing bug (PR 71151) that was fixed for 6.2. I've added it to the Caveats section; is there a better place? If not, ok to commit? Regards Senthil Index: chan

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

2016-08-03 Thread Senthil Kumar Selvaraj
Hi, Committed below patch to trunk as obvious. Regards Senthil 2016-08-03 Senthil Kumar Selvaraj * gcc.dg/init-excess-2.c: Require int32plus. * gcc.dg/pr44024.c: Skip if target keeps null pointer checks. * gcc.dg/pr59963-2.c: Require int32plus. * gcc.dg

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

2016-08-02 Thread Senthil Kumar Selvaraj
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 : >>>> Problem with -fcaller-saves is that there are situations where it triggers >>>&

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

2016-08-01 Thread Senthil Kumar Selvaraj
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. >> >> Example: >> >>

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

2016-07-28 Thread Senthil Kumar Selvaraj
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 o

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

2016-07-26 Thread Senthil Kumar Selvaraj
Mike Stump writes: > On Jul 25, 2016, at 5:00 AM, Senthil Kumar Selvaraj > wrote: >> >> The below patch fixes tests that fail for the avr target, because they >> assume ints are atleast 32 bits wide and pointers and longs have the >> same size. >> >

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

2016-07-25 Thread Senthil Kumar Selvaraj
he pointer <-> 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 * gcc.dg/torture/pr69352.c (foo): Cast to intptr_t instead of long. * gcc.dg/t

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

2016-07-19 Thread Senthil Kumar Selvaraj
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, but

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

2016-07-19 Thread Senthil Kumar Selvaraj
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 * gcc.dg/params/blocksort-part.c: Conditionally define Int32

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

2016-07-13 Thread Senthil Kumar Selvaraj
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 pressure.

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

2016-07-13 Thread Senthil Kumar Selvaraj
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 * c-c++-common

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

2016-07-04 Thread Senthil Kumar Selvaraj
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

2016-07-04 Thread Senthil Kumar Selvaraj
e set 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 PR target/50739 * gcc.target/avr/pr50739.c: New test. gcc/ChangeLog: 2016-07-05 Senthil

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

2016-06-23 Thread Senthil Kumar Selvaraj
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 execut

Re: [Patch, avr] Fix PR 71151

2016-06-22 Thread Senthil Kumar Selvaraj
Georg-Johann Lay writes: > Senthil Kumar Selvaraj schrieb: >> Senthil Kumar Selvaraj writes: >> >>> Georg-Johann Lay writes: >>> >>>> Senthil Kumar Selvaraj schrieb: >>>>> Hi, >>>>> >>>>> [set JUMP_TABL

[Patch, testsuite] Mark some more tests as UNSUPPORTED for avr

2016-06-20 Thread Senthil Kumar Selvaraj
Hi, This patch fixes some bogus failures for the avr target by requiring int32plus or ptr32plus support. Ok for trunk? Regards Senthil gcc/testsuite/ChangeLog: 2016-06-20 Senthil Kumar Selvaraj * c-c++-common/pr68657-1.c: Require ptr32plus support. * c-c++-common

Re: [Patch, avr] Fix PR 71151

2016-06-18 Thread Senthil Kumar Selvaraj
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

2016-06-18 Thread Senthil Kumar Selvaraj
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 + + * MAINTAINERS

Re: [Patch, avr] Fix PR 71151

2016-06-16 Thread Senthil Kumar Selvaraj
Denis Chertykov writes: > 2016-06-16 10:27 GMT+03:00 Senthil Kumar Selvaraj > : >> >> Senthil Kumar Selvaraj writes: >> >>> Georg-Johann Lay writes: >>> >>>> Senthil Kumar Selvaraj schrieb: >>>&g

Re: [Patch, avr] Fix PR 71151

2016-06-16 Thread Senthil Kumar Selvaraj
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

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

2016-06-08 Thread Senthil Kumar Selvaraj
? I don't have commit access. Regards Senthil gcc/testsuite/ChangeLog 2016-06-08 Senthil Kumar Selvaraj * gcc.c-torture/execute/bswap-2.c: Require int32plus. * gcc.dg/torture/pr68067-1.c: Likewise. * gcc.dg/torture/pr68067-2.c: Likewise. diff --git gcc/test

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

2016-06-08 Thread Senthil Kumar Selvaraj
x 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 * gcc.dg/stack-usage-1.c (SIZE): Consider return address when setting S

Re: [Patch, avr] Fix PR 71151

2016-06-07 Thread Senthil Kumar Selvaraj
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

2016-06-03 Thread Senthil Kumar Selvaraj
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 * config/avr/avr.c (avr_asm_function_rodata_section): Remove. * config/avr/avr.c (TARGET_ASM_FUNCTION_RODATA_SE

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

2016-05-25 Thread Senthil Kumar Selvaraj
don't have commit access. Regards Senthil 2016-05-25 Senthil Kumar Selvaraj * c-c++-common/Wduplicated-cond-1.c: Use smaller const literal. * c-c++-common/pr60226.c: Require int32plus. * gcc.c-torture/execute/pr70602.c: Likewise. * gcc.dg/Warray-bounds

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

2016-05-19 Thread Senthil Kumar Selvaraj
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

2016-05-13 Thread Senthil Kumar Selvaraj
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 * config/avr/avr.c (avr_expand_prologue): Add INCOMING_FRAME_SP_OFFSET to com

Re: [Patch] Fix PR 60040

2016-04-28 Thread Senthil Kumar Selvaraj
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 @@ >> -20

Re: [Patch] Fix PR 60040

2016-04-27 Thread Senthil Kumar Selvaraj
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

2016-04-25 Thread Senthil Kumar Selvaraj
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] Fix PR 60040

2016-04-15 Thread Senthil Kumar Selvaraj
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, testsuite] Require int32plus and scheduling support for some tests

2016-04-13 Thread Senthil Kumar Selvaraj
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 > requi

[Patch] Fix PR 60040

2016-04-07 Thread Senthil Kumar Selvaraj
sions. If ok, could someone commit please? I don't have commit access. Regards Senthil gcc/ChangeLog 2016-04-07 Joern Rennecke Senthil Kumar Selvaraj PR target/60040 * reload1.c (find_reload_regs): Add tentative parameter. and don't report s

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

2016-04-04 Thread Senthil Kumar Selvaraj
lus - 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 * gcc.c-torture/compile/pr69102.c

Re: [Patch, testsuite] Skip testcase for avr

2016-03-19 Thread Senthil Kumar Selvaraj
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/testsuite/

  1   2   >