Re: [PATCH] Factor out division by squares and remove division around comparisons (2/2)

2017-11-19 Thread Jeff Law
On 09/06/2017 03:55 AM, Jackson Woodruff wrote: > Hi all, > > A minor improvement came to mind while updating other parts of this patch. > > I've updated a testcase to make it more clear and a condition now uses a > call to is_division_by rather than manually checking those conditions. > >

Re: [PATCH] Factor out division by squares and remove division around comparisons (2/2)

2017-11-16 Thread Wilco Dijkstra
ping From: Jackson Woodruff <jackson.woodr...@foss.arm.com> Sent: 06 September 2017 10:55 To: Richard Biener Cc: Wilco Dijkstra; kyrylo.tkac...@foss.arm.com; Joseph S. Myers; GCC Patches Subject: Re: [PATCH] Factor out division by squares and remove division around comparisons (2/2)  

Re: [PATCH] Factor out division by squares and remove division around comparisons (2/2)

2017-09-06 Thread Jackson Woodruff
Hi all, A minor improvement came to mind while updating other parts of this patch. I've updated a testcase to make it more clear and a condition now uses a call to is_division_by rather than manually checking those conditions. Jackson On 08/30/2017 05:32 PM, Jackson Woodruff wrote: Hi all,

Re: [PATCH] Factor out division by squares and remove division around comparisons (2/2)

2017-08-30 Thread Jackson Woodruff
Hi all, I've attached a new version of the patch in response to a few of Wilco's comments in person. The end product of the pass is still the same, but I have fixed several bugs. Now tested independently of the other patches. On 08/15/2017 03:07 PM, Richard Biener wrote: On Thu, Aug 10,

Re: [PATCH] Factor out division by squares and remove division around comparisons (2/2)

2017-08-15 Thread Richard Biener
On Thu, Aug 10, 2017 at 4:10 PM, Jackson Woodruff wrote: > Hi all, > > The patch implements the some of the division optimizations discussed in > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71026 . > > We now reassociate (as discussed in the bug report): > > x