This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH], Fix PR target/85424, PowerPC __builtin_packlongdouble bug discovered in PR target/85075
- 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, 16 Apr 2018 16:41:50 -0500
- Subject: Re: [PATCH], Fix PR target/85424, PowerPC __builtin_packlongdouble bug discovered in PR target/85075
- References: <20180416191425.GA26180@ibm-tiger.the-meissners.org>
On Mon, Apr 16, 2018 at 03:14:25PM -0400, Michael Meissner wrote:
> With the code changes in the patch for PR target/85075, I noticed that the
> PowerPC big-endian build stopped in building big endian, 32-bit libgcc when
> configured for power8. The issue was this latent bug. Segher asked me to
> re-submit the bug separately, and I'm doing this for PR target/85424.
>
> This is the patch that I did in PR target/85075. For that set, I did bootstrap
> builds for big endian power7 and little endian power8 (along with a previous
> build on a big endian power8 system).
>
> This patch will need to be back ported to GCC 7. I suspect that it may not be
> needed in GCC 6, since that revision used RELOAD instead of LRA. But given it
> is hard to replicate the bug, I will also back port it to GCC 6.
>
> Can I apply this to GCC 8, and after a waiting period, apply it to GCC 7 and
> GCC 6?
This looks fine. Yes please. Thanks!
Segher
> 2018-04-15 Michael Meissner <meissner@linux.vnet.ibm.com>
>
> PR target/85424
> * config/rs6000/rs6000.md (pack<mode>): Do not try handle a pack
> where the inputs overlap with the output.