[Bug c/66516] missing diagnostic on taking the address of a builtin function

[Bug c/66516] missing diagnostic on taking the address of a builtin function

[msebor at gcc dot gnu.org]

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

Martin Sebor <msebor at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2015-06-15
           Assignee|unassigned at gcc dot gnu.org      |msebor at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #1 from Martin Sebor <msebor at gcc dot gnu.org> ---
Working on a patch.

[Bug c/66516] missing diagnostic on taking the address of a builtin function

[msebor at gcc dot gnu.org]

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

--- Comment #2 from Martin Sebor <msebor at gcc dot gnu.org> ---
Patch posted for review:
https://gcc.gnu.org/ml/gcc-patches/2015-06/msg02043.html

[Bug c/66516] missing diagnostic on taking the address of a builtin function

[msebor at gcc dot gnu.org]

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

--- Comment #3 from Martin Sebor <msebor at gcc dot gnu.org> ---
Author: msebor
Date: Thu Sep  3 16:23:11 2015
New Revision: 227458

URL: https://gcc.gnu.org/viewcvs?rev=227458&root=gcc&view=rev
Log:
gcc/ChangeLog
2015-09-03  Martin Sebor  <msebor@redhat.com>

        PR c/66516
        * doc/extend.texi (Other Builtins): Document when the address
        of a built-in function can be taken.

gcc/c-family/ChangeLog
2015-09-03  Martin Sebor  <msebor@redhat.com>

        PR c/66516
        * c-common.h (c_decl_implicit, reject_gcc_builtin): Declare new
        functions.
        * c-common.c (reject_gcc_builtin): Define.

gcc/c/ChangeLog
2015-09-03  Martin Sebor  <msebor@redhat.com>

        PR c/66516
        * c/c-typeck.c (convert_arguments, parser_build_unary_op)
        (build_conditional_expr, c_cast_expr, convert_for_assignment)
        (build_binary_op, _objc_common_truthvalue_conversion): Call
        reject_gcc_builtin.
        (c_decl_implicit): Define.

gcc/cp/ChangeLog
2015-09-03  Martin Sebor  <msebor@redhat.com>

        PR c/66516
        * cp/cp-tree.h (mark_rvalue_use, decay_conversion): Add new
        argument(s).
        * cp/expr.c (mark_rvalue_use): Use new argument.
        * cp/call.c (build_addr_func): Call decay_conversion with new
        argument.
        * cp/pt.c (convert_template_argument): Call reject_gcc_builtin.
        * cp/typeck.c (decay_conversion): Use new argument.
        (c_decl_implicit): Define.

gcc/testsuite/ChangeLog
2015-09-03  Martin Sebor  <msebor@redhat.com>

        PR c/66516
        * g++.dg/addr_builtin-1.C: New test.
        * gcc.dg/addr_builtin-1.c: New test.

Added:
    trunk/gcc/builtins.c.orig
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/c-family/ChangeLog
    trunk/gcc/c-family/c-common.c
    trunk/gcc/c-family/c-common.h
    trunk/gcc/c/ChangeLog
    trunk/gcc/c/c-typeck.c
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/call.c
    trunk/gcc/cp/cp-tree.h
    trunk/gcc/cp/expr.c
    trunk/gcc/cp/pt.c
    trunk/gcc/cp/typeck.c
    trunk/gcc/doc/extend.texi
    trunk/gcc/testsuite/ChangeLog

[Bug c/66516] missing diagnostic on taking the address of a builtin function

[msebor at gcc dot gnu.org]

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

Martin Sebor <msebor at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |6.0

--- Comment #4 from Martin Sebor <msebor at gcc dot gnu.org> ---
Fixed on trunk.