[Bug c++/94771] g++.dg/concepts/diagnostic10.C fails on mingw

2020-04-28 Thread sbence92 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94771

--- Comment #4 from Bence Szabó  ---
Interesting, the t.f syntax is not allowed anymore even with MSVC:
https://godbolt.org/z/YCK-mv

msvc also doesn't list under extensions_
https://docs.microsoft.com/en-us/cpp/build/reference/microsoft-extensions-to-c-and-cpp?view=vs-2015

and some recent overview on ms extensions:
https://stackoverflow.com/questions/56554567/what-does-the-ms-extensions-flag-do-exactly-with-gcc

Is this a dinosaur that forgot to die?

[Bug c++/94771] g++.dg/concepts/diagnostic10.C fails on mingw

2020-04-27 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94771

Jonathan Wakely  changed:

   What|Removed |Added

 CC|iains at gcc dot gnu.org   |jason at gcc dot gnu.org

--- Comment #3 from Jonathan Wakely  ---
CC Jason to ask why the change does:

+  /* Don't enforce this in MS mode.  */
+  if (flag_ms_extensions)
+return false;

[Bug c++/94771] g++.dg/concepts/diagnostic10.C fails on mingw

2020-04-27 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94771

Jonathan Wakely  changed:

   What|Removed |Added

 Status|WAITING |NEW

--- Comment #2 from Jonathan Wakely  ---
The fix for Bug 52597 does seem related, see the change to
invalid_nonstatic_memfn_p in r197131.

I don't know what the MS extension is that requires the t.f syntax to be
allowed. Maybe the extension should be disabled in a requires-expression?

[Bug c++/94771] g++.dg/concepts/diagnostic10.C fails on mingw

2020-04-27 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94771

Iain Sandoe  changed:

   What|Removed |Added

 Status|UNCONFIRMED |WAITING
 Ever confirmed|0   |1
   Last reconfirmed||2020-04-27

--- Comment #1 from Iain Sandoe  ---
the commit you mention appears to be unrelated to the test fail, please could
you identify commit the failure started?