Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-06 Thread Jeff Law
On 7/3/24 2:32 PM, Georg-Johann Lay wrote: Am 03.07.24 um 21:39 schrieb Jeff Law: On 7/3/24 1:26 PM, Georg-Johann Lay wrote: Am 02.07.24 um 15:48 schrieb Richard Biener: On Tue, Jul 2, 2024 at 3:43 PM Georg-Johann Lay wrote: Hi Jeff, This is a patch to get correct code out of 64-b

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-04 Thread Georg-Johann Lay
Am 04.07.24 um 13:25 schrieb Richard Biener: On Thu, Jul 4, 2024 at 1:08 PM Georg-Johann Lay wrote: Am 04.07.24 um 11:49 schrieb Richard Biener: On Thu, Jul 4, 2024 at 11:24 AM Richard Biener wrote: On Wed, Jul 3, 2024 at 9:26 PM Georg-Johann Lay wrote: Am 02.07.24 um 15:48 schrieb Richard

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-04 Thread Richard Biener
On Thu, Jul 4, 2024 at 1:08 PM Georg-Johann Lay wrote: > > > > Am 04.07.24 um 11:49 schrieb Richard Biener: > > On Thu, Jul 4, 2024 at 11:24 AM Richard Biener > > wrote: > >> > >> On Wed, Jul 3, 2024 at 9:26 PM Georg-Johann Lay wrote: > >>> > >>> > >>> > >>> Am 02.07.24 um 15:48 schrieb Richard

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-04 Thread Georg-Johann Lay
Am 04.07.24 um 11:49 schrieb Richard Biener: On Thu, Jul 4, 2024 at 11:24 AM Richard Biener wrote: On Wed, Jul 3, 2024 at 9:26 PM Georg-Johann Lay wrote: Am 02.07.24 um 15:48 schrieb Richard Biener: On Tue, Jul 2, 2024 at 3:43 PM Georg-Johann Lay wrote: Hi Jeff, This is a patch to

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-04 Thread Richard Biener
On Thu, Jul 4, 2024 at 11:24 AM Richard Biener wrote: > > On Wed, Jul 3, 2024 at 9:26 PM Georg-Johann Lay wrote: > > > > > > > > Am 02.07.24 um 15:48 schrieb Richard Biener: > > > On Tue, Jul 2, 2024 at 3:43 PM Georg-Johann Lay wrote: > > >> > > >> Hi Jeff, > > >> > > >> This is a patch to get c

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-04 Thread Richard Biener
On Wed, Jul 3, 2024 at 9:26 PM Georg-Johann Lay wrote: > > > > Am 02.07.24 um 15:48 schrieb Richard Biener: > > On Tue, Jul 2, 2024 at 3:43 PM Georg-Johann Lay wrote: > >> > >> Hi Jeff, > >> > >> This is a patch to get correct code out of 64-bit > >> loads from address-space __memx. > >> > >> The

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-03 Thread Georg-Johann Lay
Am 03.07.24 um 21:39 schrieb Jeff Law: On 7/3/24 1:26 PM, Georg-Johann Lay wrote: Am 02.07.24 um 15:48 schrieb Richard Biener: On Tue, Jul 2, 2024 at 3:43 PM Georg-Johann Lay wrote: Hi Jeff, This is a patch to get correct code out of 64-bit loads from address-space __memx. The AVR a

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-03 Thread Jeff Law
On 7/3/24 1:26 PM, Georg-Johann Lay wrote: Am 02.07.24 um 15:48 schrieb Richard Biener: On Tue, Jul 2, 2024 at 3:43 PM Georg-Johann Lay wrote: Hi Jeff, This is a patch to get correct code out of 64-bit loads from address-space __memx. The AVR address-spaces may require that move insns

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-03 Thread Georg-Johann Lay
Am 02.07.24 um 15:48 schrieb Richard Biener: On Tue, Jul 2, 2024 at 3:43 PM Georg-Johann Lay wrote: Hi Jeff, This is a patch to get correct code out of 64-bit loads from address-space __memx. The AVR address-spaces may require that move insns issue calls to library support functions, a fa

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-02 Thread Jeff Law
On 7/2/24 7:57 AM, Richard Biener wrote: On Tue, Jul 2, 2024 at 3:52 PM Georg-Johann Lay wrote: ...just noticed that disabling -ftree-ter would also fix PR53049 where it moves a volatile asm across a library call. As explained in the PR this only avoids the issue by accident (and with the

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-02 Thread Richard Biener
On Tue, Jul 2, 2024 at 3:52 PM Georg-Johann Lay wrote: > > ...just noticed that disabling -ftree-ter would also fix PR53049 > where it moves a volatile asm across a library call. As explained in the PR this only avoids the issue by accident (and with the other comment I'm trying to make sure this

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-02 Thread Georg-Johann Lay
...just noticed that disabling -ftree-ter would also fix PR53049 where it moves a volatile asm across a library call. Am 02.07.24 um 15:42 schrieb Georg-Johann Lay: Hi Jeff, This is a patch to get correct code out of 64-bit loads from address-space __memx. The AVR address-spaces may require t

Re: [patch,avr] PR87376: Disable -ftree-ter

2024-07-02 Thread Richard Biener
On Tue, Jul 2, 2024 at 3:43 PM Georg-Johann Lay wrote: > > Hi Jeff, > > This is a patch to get correct code out of 64-bit > loads from address-space __memx. > > The AVR address-spaces may require that move insns issue > calls to library support functions, a fact that -ftree-ter > doesn't account f

[patch,avr] PR87376: Disable -ftree-ter

2024-07-02 Thread Georg-Johann Lay
Hi Jeff, This is a patch to get correct code out of 64-bit loads from address-space __memx. The AVR address-spaces may require that move insns issue calls to library support functions, a fact that -ftree-ter doesn't account for. tree-ssa-ter.cc then replaces an expression across such a library