RE: [PATCH] PR61868

Thanks, Jeff. Checked in. Bingfeng > -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: 31 July 2014 04:43 > To: Bingfeng Mei; Richard Biener > Cc: Andi Kleen; gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] PR61868 > > On 07/30/14 09:20, Bin

RE: [PATCH] PR61868

Yes, that fix is better. Here are updated patches. LTO-bootstrapped and tested. OK? Bingfeng Index: ChangeLog === --- ChangeLog (revision 213152) +++ ChangeLog (working copy) @@ -1,3 +1,10 @@ +2014-07-29 Bingfeng Mei

RE: [PATCH] PR61868

; flag_random_seed = val; + init_random_seed (); return old; } Bingfeng > -Original Message- > From: Richard Biener [mailto:richard.guent...@gmail.com] > Sent: 30 July 2014 10:43 > To: Bingfeng Mei > Cc: Andi Kleen; gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] PR61868 > > On

RE: [PATCH] PR61868

It did take a lot of time, but passed the LTO bootstrap. Bingfeng > -Original Message- > From: Richard Biener [mailto:richard.guent...@gmail.com] > Sent: 30 July 2014 10:43 > To: Bingfeng Mei > Cc: Andi Kleen; gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] PR61868 &

RE: [PATCH] PR61868

? Bingfeng > -Original Message- > From: Richard Biener [mailto:richard.guent...@gmail.com] > Sent: 30 July 2014 08:47 > To: Bingfeng Mei; Andi Kleen > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] PR61868 > > On Tue, Jul 29, 2014 at 3:01 PM, Bingfeng Mei wrote:

[PATCH] PR61868

, Bingfeng Mei === --- ChangeLog (revision 213152) +++ ChangeLog (working copy) @@ -1,3 +1,9 @@ +2014-07-29 Bingfeng Mei + + PR lto/61868 + * toplev.c (init_random_seed): Generate random seed when + flag_random_seed is

RE: [PATCH] default_add_stmt_cost should call target specific builitin_vectorization_cost.

Thanks. Committed. Bingfeng -Original Message- From: Richard Biener [mailto:richard.guent...@gmail.com] Sent: 06 June 2014 10:40 To: Bingfeng Mei Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] default_add_stmt_cost should call target specific builitin_vectorization_cost. On Fri, Jun

[PATCH] default_add_stmt_cost should call target specific builitin_vectorization_cost.

TARGET_VECTORIZE_BUILTIN_VECTORIZATION_COST is defined, it will use that cost. Otherwise, it will fall back to default_builtin_vectorization_cost. Bootstrapped and tested. OK for trunk? Thanks, Bingfeng Mei Index: gcc/ChangeLog

RE: VEC_WIDEN_MULT_(LO|HI)_EXPR vs. VEC_WIDEN_MULT_(EVEN|ODD)_EXPR in vectorization.

) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,9 @@ +2014-01-28 Bingfeng Mei + + * doc/md.texi: Mention that a target shouldn't implement + vec_widen_(s|u)mul_even/odd pair if it is less efficient + than hi/lo pair. + 2014-01-28 Richard Biener Revert Index: g

[COMMITTED] [PATCH] fixed pr59651 & new test case

Jakub, thanks. Committed with suggested changes. Bingfeng -Original Message- From: Jakub Jelinek [mailto:ja...@redhat.com] Sent: 03 January 2014 14:26 To: Bingfeng Mei Cc: gcc-patches@gcc.gnu.org; tbela...@arm.com Subject: Re: [PING] [PATCH] fixed pr59651 & new test case On Fri, Ja

[PING] [PATCH] fixed pr59651 & new test case

Updated: the patch passes aarch64 tests. Added missing ChangeLog entry for testsuite. OK to commit? Bingfeng -Original Message- From: Bingfeng Mei Sent: 02 January 2014 15:02 To: gcc-patches@gcc.gnu.org Cc: 'tbela...@arm.com' Subject: [PATCH] fixed pr59651 & new test c

[PATCH] fixed pr59651 & new test case

Hi, This patch fixes pr59651. The original regression for pr52943 only appears on AArch64 target. I constructed a similar test that also exposes bug on x86-64. The problem is that calculation of address range in alias versioning for loops with negative step is wrong during vectorization. For ex

[COMMITTED] RE: [PATCH] [followup to PR59569] new vect tests for store with negative step

Thanks. Committed with suggested change. Merry Christmas! Bingfeng -Original Message- From: Jakub Jelinek [mailto:ja...@redhat.com] Sent: 23 December 2013 16:48 To: Bingfeng Mei Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] [followup to PR59569] new vect tests for store with

