]> git.ipfire.org Git - thirdparty/gcc.git/commit
bitint: Avoid rewriting large/huge _BitInt vars into SSA after bitint lowering [PR114278]
authorJakub Jelinek <jakub@redhat.com>
Mon, 11 Mar 2024 10:00:54 +0000 (11:00 +0100)
committerJakub Jelinek <jakub@redhat.com>
Mon, 11 Mar 2024 10:00:54 +0000 (11:00 +0100)
commitdbe5ccda4dbbd064c703cd3ab2a58ea40f08dd1a
treeab899d28cf16f8907907fb759c2cd9dc4501d609
parent0c4df2c3c38ca15c123e9a801b617e63256c83a3
bitint: Avoid rewriting large/huge _BitInt vars into SSA after bitint lowering [PR114278]

The following testcase ICEs, because update-address-taken subpass of
fre5 rewrites
  _BitInt(128) b;
  vector(16) unsigned char _3;

  <bb 2> [local count: 1073741824]:
  _3 = MEM <vector(16) unsigned char> [(char * {ref-all})p_2(D)];
  MEM <vector(16) unsigned char> [(char * {ref-all})&b] = _3;
  b ={v} {CLOBBER(eos)};
to
  _BitInt(128) b;
  vector(16) unsigned char _3;

  <bb 2> [local count: 1073741824]:
  _3 = MEM <vector(16) unsigned char> [(char * {ref-all})p_2(D)];
  b_5 = VIEW_CONVERT_EXPR<_BitInt(128)>(_3);
but we can't have large/huge _BitInt vars in SSA form after the bitint
lowering except for function arguments loaded from memory, as expansion
isn't able to deal with those, it relies on bitint lowering to lower
those operations.
The following patch fixes that by setting DECL_NOT_GIMPLE_REG_P for
large/huge _BitInt vars after bitint lowering, such that we don't
rewrite them into SSA form.

2024-03-11  Jakub Jelinek  <jakub@redhat.com>

PR tree-optimization/114278
* tree-ssa.cc (maybe_optimize_var): If large/huge _BitInt vars are no
longer addressable, set DECL_NOT_GIMPLE_REG_P on them.

* gcc.dg/bitint-99.c: New test.
gcc/testsuite/gcc.dg/bitint-99.c [new file with mode: 0644]
gcc/tree-ssa.cc