[PR83663] Revert r255946

2018-01-08 Thread Vidya Praveen
Hello, This patch reverts the changes introduced by r255946 and further changes to that done by r256195, as the former causes large number of regressions on aarch64_be* targets. This should be respun with the mismatch in lane numbering in AArch64 and GCC's numbering fixed as explained in PR83663.

[patch][arm] (respin) Improve error checking in parsecpu.awk

2017-09-22 Thread Vidya Praveen
Hello, This patch by Richard Earnshaw was reverted along with the commit that preceded it as the preceding commit was causing cross-native builds to fail and I presumed this patch was related too. Now I am respinning as the issue that caused the cross-native failure have been fixed. This patch

[patch][arm] (respin) auto-generate arm-isa.h from CPU descriptions

2017-09-22 Thread Vidya Praveen
Hello, This patch by Richard Earnshaw was reverted earlier as it was breaking cross-native builds. Respinning now with a minor change that fixes the build issue - adding arm-isa.h to GTM_H. Also remove a redundant dependency (TM_H includes GTM_H). Tested by building cross/cross-native arm-none-li

Re: [RFC, vectorizer] Allow single element vector types for vector reduction operations

2017-09-11 Thread Vidya Praveen
On Tue, Sep 05, 2017 at 03:12:47PM +0200, Richard Biener wrote: > On Tue, 5 Sep 2017, Tamar Christina wrote: > > > > > > > > -Original Message- > > > From: Richard Biener [mailto:rguent...@suse.de] > > > Sent: 05 September 2017 13:51 > > > To: Tamar Christina > > > Cc: Andrew Pinski; And

Re: [COMMITTED][arm] Revert r251800 & r251799

2017-09-11 Thread Vidya Praveen
Now with the patch :-) VP. On Mon, Sep 11, 2017 at 03:20:12PM +0100, Vidya Praveen wrote: > Hello, > > The following two related patches need to be reverted as it causes > cross-native > builds to fail with the following message: > > g++ -c -DIN_GCC

[COMMITTED][arm] Revert r251800 & r251799

2017-09-11 Thread Vidya Praveen
New keyword. * config/arm/t-arm (OPTIONS_H_EXTRA): Add arm-isa.h (arm-isa.h): Add rule to generate file. * common/config/arm/arm-common.c: (arm_canon_arch_option): Use lower case for feature bit names. Regards, VP. gcc/ChangeLog: 2017-09-11 Vidya Praveen Revert r251800 and r251799.

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-18 Thread Vidya Praveen
On Wed, Jan 18, 2017 at 11:10:32AM +0100, Martin Liška wrote: > Hello. > > After basic understanding of loop predictive commoning, the problematic > combined chain is: > > Loads-only chain 0x38b6730 (combined) > max distance 0 > references: > MEM[(real(kind=8) *)vectp_a.29_81] (id 1) >

Re: [PATCH, rs6000, testsuite, PR65456] Changes for unaligned vector load/store support on POWER8

2015-06-16 Thread Vidya Praveen
On Mon, Jun 15, 2015 at 08:14:31PM +0100, Bill Schmidt wrote: > On Fri, 2015-06-12 at 17:36 +0100, Vidya Praveen wrote: > > On Thu, Apr 30, 2015 at 01:34:18PM +0100, Bill Schmidt wrote: > > > On Thu, 2015-04-30 at 18:26 +0800, Bin.Cheng wrote: > > > > On Mon, Apr 27,

Re: [PATCH, rs6000, testsuite, PR65456] Changes for unaligned vector load/store support on POWER8

2015-06-12 Thread Vidya Praveen
On Thu, Apr 30, 2015 at 01:34:18PM +0100, Bill Schmidt wrote: > On Thu, 2015-04-30 at 18:26 +0800, Bin.Cheng wrote: > > On Mon, Apr 27, 2015 at 9:26 PM, Bill Schmidt > > wrote: > > > On Mon, 2015-04-27 at 14:23 +0800, Bin.Cheng wrote: > > >> On Mon, Mar 30, 2015 at 1:42 AM, Bill Schmidt > > >> wr

Re: [PATCH, RFC] New memory usage statistics infrastructure

2015-06-01 Thread Vidya Praveen
On 01/06/15 15:21, Vidya Praveen wrote: On 01/06/15 15:08, Martin Liška wrote: On 06/01/2015 02:18 PM, Richard Biener wrote: On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška wrote: On 05/29/2015 06:09 PM, Vidya Praveen wrote: Martin, The following change: @@ -2655,10 +2655,10 @@ s-iov: build

