public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/94025] Expected-to-fail compilation goes through by not detecting mutable-specifier on lambda
       [not found] <bug-94025-4@http.gcc.gnu.org/bugzilla/>
@ 2020-04-14 20:16 ` daniel.kruegler at googlemail dot com
  2020-04-18 18:25 ` daniel.kruegler at googlemail dot com
  1 sibling, 0 replies; 2+ messages in thread
From: daniel.kruegler at googlemail dot com @ 2020-04-14 20:16 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94025

Daniel Krügler <daniel.kruegler at googlemail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |daniel.kruegler@googlemail.
                   |                            |com

--- Comment #1 from Daniel Krügler <daniel.kruegler at googlemail dot com> ---
In my opinion, this issue does not demonstrate a bug, but is based on an
incomplete analysis of what is going on here. 

1) It is correct, that the lambda function call operator is non-const in this
case. The result is that the function call operator of the lambda expression
will *not* be called in the shown example.

2) We have here a lambda expression without any capture. This means that the
standard requires the existence of an *additional* conversion function to a
pointer to function ([expr.prim.lambda.closure] p7 quoted from N4849). And
[expr.prim.lambda.closure] p11 says:

"The conversion function [..] is public, constexpr, non-virtual, non-explicit,
const, and has a non-throwing exception specification (14.5)."

So effectively a second function call resolution is in affect here, selecting
the conversion function (which is a const member function as specified above)
to function pointer as the only viable candidate (If both were viable, the
conversion function would be less preferred) via the surrogate call function
([over.call.object]). That explains IMO why the code is well-formed.

If you would try to mimic that with a user-defined class type, it would look
similar to the following one:

struct Lambda
{
  using f_t = void();
  f_t operator(); // "mutable"
  using fptr_t = f_t*;
  operator fptr_t() const;
};

Note that I use here the very rarely used syntax to declare (but not define) a
member function using a typedef for a function type to show the involved
function types more precisely.

The example would become invalid once you introduce a capture, because in this
case there would be no conversion function anymore.

I'm surprised that the Visual Studio compiler (I tested 2019) rejects the
original example, this looks like a bug to me, especially since that compiler
also handles the call resolution for the above defined Lambda type correctly. I
plan to report an issue for that compiler.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Bug c++/94025] Expected-to-fail compilation goes through by not detecting mutable-specifier on lambda
       [not found] <bug-94025-4@http.gcc.gnu.org/bugzilla/>
  2020-04-14 20:16 ` [Bug c++/94025] Expected-to-fail compilation goes through by not detecting mutable-specifier on lambda daniel.kruegler at googlemail dot com
@ 2020-04-18 18:25 ` daniel.kruegler at googlemail dot com
  1 sibling, 0 replies; 2+ messages in thread
From: daniel.kruegler at googlemail dot com @ 2020-04-18 18:25 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94025

--- Comment #2 from Daniel Krügler <daniel.kruegler at googlemail dot com> ---
(In reply to Daniel Krügler from comment #1)
[..]
> I'm surprised that the Visual Studio compiler (I tested 2019) rejects the
> original example, this looks like a bug to me, especially since that
> compiler also handles the call resolution for the above defined Lambda type
> correctly. I plan to report an issue for that compiler.

I have opened a corresponding bug report against the VS 2019 compiler:

https://developercommunity.visualstudio.com/content/problem/990374/conversion-function-to-function-pointer-for-mutabl-1.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-04-18 18:25 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-94025-4@http.gcc.gnu.org/bugzilla/>
2020-04-14 20:16 ` [Bug c++/94025] Expected-to-fail compilation goes through by not detecting mutable-specifier on lambda daniel.kruegler at googlemail dot com
2020-04-18 18:25 ` daniel.kruegler at googlemail dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).