public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug sanitizer/90746] __sanitizer_cov_trace_pc should not be tail called
       [not found] <bug-90746-4@http.gcc.gnu.org/bugzilla/>
@ 2023-05-03 16:51 ` cvs-commit at gcc dot gnu.org
  2023-05-03 16:52 ` amonakov at gcc dot gnu.org
  1 sibling, 0 replies; 2+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-03 16:51 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Alexander Monakov <amonakov@gcc.gnu.org>:

https://gcc.gnu.org/g:cef0c0bb13e2953b41caca0506ab1d41c56f29de

commit r14-457-gcef0c0bb13e2953b41caca0506ab1d41c56f29de
Author: Alexander Monakov <amonakov@ispras.ru>
Date:   Thu Jan 19 19:25:04 2023 +0300

    do not tailcall __sanitizer_cov_trace_pc [PR90746]

    When instrumentation is requested via -fsanitize-coverage=trace-pc, GCC
    emits calls of __sanitizer_cov_trace_pc callback in each basic block.
    This callback is supposed to be implemented by the user, and should be
    able to identify the containing basic block by inspecting its return
    address. Tailcalling the callback prevents that, so disallow it.

    gcc/ChangeLog:

            PR sanitizer/90746
            * calls.cc (can_implement_as_sibling_call_p): Reject calls
            to __sanitizer_cov_trace_pc.

    gcc/testsuite/ChangeLog:

            PR sanitizer/90746
            * gcc.dg/sancov/basic0.c: Verify absence of tailcall.

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

* [Bug sanitizer/90746] __sanitizer_cov_trace_pc should not be tail called
       [not found] <bug-90746-4@http.gcc.gnu.org/bugzilla/>
  2023-05-03 16:51 ` [Bug sanitizer/90746] __sanitizer_cov_trace_pc should not be tail called cvs-commit at gcc dot gnu.org
@ 2023-05-03 16:52 ` amonakov at gcc dot gnu.org
  1 sibling, 0 replies; 2+ messages in thread
From: amonakov at gcc dot gnu.org @ 2023-05-03 16:52 UTC (permalink / raw)
  To: gcc-bugs

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

Alexander Monakov <amonakov at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |amonakov at gcc dot gnu.org
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Alexander Monakov <amonakov at gcc dot gnu.org> ---
Fixed for gcc-14.

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

end of thread, other threads:[~2023-05-03 16:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-90746-4@http.gcc.gnu.org/bugzilla/>
2023-05-03 16:51 ` [Bug sanitizer/90746] __sanitizer_cov_trace_pc should not be tail called cvs-commit at gcc dot gnu.org
2023-05-03 16:52 ` amonakov at gcc dot gnu.org

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).