[Bug c/100600] New: ICE: in verify_hash_value, at fold-const.c:3929

[Bug c/100600] New: ICE: in verify_hash_value, at fold-const.c:3929

[cnsun at uwaterloo dot ca]

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

            Bug ID: 100600
           Summary: ICE: in verify_hash_value, at fold-const.c:3929
           Product: gcc
           Version: tree-ssa
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
          Assignee: unassigned at gcc dot gnu.org
          Reporter: cnsun at uwaterloo dot ca
  Target Milestone: ---

$ gcc-trunk -v
Using built-in specs.
COLLECT_GCC=gcc-trunk
COLLECT_LTO_WRAPPER=/scratch/software/gcc-trunk/libexec/gcc/x86_64-pc-linux-gnu/12.0.0/lto-wrapper
Target: x86_64-pc-linux-gnu
Configured with: /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/configure
--enable-languages=c,c++,lto --enable-checking-yes --enable-multiarch
--prefix=/scratch/software/gcc-trunk --disable-bootstrap
Thread model: posix
Supported LTO compression algorithms: zlib
gcc version 12.0.0 20210514 (experimental) [master revision
:e5c3c8afa:f3b1516d9dfd969d7cc1ca6f26dec13478a1c458] (GCC)

$ cat mutant.c
a, b, c;
long d(long, long e, long f, long g) {
  long h, i;
  for (; h < e; h++) {
    i = f;
    for (; i < g; i++)
      c = b + a;
  }
  return h + i;
}
long j(long, long e, long, long g) {
  long h, i;
  for (; h < e; h++)
    for (; i < g; i++)
      c = b + a;
  return h + i;
}

$ gcc-trunk -O3 mutant.c
mutant.c:1:1: warning: data definition has no type or storage class
    1 | a, b, c;
      | ^
mutant.c:1:1: warning: type defaults to ‘int’ in declaration of ‘a’
[-Wimplicit-int]
mutant.c:1:4: warning: type defaults to ‘int’ in declaration of ‘b’
[-Wimplicit-int]
    1 | a, b, c;
      |    ^
mutant.c:1:7: warning: type defaults to ‘int’ in declaration of ‘c’
[-Wimplicit-int]
    1 | a, b, c;
      |       ^
during IPA pass: icf
mutant.c:17:1: internal compiler error: in verify_hash_value, at
fold-const.c:3929
   17 | }
      | ^
0x6b4fe9 operand_compare::verify_hash_value(tree_node const*, tree_node const*,
unsigned int, bool*)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/fold-const.c:3929
0x19e8d35 ipa_icf_gimple::func_checker::operand_equal_p(tree_node const*,
tree_node const*, unsigned int)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf-gimple.c:312
0x19e606b ipa_icf_gimple::func_checker::compare_operand(tree_node*, tree_node*,
ipa_icf_gimple::func_checker::operand_access_type)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf-gimple.c:397
0x19d642a ipa_icf::sem_function::compare_phi_node(basic_block_def*,
basic_block_def*)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf.c:1584
0x19dccf1 ipa_icf::sem_function::equals_private(ipa_icf::sem_item*)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf.c:925
0x19dcdf3 ipa_icf::sem_function::equals_private(ipa_icf::sem_item*)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf.c:836
0x19dcdf3 ipa_icf::sem_function::equals(ipa_icf::sem_item*,
hash_map<symtab_node*, ipa_icf::sem_item*,
simple_hashmap_traits<default_hash_traits<symtab_node*>, ipa_icf::sem_item*>
>&)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf.c:813
0x19d7457 ipa_icf::sem_item_optimizer::subdivide_classes_by_equality(bool)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf.c:2732
0x19e34ac ipa_icf::sem_item_optimizer::execute()
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf.c:2464
0x19e51aa ipa_icf_driver
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf.c:3600
0x19e51aa ipa_icf::pass_ipa_icf::execute(function*)
        /tmp/tmp.eZIsobWkq2-gcc-builder/gcc/gcc/ipa-icf.c:3647
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See <https://gcc.gnu.org/bugs/> for instructions.

