Hi Mike, On Wed, Mar 15, 2017 at 06:31:12PM -0400, Michael Meissner wrote: > This patch is a simpler approach to fixing PR 71294, then I proposed last > year.
Much simpler! Thank you. > I have built compilers on a little endian Power8 system (64-bit only), big > endian Power8 system (64-bit only) and big endian Power7 system (32-bit and > 64-bit). At the moment, the two power8 systems have no regressions, and the > power7 system is still building. Assuming the power7 system completes without > regressions, can I check this patch into the trunk? Yes please. > Assuming the patch applies and causes no regressions, can I check this patch > into GCC 6 and 5 as well. Since GCC 6/5 do run with reload as the default, > they would likely see the issue. GCC 7 has switched to LRA, which is more > robust, and doesn't show the problem (unless you use the -mno-lra switch). Okay after a week or so of soaking. Segher
