On Thu, May 21, 2015 at 4:10 AM, Jakub Jelinek <ja...@redhat.com> wrote: > On Thu, May 21, 2015 at 01:02:12PM +0200, Thomas Schwinge wrote: >> Hi! >> >> On Thu, 7 May 2015 19:32:26 +0100, Julian Brown <jul...@codesourcery.com> >> wrote: >> > Here's a new version of the patch [...] >> >> > OK for trunk? >> >> Makes sense to me (with just a request to drop the testsuite changes, see >> below), to get the existing regressions under control. Jakub? > > Ok for trunk. >> >> > PR libgomp/65742 >> > >> > gcc/ >> > * builtins.c (expand_builtin_acc_on_device): Don't use open-coded >> > sequence for !ACCEL_COMPILER. >> >
It breaks bootstrap on x86: https://gcc.gnu.org/ml/gcc-regression/2015-05/msg00389.html I checked in this to fix it. -- H.J. --- Index: gcc/ChangeLog =================================================================== --- gcc/ChangeLog (revision 223804) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,8 @@ +2015-05-28 H.J. Lu <hongjiu...@intel.com> + + * builtins.c (expand_builtin_acc_on_device): Mark parameters + with ATTRIBUTE_UNUSED. + 2015-05-28 Julian Brown <jul...@codesourcery.com> PR libgomp/65742 Index: gcc/builtins.c =================================================================== --- gcc/builtins.c (revision 223804) +++ gcc/builtins.c (working copy) @@ -5911,7 +5911,8 @@ acceleration device (ACCEL_COMPILER conditional). */ static rtx -expand_builtin_acc_on_device (tree exp, rtx target) +expand_builtin_acc_on_device (tree exp ATTRIBUTE_UNUSED, + rtx target ATTRIBUTE_UNUSED) { #ifdef ACCEL_COMPILER if (!validate_arglist (exp, INTEGER_TYPE, VOID_TYPE))
