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

--- Comment #3 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jakub Jelinek <ja...@gcc.gnu.org>:

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

commit r14-8681-gceb242f5302027c44a7dca86c344863004b6fec4
Author: Jakub Jelinek <ja...@redhat.com>
Date:   Thu Feb 1 09:16:57 2024 +0100

    gimple-low: Remove .ASAN_MARK calls on TREE_STATIC variables [PR113531]

    Since the r14-1500-g4d935f52b0d5c0 commit we promote an initializer_list
    backing array to static storage where appropriate, but this happens after
    we decided to add it to asan_poisoned_variables.  As a result we add
    unpoison/poison for it to the gimple.  But then sanopt removes the
unpoison.
    So the second time we call the function and want to load from the array
asan
    still considers it poisoned.

    The following patch fixes it by removing the .ASAN_MARK internal calls
    during gimple lowering if they refer to TREE_STATIC vars.

    2024-02-01  Jakub Jelinek  <ja...@redhat.com>
                Jason Merrill  <ja...@redhat.com>

            PR c++/113531
            * gimple-low.cc (lower_stmt): Remove .ASAN_MARK calls
            on variables which were promoted to TREE_STATIC.

            * g++.dg/asan/initlist1.C: New test.

    Co-authored-by: Jason Merrill <ja...@redhat.com>

Reply via email to