RE: [PATCH PR96053] Add "#pragma GCC no_reduc_chain"

Sorry for the late reply! -Original Message- From: Richard Biener [mailto:rguent...@suse.de] Sent: Wednesday, July 22, 2020 3:02 PM > First of all I think giving users more control over vectorization is > good. Now as for "#pragma GCC no_reduc_chain" I'd like to avoid > negatives and t

[PATCH PR96053] Add "#pragma GCC no_reduc_chain"

Hi, It is the patch to add "#pragma GCC no_reduc_chain" for pr96053. It only completes the front end of C language. For the testcase, it successfully skipped doing slp by finding sequences from reduction chains. Without "#pragma GCC no_reduc_chain", it will fail to do vectorization. Please

RE: [PATCH] pass correct parameters to c_parser_do_statement

Sorry for my mistake. The previous patch description is incorrect. A new patch was attached. Can anyone help me install this patch? Thanks, Kaipeng Zhou > -Original Message- > From: zhoukaipeng (A) > Sent: Tuesday, July 7, 2020 11:26 AM > To: gcc-patches@gcc.gnu.org >

[PATCH] pass correct parameters to c_parser_do_statement

Hi, It is a patch to pass correct parameters to c_parser_do_statement. Can anyone help me install this patch? Thanks, Kaipeng Zhou code-cleanup-v1.diff Description: code-cleanup-v1.diff

RE: [PATCH PR95854] ICE in find_bswap_or_nop_1 of pass store-merging

> On Sun, 28 Jun 2020, zhoukaipeng (A) wrote: > > > Hi, > > > > Thanks for your good suggestions! > > > > This patch was remade and attached. Does the v2 patch look better? > > > > Bootstrap and new testcase tested on aarch64 & x86 Linux pl

RE: [PATCH PR95854] ICE in find_bswap_or_nop_1 of pass store-merging

Hi, Thanks for your good suggestions! This patch was remade and attached. Does the v2 patch look better? Bootstrap and new testcase tested on aarch64 & x86 Linux platform. Kaipeng Zhou PR95854-v2.diff Description: PR95854-v2.diff

[PATCH PR95854] ICE in find_bswap_or_nop_1 of pass store-merging

Hi, This is a fix for pr95854. Only add a judgement to make sure operand1 and operand2 are both INTEGER_CST. Bootstrap and tested on aarch64 Linux platform. No new regression witnessed. Is it ok to be merged? Thanks, Kaipeng Zhou 0001-store-merging-ICE-in-find_bswap_or_nop_1-PR95854.patch De

RE: [PATCH PR95199 v2] vect: CSE for bump and offset in strided load/store operations

Bootstrapped and tested on aarch64-linux-gnu & x86_64-linux-gnu. Could you please help install this patch? Kaipeng Zhou > -Original Message- > From: Richard Biener [mailto:rguent...@suse.de] > Sent: Tuesday, June 16, 2020 5:49 PM > To: zhoukaipeng (A) > Cc: Rich

[PATCH PR95199 v2] vect: CSE for bump and offset in strided load/store operations

0 2:21 PM > To: zhoukaipeng (A) > Cc: Richard Sandiford ; gcc- > patc...@gcc.gnu.org; am...@gcc.gnu.org; rgue...@gcc.gnu.org > Subject: RE: [PATCH PR95199] vect: Remove extra variable created for > memory reference > > On Sun, 14 Jun 2020, zhoukaipeng (A) wrote: > > &g

RE: [PATCH PR95199] vect: Remove extra variable created for memory reference

ge- > From: Richard Sandiford [mailto:richard.sandif...@arm.com] > Sent: Friday, June 12, 2020 3:48 PM > To: zhoukaipeng (A) > Cc: gcc-patches@gcc.gnu.org; rguent...@suse.de; am...@gcc.gnu.org; > rgue...@gcc.gnu.org > Subject: Re: [PATCH PR95199] vect: Remove extra variable created for > m

[PATCH PR95199] vect: Remove extra variable created for memory reference

Hi, This is a fix for pr95199. In vectorization, vinfo->ivexpr_map is supposed to catch those "IV base and/or step expressions". Just call cse_and_gimplify_to_preheader to handle gathering/scattering to avoid the extra variable. Bootstrap and tested on aarch64/x86_64 Linux platform. No new reg