On 2/25/21 7:08 PM, David Edelsohn wrote: > On Thu, Feb 25, 2021 at 8:05 PM Peter Bergner <berg...@linux.ibm.com> wrote: >> >> The initialization of compat builtins assumes the builtin we are creating >> a compatible builtin for exists and ICEs if it doesn't. However, there are >> valid reasons why some builtins are disabled for a particular compile. >> In this case, the MMA builtins are disabled for -mcpu=440 (and other cpus), >> so instead of ICEing, we should just skip adding the MMA compat builtin. >> >> This passed bootstrap and regtesting on powerpc64-linux, with running the >> testsuite in both 32-bit and 64-bit modes, with no regressions. >> Ok for mainline? > > Okay.
Thanks, pushed. On 2/25/21 7:09 PM, Segher Boessenkool wrote: > On Thu, Feb 25, 2021 at 07:05:26PM -0600, Peter Bergner wrote: >> The compat builtin patch was approved for backporting to GCC10, so we'll >> need this fix to go along with it. > > Okay for that as well of course. Thanks, I'll give this a day or two and then push the two to gcc10. Do you want them committed separately or squashed into one commit since the 2nd patch fixes the issue in the first patch? Peter
