Re: Remove redundant zero extend

2020-11-16 Thread Jeff Law via Gcc-patches
On 3/11/20 7:34 AM, Nidal Faour via Gcc-patches wrote: > This patch is a code density oriented and attempt to remove redundant > sign/zero extension from assignment statement. > The approach taken is to use VRP data while expanding the assignment to RTL > to determine whether a sign/zero extens

Re: Remove redundant zero extend

2020-03-13 Thread Jeff Law via Gcc-patches
On Thu, 2020-03-12 at 14:43 -0700, Jim Wilson wrote: > On Thu, Mar 12, 2020 at 2:38 AM Richard Biener via Gcc-patches > wrote: > > On Thu, Mar 12, 2020 at 4:06 AM Jeff Law via Gcc-patches > > wrote: > > > On Wed, 2020-03-11 at 13:04 +, Nidal Faour via Gcc-patches wrote: > > > > This patch is

Re: Remove redundant zero extend

2020-03-12 Thread Segher Boessenkool
Hi! On Thu, Mar 12, 2020 at 02:43:06PM -0700, Jim Wilson wrote: > I looked at combine because I'm familiar with that pass, but the ree > pass might be the right place to handle this. IMO, part of this should perhaps be done on Gimple already. But the part that should be done on RTL should be don

Re: Remove redundant zero extend

2020-03-12 Thread Jim Wilson
On Thu, Mar 12, 2020 at 2:38 AM Richard Biener via Gcc-patches wrote: > > On Thu, Mar 12, 2020 at 4:06 AM Jeff Law via Gcc-patches > wrote: > > > > On Wed, 2020-03-11 at 13:04 +, Nidal Faour via Gcc-patches wrote: > > > This patch is a code density oriented and attempt to remove redundant >

Re: Remove redundant zero extend

2020-03-12 Thread Richard Biener via Gcc-patches
On Thu, Mar 12, 2020 at 4:06 AM Jeff Law via Gcc-patches wrote: > > On Wed, 2020-03-11 at 13:04 +, Nidal Faour via Gcc-patches wrote: > > This patch is a code density oriented and attempt to remove redundant > > sign/zero > > extension from assignment statement. > > The approach taken is to u

Re: Remove redundant zero extend

2020-03-11 Thread Jeff Law via Gcc-patches
On Wed, 2020-03-11 at 13:04 +, Nidal Faour via Gcc-patches wrote: > This patch is a code density oriented and attempt to remove redundant > sign/zero > extension from assignment statement. > The approach taken is to use VRP data while expanding the assignment to RTL to > determine whether a si

RE: Remove redundant zero extend

2020-03-11 Thread Nidal Faour via Gcc-patches
: New test + 2020-03-09 Marek Polacek PR c++/92031 - bogus taking address of rvalue error. Regards, Nidal Faour >-Original Message- >From: David Malcolm >Sent: Wednesday, 11 March 2020 15:12 >To: Nidal Faour ; gcc-patches@gcc.gnu.org >Cc: Ofer Shinaar ; Craig B

Re: Remove redundant zero extend

2020-03-11 Thread David Malcolm via Gcc-patches
On Wed, 2020-03-11 at 13:04 +, Nidal Faour via Gcc-patches wrote: [...] > diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog > index f91af78a302..c5a701e08af 100644 > --- a/gcc/testsuite/ChangeLog > +++ b/gcc/testsuite/ChangeLog > @@ -1,3 +1,7 @@ > +2020-02-19 Nidal Faour > +