]> git.ipfire.org Git - thirdparty/gcc.git/commit
Update c-c++-common/tsan/atomic_stack.c
authorH.J. Lu <hjl.tools@gmail.com>
Mon, 27 Sep 2021 23:25:52 +0000 (16:25 -0700)
committerH.J. Lu <hjl.tools@gmail.com>
Fri, 1 Oct 2021 16:02:54 +0000 (09:02 -0700)
commitb47fd7f49ee743b8fe60781fd411e4d39b063e20
tree1c6dcf73bd021aba37edf9904368d9525aabafcf
parent984400f04ebece71be188278c3224f49d3a381a9
Update c-c++-common/tsan/atomic_stack.c

Print out from __tsan_atomic32_fetch_add was removed by

commit da7a5c09c86c3f639c63ce8843d6f21c915ae1c6
Author: Dmitry Vyukov <dvyukov@google.com>
Date:   Wed Jul 28 16:57:39 2021 +0200

    tsan: don't print __tsan_atomic* functions in report stacks

    Currently __tsan_atomic* functions do FuncEntry/Exit using caller PC
    and then use current PC (pointing to __tsan_atomic* itself) during
    memory access handling. As the result the top function in reports
    involving atomics is __tsan_atomic* and the next frame points to user code.

    Remove FuncEntry/Exit in atomic functions and use caller PC
    during memory access handling. This removes __tsan_atomic*
    from the top of report stacks, so that they point right to user code.

    The motivation for this is performance.
    Some atomic operations are very hot (mostly loads),
    so removing FuncEntry/Exit is beneficial.
    This also reduces thread trace consumption (1 event instead of 3).

    __tsan_atomic* at the top of the stack is not necessary
    and does not add any new information. We already say
    "atomic write of size 4", "__tsan_atomic32_store" does not add
    anything new.

    It also makes reports consistent between atomic and non-atomic
    accesses. For normal accesses we say "previous write" and point
    to user code; for atomics we say "previous atomic write" and now
    also point to user code.

    Reviewed By: vitalybuka

    Differential Revision: https://reviews.llvm.org/D106966

* c-c++-common/tsan/atomic_stack.c: Don't expect print out from
__tsan_atomic32_fetch_add.
gcc/testsuite/c-c++-common/tsan/atomic_stack.c