This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 1/6] ifcvt: Store the number of created cmovs.

From: Jeff Law <law at redhat dot com>
To: Robin Dapp <rdapp at linux dot ibm dot com>, gcc-patches at gcc dot gnu dot org
Cc: krebbel at linux dot ibm dot com, iii at linux dot ibm dot com
Date: Wed, 14 Nov 2018 13:22:49 -0700
Subject: Re: [PATCH 1/6] ifcvt: Store the number of created cmovs.
References: <20181114130752.5057-1-rdapp@linux.ibm.com> <20181114130752.5057-2-rdapp@linux.ibm.com>

On 11/14/18 6:07 AM, Robin Dapp wrote:
> This patch saves the number of created conditional moves by
> noce_convert_multiple_sets in the IF_INFO struct.  This may be used by
> the backend to easier decide whether to accept a generated sequence or
> not.
> 
> --
> 
> gcc/ChangeLog:
> 
> 2018-11-14  Robin Dapp  <rdapp@linux.ibm.com>
> 
> 	* ifcvt.c (noce_convert_multiple_sets): Set cmov count.
> 	(noce_find_if_block): Set cmov count.
> 	* ifcvt.h (struct noce_if_info): Add cmov count.
So this series came in after stage1 close.    I'm not aware of a bug
this series is meant to fix, but perhaps you just didn't include a
reference to it.

Anyway, patches #1 and #3 are OK for the trunk right now.

Jeff

References:
- [PATCH 0/6] If conversion with multiple sets.
  - From: Robin Dapp
- [PATCH 1/6] ifcvt: Store the number of created cmovs.
  - From: Robin Dapp

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]