[Bug go/101407] non-determinism in -fdump-go-spec

cvs-commit at gcc dot gnu.org gcc-bugzilla@gcc.gnu.org
Sun Jul 18 23:29:13 GMT 2021


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

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Jakub Jelinek
<jakub@gcc.gnu.org>:

https://gcc.gnu.org/g:31b76a815fc177dd579adc03b671ba9a8846ae6c

commit r11-8771-g31b76a815fc177dd579adc03b671ba9a8846ae6c
Author: Jakub Jelinek <jakub@redhat.com>
Date:   Wed Jul 14 10:22:50 2021 +0200

    godump: Fix -fdump-go-spec= reproduceability issue [PR101407]

    pot_dummy_types is a hash_set from whose traversal the code prints some
type
    lines.  hash_set normally uses default_hash_traits which for pointer types
    (the hash set hashes const char *) uses pointer_hash which hashes the
    addresses of the pointers except of the least significant 3 bits.
    With address space randomization, that results in non-determinism in the
    -fdump-go-specs= generated file, each invocation can have different order
of
    the lines emitted from pot_dummy_types traversal.

    This patch fixes it by hashing the string contents instead to make the
    hashes reproduceable.

    2021-07-14  Jakub Jelinek  <jakub@redhat.com>

            PR go/101407
            * godump.c (godump_str_hash): New type.
            (godump_container::pot_dummy_types): Use string_hash instead of
            ptr_hash in the hash_set.

    (cherry picked from commit 3be762c2ed79e36b9c8faaea2be04725c967a34e)


More information about the Gcc-bugs mailing list