On Fri, 2018-02-16 at 11:56 -0600, Segher Boessenkool wrote: > Hi! Segher:
Per our discussion on IRC. I created the bugzilla: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84422 for this issue so it is visible to people outside IBM. I have included this test file int the above bugzilla along with the other four builtin test case files that ICE when compiled with the -mcpu=power7 option. > On Fri, Feb 16, 2018 at 08:58:49AM -0800, Carl Love wrote: > > This patch changes the expansion macros to the equivalent Power 8 > > macros. The test now cleanly exits with the message > > > > error: builtin function ‘__builtin_vsx_float2_v2di’ requires > > the ‘- > > mpower8-vector’ option > > > > rather then giving an internal compiler error when compiled with > > the > > cpu=power7 option. > > :-) > > > The patch was tested by running the full regression suite to ensure > > no > > new regressions were introduced. Additionally, the patch was > > tested by > > hand compiling with the -mcpu=power7 option to verify the issue is > > fixed. The testing was don on powerpc64le-unknown-linux-gnu > > (Power 8 > > LE) > > Do you have a new testcase, too? Or is this all covered by existing > tests (which?) The issue was found when compiling the existing test case file with -mcpu=power7. No additional tests are needed for this issue. > > Looks great, thanks! Okay for trunk (and for backports after a > while, > if needed). Carl Love