[PATCH] [followup to PR59569] new vect tests for store with negative step

Hi, Here are two vectorization tests for store with negative step. This is follow-up to PR59569 fix, which contains two tests for ICE. These tests are for vectorization tests and executable. OK to commit? Thanks, Bingfeng patch_vect_tests Description: patch_vect_tests

[COMMITTED]RE: [PATCH] Fix PR59569

Committed. I will prepare some new tests as you suggested. Thanks, Bingfeng -Original Message- From: Jakub Jelinek [mailto:ja...@redhat.com] Sent: 23 December 2013 14:53 To: Bingfeng Mei Cc: gcc-patches@gcc.gnu.org; H.J. Lu (hjl.to...@gmail.com) Subject: Re: [PATCH] Fix PR59569 On Mon

RE: [PATCH] Fix PR59569

All the 3 tests are tested and the first two are included in my patch. Didn't include the third one as it is not reduced. Bingfeng -Original Message- From: H.J. Lu [mailto:hjl.to...@gmail.com] Sent: 23 December 2013 14:28 To: Bingfeng Mei Cc: gcc-patches@gcc.gnu.org; Jakub Jeline

[PATCH] Fix PR59569

Hi, Jakub, Thanks for suggestion. Please find attached patch. GCC is bootstrapped and passes testsuite on x86-64. Let me know if it is OK to commit. (Sorry if you received this mail twice as I forgot to set to text format). Thanks, Bingfeng Mei patch_pr59569 Description: patch_pr59569

[PING] RE: [PATCH] Vectorization for store with negative step

OK to commit? Thanks, Bingfeng -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Bingfeng Mei Sent: 18 December 2013 16:25 To: Jakub Jelinek Cc: Richard Biener; gcc-patches@gcc.gnu.org Subject: RE: [PATCH] Vectorization for store

RE: [PATCH] Vectorization for store with negative step

Hi, Jakub, Sorry for all the formatting issues. Haven't submit a patch for a while :-). Please find the updated patch. Thanks, Bingfeng -Original Message- From: Jakub Jelinek [mailto:ja...@redhat.com] Sent: 18 December 2013 13:38 To: Bingfeng Mei Cc: Richard Biener; gcc-pa

[PATCH] Vectorization for store with negative step

Hi, I created PR59544 and here is the patch. OK to commit? Thanks, Bingfeng 2013-12-18 Bingfeng Mei PR tree-optimization/59544 * tree-vect-stmts.c (perm_mask_for_reverse): Move before vectorizable_store. (vectorizable_store): Handle negative step. 2013-12-18 Bingfeng

RE: Vectorization for store with negative step

:26 To: Bingfeng Mei Cc: gcc-patches@gcc.gnu.org Subject: Re: Vectorization for store with negative step On Mon, Dec 16, 2013 at 5:54 PM, Bingfeng Mei wrote: > Hi, > I was looking at some loops that can be vectorized by LLVM, but not GCC. One > type of loop is with store of negative step.

Vectorization for store with negative step

B0_1 # BB#2: # %for.end ret Any comment? Bingfeng Mei Broadcom UK patch Description: patch

RE: [PATCH] correct TARGET_ADDR_SPACE_SUBSET_P documentation.

Richard, Here are updated patches. tm.texi.in doesn't need any change as the description of the hook is correct, just the order of parameters needs change. Thanks, Bingfeng 2011-09-26 Bingfeng Mei * doc/tm.texi: Correct documentation for TARGET_ADDR_SPACE_SUB

[PATCH] correct TARGET_ADDR_SPACE_SUBSET_P documentation.

or trunk? Cheers, Bingfeng Mei 2011-09-26 Bingfeng Mei * doc/tm.texi: Correct documentation for TARGET_ADDR_SPACE_SUBSET_P. Index: doc/tm.texi === --- doc/tm.texi (revision 178972) +++ doc/tm.texi (working copy) @@ -10256,7

RE: [PATCH] derive alias information from named address spaces.

Hello, Here is the updated patch. OK now? Thanks, Bingfeng 2011-09-22 Bingfeng Mei * alias.c (mems_in_disjoint_address_spaces_p) New function. (nonoverlapping_memrefs_p): Use mems_in_disjoint_address_sapces_p to derive alias information

[PATCH] derive alias information from named address spaces.

Hello, The following patch is to derive more alias information from named address spaces using existing target hook. It was discussed in http://gcc.gnu.org/ml/gcc/2011-09/msg00180.html Bootstrapped and tested on x86-64. OK for trunk? Thanks, Bingfeng 2011-09-20 Bingfeng Mei