This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR24574
- From: Alexander Monakov <amonakov at ispras dot ru>
- To: Richard Biener <rguenther at suse dot de>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 13 Jul 2016 17:08:11 +0300 (MSK)
- Subject: Re: [PATCH] Fix PR24574
- Authentication-results: sourceware.org; auth=none
- References: <alpine.LSU.2.11.1607131550440.30444@t29.fhfr.qr>
Hi,
On Wed, 13 Jul 2016, Richard Biener wrote:
> The following adds the ability to transform
>
> if (x != 0)
> x = x / 10;
>
> to
>
> x = x / 10;
>
> as requested by PR. Plus it adds some more ops where such transform
> is possible.
In the bugzilla, you said,
> Only for -Os, it's better to avoid the expensive division otherwise.
But the patch seems to apply the transform irrespective of -Os; is that
deliberate? (fwiw I agree with your bugzilla comment above)
Thanks.
Alexander