Re: [PATCH] Fix PR 91605

2019-09-03 Thread Jeff Law
On 9/3/19 12:46 PM, Bernd Edlinger wrote: > On 9/3/19 8:40 PM, Jeff Law wrote: >> On 9/1/19 4:36 AM, Bernd Edlinger wrote: >>> Hi, >>> >>> this fixes an oversight in r274986. >>> We need to avoid using movmisalign on DECL_P which are not in memory, >>> similar to the !mem_ref_refers_to_non_mem_p wh

Re: [PATCH] Fix PR 91605

2019-09-03 Thread Bernd Edlinger
On 9/3/19 8:40 PM, Jeff Law wrote: > On 9/1/19 4:36 AM, Bernd Edlinger wrote: >> Hi, >> >> this fixes an oversight in r274986. >> We need to avoid using movmisalign on DECL_P which are not in memory, >> similar to the !mem_ref_refers_to_non_mem_p which unfortunately can't >> handle DECL_P. >> >> >>

Re: [PATCH] Fix PR 91605

2019-09-03 Thread Jeff Law
On 9/1/19 4:36 AM, Bernd Edlinger wrote: > Hi, > > this fixes an oversight in r274986. > We need to avoid using movmisalign on DECL_P which are not in memory, > similar to the !mem_ref_refers_to_non_mem_p which unfortunately can't > handle DECL_P. > > > Bootstrapped and reg-tested on x86_64-pc-l

Re: [PATCH] Fix PR 91605

2019-09-02 Thread Richard Biener
On Mon, 2 Sep 2019, Bernd Edlinger wrote: > On 9/2/19 9:50 AM, Richard Biener wrote: > > On Sun, 1 Sep 2019, Bernd Edlinger wrote: > > > >> Hi, > >> > >> this fixes an oversight in r274986. > >> We need to avoid using movmisalign on DECL_P which are not in memory, > >> similar to the !mem_ref_ref

Re: [PATCH] Fix PR 91605

2019-09-02 Thread Bernd Edlinger
On 9/2/19 9:50 AM, Richard Biener wrote: > On Sun, 1 Sep 2019, Bernd Edlinger wrote: > >> Hi, >> >> this fixes an oversight in r274986. >> We need to avoid using movmisalign on DECL_P which are not in memory, >> similar to the !mem_ref_refers_to_non_mem_p which unfortunately can't >> handle DECL_P

Re: [PATCH] Fix PR 91605

2019-09-02 Thread Richard Biener
On Sun, 1 Sep 2019, Bernd Edlinger wrote: > Hi, > > this fixes an oversight in r274986. > We need to avoid using movmisalign on DECL_P which are not in memory, > similar to the !mem_ref_refers_to_non_mem_p which unfortunately can't > handle DECL_P. > But - && (DECL_P (to) || !mem_ref_refe