On Mon, May 18, 2020 at 10:04:07PM +0200, Jakub Jelinek wrote: > On Mon, May 18, 2020 at 03:51:36PM -0400, Jason Merrill wrote: > > > --- a/libgomp/testsuite/libgomp.c++/for-27.C > > > +++ b/libgomp/testsuite/libgomp.c++/for-27.C > > > @@ -151,6 +151,12 @@ f4 (const I<int> &x, const I<int> &y) > > > else if (results[i]) \ > > > abort () > > > +// FIXME: OpenMP seems to count on this being instantiated; the copy > > > +// constructor is called in functions fn2/fn3/fn4. But in C++17 we > > > +// elide this copy constructor, so it's never been instantiated. For > > > +// now instantiate it explicitly. > > > > This sounds like a bug that needs to be fixed before the default can change. > > I guess the openmp code needs to mark_used the copy constructor or the like. > > I'll have a look, though no guarantee I manage it this week.
Thanks, do you want a PR for it? Marek
