This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Date: Mon, 5 Feb 2018 05:57:25 -0600
- Subject: Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression
- Authentication-results: sourceware.org; auth=none
- References: <20180201193116.GA15164@ibm-tiger.the-meissners.org>
On Thu, Feb 01, 2018 at 02:31:17PM -0500, Michael Meissner wrote:
> This patch fixes the optimization regression that occurred on GCC 7 where
> conversions from the various floating point types to small integers would at
> times generate a store and a load.
[ snip big explanation; thanks for that! ]
Could you merge the signed and unsigned patterns, using any_fix? Or is
there a reason that cannot work (other than that <su> unsigned_fix seems
buggy, it should say "u")?
Okay for trunk even without that (but please try). Also okay for 7 after
looking for fallout.
Thanks!
Segher