Re: [PATCH, RFC] New memory usage statistics infrastructure

2015-06-01 Thread Vidya Praveen
On 01/06/15 15:08, Martin Liška wrote: On 06/01/2015 02:18 PM, Richard Biener wrote: On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška wrote: On 05/29/2015 06:09 PM, Vidya Praveen wrote: Martin, The following change: @@ -2655,10 +2655,10 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER

Re: [PATCH, RFC] New memory usage statistics infrastructure

2015-05-29 Thread Vidya Praveen
Martin, The following change: @@ -2655,10 +2655,10 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER) $(DEVPHASE) GCOV_OBJS = gcov.o gcov$(exeext): $(GCOV_OBJS) $(LIBDEPS) - +$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GCOV_OBJS) $(LIBS) -o $@ + +$(LINKER) $(ALL_LINKERFLAGS) $(LDF

Re: [Patch,testsuite] Fix bind_pic_locally

2014-06-25 Thread Vidya Praveen
PING! On Wed, Jun 04, 2014 at 02:56:00PM +0100, Vidya Praveen wrote: > Hello, > > This is to follow up the patch I had posted to fix bind_pic_locally some time > ago (sorry, this went in to my back log for a while). > > To summarize, multilib_flags when it contains -fpic o

Re: [Patch,testsuite] Fix tests that fail due to symbol visibility when -fPIC

2014-06-25 Thread Vidya Praveen
PING! On Wed, Jun 04, 2014 at 03:01:38PM +0100, Vidya Praveen wrote: > Hello, > > The following test cases fail when -fPIC is passed as dejagnu multilib > flag > since -fPIC causes the 'availability' of the functions to be overwritable. > I > have fixed

[Patch,testsuite] Fix tests that fail due to symbol visibility when -fPIC

2014-06-04 Thread Vidya Praveen
inline_4.c g++.dg/ipa/devirt-25.C Tested on: aarch64-none-elf aarch64-none-linux-gnu arm-none-linux-gnueabihf x86_64-unknown-linux-gnu OK for trunk? Cheers VP. ~~~ gcc/testsuite/ChangeLog: 2014-06-04 Vidya Praveen * gcc.dg/inline-22.c: Add bind_pic_locally.

[Patch,testsuite] Fix bind_pic_locally

2014-06-04 Thread Vidya Praveen
/testsuite/ChangeLog: 2014-06-04 Vidya Praveen * lib/target-support.exp (bind_pic_locally): Save the flags to 'flags_to_postpone' instead of appending to 'flags'. * lib/gcc.exp (gcc_target_compile): Append board_info's multilib_flags with fla

[Patch,AArch64] Support SISD variants of SCVTF,UCVTF

2014-01-13 Thread Vidya Praveen
the instructions. Tested for aarch64-none-elf. OK for trunk? Cheers VP. gcc/ChangeLog: 2013-01-13 Vidya Praveen * aarch64.md (float2): Remove. (floatuns2): Remove. (2): New pattern for equal width float and floatuns conversions. (2): New pattern for

Re: [Patch,testsuite] Fix testcases that use bind_pic_locally

2014-01-09 Thread Vidya Praveen
On Wed, Jan 08, 2014 at 12:28:56PM +, Jakub Jelinek wrote: > On Wed, Jan 08, 2014 at 11:49:08AM +0000, Vidya Praveen wrote: > > On Tue, Jan 07, 2014 at 09:35:54PM +, Mike Stump wrote: > > > On Dec 17, 2013, at 6:06 AM, Vidya Praveen wrote: > > > > bind_pic

Re: [Patch,testsuite] Fix testcases that use bind_pic_locally

2014-01-08 Thread Vidya Praveen
On Tue, Jan 07, 2014 at 09:35:54PM +, Mike Stump wrote: > On Dec 17, 2013, at 6:06 AM, Vidya Praveen wrote: > > bind_pic_locally is broken for targets that doesn't pass -fPIC/-fpic by > > default [1][2]. > > Let's give Jakub 2 days to weigh in? If no objectio

Re: [Patch,testsuite] Fix testcases that use bind_pic_locally

2014-01-07 Thread Vidya Praveen
Ping! On Tue, Dec 17, 2013 at 02:06:13PM +, Vidya Praveen wrote: > Hello, > > bind_pic_locally is broken for targets that doesn't pass -fPIC/-fpic by > default [1][2]. > > One of the suggestions was to have a effective target check called > bind_pic_

[Patch,testsuite] Fix testcases that use bind_pic_locally

2013-12-17 Thread Vidya Praveen
ll the testcases that use bind_pic_locally to include this bind_pic_locally_ok check. Tested for aarch64-none-elf, arm-none-eabi, arm-none-linux-gnueabihf. OK? Cheers VP. [1] http://gcc.gnu.org/ml/gcc/2013-09/msg00207.html [2] http://gcc.gnu.org/ml/gcc-patches/2013-10/msg00462.html gcc/testsuite/Chan

Re: Re: [Patch] Fix gcc.dg/20050922-*.c

2013-11-18 Thread Vidya Praveen
Mike, On 25/10/13 00:37, Mike Stump wrote: On Oct 24, 2013, at 2:26 AM, Vidya Praveen wrote: On Mon, Oct 21, 2013 at 06:40:28PM +0100, Mike Stump wrote: On Oct 21, 2013, at 3:28 AM, Vidya Praveen wrote: Tests gcc.dg/20050922-1.c and gcc.dg/20050922-2.c includes stdlib.h. This can be a

Re: [Patch] Fix gcc.dg/20050922-*.c

2013-10-24 Thread Vidya Praveen
On Mon, Oct 21, 2013 at 05:47:44PM +0100, Jeff Law wrote: > On 10/21/13 04:28, Vidya Praveen wrote: > > Hello, > > > > Tests gcc.dg/20050922-1.c and gcc.dg/20050922-2.c includes stdlib.h. This > > can > > be a issue especially since they define uint32_t.

Re: [Patch] Fix gcc.dg/20050922-*.c

2013-10-24 Thread Vidya Praveen
On Mon, Oct 21, 2013 at 06:40:28PM +0100, Mike Stump wrote: > On Oct 21, 2013, at 3:28 AM, Vidya Praveen wrote: > > Tests gcc.dg/20050922-1.c and gcc.dg/20050922-2.c includes stdlib.h. This > > can > > be a issue especially since they define uint32_t. > > > OK

[Patch] Fix gcc.dg/20050922-*.c

2013-10-21 Thread Vidya Praveen
-none-eabi and x86_64-linux-gnu OK for trunk, 4.7, 4.8? VP. --- gcc/testsuite/ChangeLog: 2013-10-21 Vidya Praveen * gcc.dg/20050922-1.c: Remove stdlib.h and declare abort(). * gcc.dg/20050922-1.c: Remove stdlib.h and declare abort() and exit(). diff --git a/gcc/testsuite

Re: [Patch] Fix the testcases that use bind_pic_locally

2013-10-08 Thread Vidya Praveen
On Tue, Oct 08, 2013 at 10:30:22AM +0100, Jakub Jelinek wrote: > On Tue, Oct 08, 2013 at 10:14:59AM +0100, Vidya Praveen wrote: > > There are several tests that use "dg-add-options bind_pic_locally" in order > > to > > add -fPIE or -fpie when -fPIC or -fpic

[Patch] Fix the testcases that use bind_pic_locally

2013-10-08 Thread Vidya Praveen
-09/msg00207.html This issue for obvious reasons, common to all targets. Tested for aarch64-none-elf. OK for trunk? Cheers VP --- gcc/testsuite/ChangeLog: 2013-10-08 Vidya Praveen * gcc.dg/inline-33.c: Remove bind_pic_locally and skip if -fPIC/-fpic is used.

[Patch,AArch64] Support SADDL/SSUBL/UADDL/USUBL

2013-09-30 Thread Vidya Praveen
, aarch64_be-none-elf with no regressions. OK for trunk? Cheers VP ~~~ gcc/ChangeLog: 2013-09-30 Vidya Praveen * aarch64-simd.md (aarch64_l2_internal): Rename to ... (aarch64_l_hi_internal): ... this; Insert '\t' to output template. (aarch64_l_l

Re: [Patch,AArch64] Support SISD Shifts (SHL/USHR/SSHL/USHL/SSHR)

2013-08-20 Thread Vidya Praveen
With the attachment this time :-) Regards VP On Tue, Aug 20, 2013 at 04:01:59PM +0100, Vidya Praveen wrote: > Hello, > > This patch supports SISD shift instructions SHL/USHR/SSHR/SSHL/USHL for > SImode and DImode. This patch also refactors the integer shifts pattern > "

[Patch,AArch64] Support SISD Shifts (SHL/USHR/SSHL/USHL/SSHR)

2013-08-20 Thread Vidya Praveen
: Likewise. (aarch64_sisd_ushl, aarch64_sisd_sshl): Likewise. (aarch64_ushl_2s, aarch64_sshl_2s, aarch64_sisd_neg_qi): Likewise. (ror3_insn): Likewise. * config/aarch64/predicates.md (aarch64_simd_register): New. gcc/testsuite/ChangeLog 2013-08-20 Vid

Re: [Patch] Fix selector for vect-iv-5.c

2013-08-01 Thread Vidya Praveen
Ping! On Tue, Jul 23, 2013 at 10:21:52AM +0100, Vidya Praveen wrote: > Hello > > gcc.dg/vect/vect-iv-5.c XPASSes for arm-*-* since gcc.dg/vect/*.c tests are > always run with -ffast-math for arm-*-*. This patch makes xfail conditional > for this test by adding effective

[Patch] Fix selector for vect-iv-5.c

2013-07-23 Thread Vidya Praveen
Hello gcc.dg/vect/vect-iv-5.c XPASSes for arm-*-* since gcc.dg/vect/*.c tests are always run with -ffast-math for arm-*-*. This patch makes xfail conditional for this test by adding effective target keyword !arm_neon_ok. OK for trunk? Regards VP -- gcc/testsuite/ChangeLog: 2013-07-22 Vidya

Added myself to MAINTAINERS (Write After Approval)

2013-06-14 Thread Vidya Praveen
2013-06-14 Vidya Praveen * MAINTAINERS (Write After Approval): Add myself. Index: MAINTAINERS === --- MAINTAINERS (revision 200091) +++ MAINTAINERS (working copy) @@ -487,6 +487,7 @@ Paul Pluzhnikov

Re: [AArch64] Support for SMLAL/SMLSL/UMLAL/UMLSL

2013-06-14 Thread Vidya Praveen
On 14/06/13 16:01, Richard Earnshaw wrote: On 14/06/13 15:33, Marcus Shawcroft wrote: On 14/06/13 14:55, Vidya Praveen wrote: [...] to support SMLAL/UMLAL instructions for 64 bit vector modes. * config/aarch64/aarch64-simd.md (*aarch64_mlsl): New pattern to support

[AArch64] Support for SMLAL/SMLSL/UMLAL/UMLSL

2013-06-14 Thread Vidya Praveen
Hello, This patch adds support to SMLAL/SMLSL/UMLAL/UMLSL instructions and adds tests for the same. Regression test run for aarch64-none-elf with no regressions. OK? ~VP --- gcc/ChangeLog 2013-06-14 Vidya Praveen * config/aarch64/aarch64-simd.md (*aarch64_mlal_lo): New

Re: [AArch64] Support for CLZ

2013-05-23 Thread Vidya Praveen
On 23/05/13 14:40, Marcus Shawcroft wrote: On 22 May 2013 12:47, Vidya Praveen wrote: Hello, This patch adds support to AdvSIMD CLZ instruction and adds tests for the same. Regression test done for aarch64-none-elf with no issues. OK? Regards VP --- gcc/ChangeLog 2013-05-22 Vidya

[AArch64] Support for CLZ

2013-05-22 Thread Vidya Praveen
Hello, This patch adds support to AdvSIMD CLZ instruction and adds tests for the same. Regression test done for aarch64-none-elf with no issues. OK? Regards VP --- gcc/ChangeLog 2013-05-22 Vidya Praveen * config/aarch64/aarch64-simd.md (clzv4si2): Support for CLZ

[AArch64] Fix the description of simd_fabd

2013-05-02 Thread Vidya Praveen
Hello, This attached patch corrects the description for simd_fabd. OK? Regards VP gcc/ChangeLog 2013-05-02 Vidya Praveen * config/aarch64/aarch64-simd.md (simd_fabd): Correct the description.diff --git a/gcc/config/aarch64/aarch64-simd.md b/gcc/config/aarch64/aarch64-simd.md

[AArch64] Support scalar form of FABD

2013-05-02 Thread Vidya Praveen
Hello, This attached patch adds support to the scalar form of FABD instruction along with the compile & execute tests for the same. Regression tested on aarch64-none-elf with no issues. OK? Regards VP --- gcc/ChangeLog 2013-05-02 Vidya Praveen * config/aarch64/aarch64-sim