Excerpts from Martin Liška's message of Mai 11, 2022 10:17 am:
> On 5/9/22 14:03, Richard Biener wrote:
>> On Thu, May 5, 2022 at 4:30 PM Martin Liška <mli...@suse.cz> wrote:
>>>
>>> On 5/5/22 14:58, Iain Buclaw wrote:
>>>> This D front-end change doesn't look right to me, besides the slight
>>>
>>> Hello.
>>>
>>> Sorry, I've re-read the patch and fixed some places where the macro usage
>>> was wrong.
>>>
>>> Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
>>
>> The middle-end parts are OK. I'd say in files where ARRAY_SIZE is already
>> used it's OK to introduce more uses. Otherwise I defer to the more specific
>> maintainers if they like this or not.
>
> All right, CCing the following maintainers for other parts:
>
> - David for JIT and Analyzer
> - Tobias for Fortran part
> - Jason for C-family part
>
Hi Martin,
When running through contrib/config-list.mk, I noticed that this also
broke the build for the following obsolete targets:
tilegx-linux-gnu
tilegxbe-linux-gnu
tilepro-linux-gnu
---
gcc/config/tilepro/gen-mul-tables.cc: In function ‘void
find_sequences(ExpressionTree&, ExpressionTreeMap&)’:
gcc/config/tilepro/gen-mul-tables.cc:465:26: error: ‘ARRAY_SIZE’ was not
declared in this scope
465 | for (size_t f = 0; f < ARRAY_SIZE (ops); f++)
| ^~~~~~~~~~
gcc/config/tilepro/gen-mul-tables.cc: In function ‘void
create_insn_code_compression_table()’:
gcc/config/tilepro/gen-mul-tables.cc:567:26: error: ‘ARRAY_SIZE’ was not
declared in this scope
567 | for (size_t i = 0; i < ARRAY_SIZE (ops); i++)
| ^~~~~~~~~~
---