[Bug ipa/100600] [11/12 Regression] ICE: in verify_hash_value, at fold-const.c:3929 since r11-5206-gd1081010a1addfcf

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2021-05-17
          Component|c                           |ipa
             Status|UNCONFIRMED                 |ASSIGNED
                 CC|                            |hubicka at gcc dot gnu.org,
                   |                            |marxin at gcc dot gnu.org
            Summary|ICE: in verify_hash_value,  |[11/12 Regression] ICE: in
                   |at fold-const.c:3929        |verify_hash_value, at
                   |                            |fold-const.c:3929 since
                   |                            |r11-5206-gd1081010a1addfcf
           Assignee|unassigned at gcc dot gnu.org      |marxin at gcc dot gnu.org

--- Comment #1 from Martin Liška <marxin at gcc dot gnu.org> ---
I'll take a look.

[Bug ipa/100600] [11/12 Regression] ICE: in verify_hash_value, at fold-const.c:3929 since r11-5206-gd1081010a1addfcf

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2
            Version|tree-ssa                    |11.1.0
   Target Milestone|---                         |11.2

[Bug ipa/100600] [11/12 Regression] ICE: in verify_hash_value, at fold-const.c:3929 since r11-5206-gd1081010a1addfcf

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|11.2                        |11.3

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
GCC 11.2 is being released, retargeting bugs to GCC 11.3

[Bug ipa/100600] [11/12 Regression] ICE: in verify_hash_value, at fold-const.c:3929 since r11-5206-gd1081010a1addfcf

[cvs-commit at gcc dot gnu.org]

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

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Liska <marxin@gcc.gnu.org>:

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

commit r12-2920-gb0de3ad2620d665ab481c2f24204f77f386ff6b2
Author: Martin Liska <mliska@suse.cz>
Date:   Fri Aug 13 12:35:47 2021 +0200

    ipa: ICF should check SSA_NAME_IS_DEFAULT_DEF

            PR ipa/100600

    gcc/ChangeLog:

            * ipa-icf-gimple.c (func_checker::compare_ssa_name): Do not
              consider equal SSA_NAMEs when one is a param.

    gcc/testsuite/ChangeLog:

            * gcc.dg/ipa/pr100600.c: New test.

[Bug ipa/100600] [11 Regression] ICE: in verify_hash_value, at fold-const.c:3929 since r11-5206-gd1081010a1addfcf

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |12.0
      Known to fail|                            |11.2.0
            Summary|[11/12 Regression] ICE: in  |[11 Regression] ICE: in
                   |verify_hash_value, at       |verify_hash_value, at
                   |fold-const.c:3929 since     |fold-const.c:3929 since
                   |r11-5206-gd1081010a1addfcf  |r11-5206-gd1081010a1addfcf

--- Comment #4 from Martin Liška <marxin at gcc dot gnu.org> ---
Fixed on master so far.

[Bug ipa/100600] [11 Regression] ICE: in verify_hash_value, at fold-const.c:3929 since r11-5206-gd1081010a1addfcf

[cvs-commit at gcc dot gnu.org]

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

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Martin Liska
<marxin@gcc.gnu.org>:

https://gcc.gnu.org/g:64336a85c96ef91f863bd8e86aa5002c3c69f386

commit r11-8870-g64336a85c96ef91f863bd8e86aa5002c3c69f386
Author: Martin Liska <mliska@suse.cz>
Date:   Fri Aug 13 12:35:47 2021 +0200

    ipa: ICF should check SSA_NAME_IS_DEFAULT_DEF

            PR ipa/100600

    gcc/ChangeLog:

            * ipa-icf-gimple.c (func_checker::compare_ssa_name): Do not
              consider equal SSA_NAMEs when one is a param.

    gcc/testsuite/ChangeLog:

            * gcc.dg/ipa/pr100600.c: New test.

[Bug ipa/100600] [11 Regression] ICE: in verify_hash_value, at fold-const.c:3929 since r11-5206-gd1081010a1addfcf

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Known to work|                            |11.2.1
      Known to fail|11.2.0                      |
             Status|ASSIGNED                    |RESOLVED

--- Comment #6 from Martin Liška <marxin at gcc dot gnu.org> ---
Fixed now.