On 9/3/19 9:05 AM, Jakub Jelinek wrote:
> On Tue, Sep 03, 2019 at 07:02:53AM +0000, Bernd Edlinger wrote:
>> 2019-09-03 Bernd Edlinger <bernd.edlin...@hotmail.de>
>>
>> PR middle-end/91603
>> PR middle-end/91612
>> PR middle-end/91613
>> * expr.c (expand_expr_real_1): decl_p_1): Refactor into...
>> (non_mem_decl_p): ...this.
>> (mem_ref_refers_to_non_mem_p): Handle DECL_P as well ase MEM_REF.
>> (expand_assignment): Call mem_ref_referes_to_non_mem_p
>> unconditionally as before.
>
> Not a review, just questioning the ChangeLog entry.
> What is the "decl_p_1): " in there? Also, the ChangeLog mentions many
> functions, but the patch in reality just modifies expand_expr_real_1
> and nothing else.
>
Ah, sorry, this is of course wrong, (I forgot to complete the sentence,
and later forgot to check it again)....
This is what I actually wanted to say:
2019-09-03 Bernd Edlinger <bernd.edlin...@hotmail.de>
PR middle-end/91603
PR middle-end/91612
PR middle-end/91613
* expr.c (expand_expr_real_1): Handle unaligned decl_rtl
and SSA_NAME referring to CONSTANT_P correctly.
testsuite:
2019-09-03 Bernd Edlinger <bernd.edlin...@hotmail.de>
PR middle-end/91603
* testsuite/gcc.target/arm/pr91603.c: New test.
Thanks
Bernd.
