Re: [PATCH] arm: Fix -mpure-code support/-mslow-flash-data for armv8-m.base [PR94538]

2020-08-27 Thread Christophe Lyon via Gcc-patches
On Thu, 27 Aug 2020 at 11:02, Ramana Radhakrishnan wrote: > > On Mon, Aug 24, 2020 at 4:35 PM Christophe Lyon > wrote: > > > > On Mon, 24 Aug 2020 at 11:09, Christophe Lyon > > wrote: > > > > > > On Sat, 22 Aug 2020 at 00:44, Ramana Radhakrishnan > > > wrote: > > > > > > > > On Wed, Aug 19,

Re: [PATCH] arm: Fix -mpure-code support/-mslow-flash-data for armv8-m.base [PR94538]

2020-08-27 Thread Ramana Radhakrishnan via Gcc-patches
On Mon, Aug 24, 2020 at 4:35 PM Christophe Lyon wrote: > > On Mon, 24 Aug 2020 at 11:09, Christophe Lyon > wrote: > > > > On Sat, 22 Aug 2020 at 00:44, Ramana Radhakrishnan > > wrote: > > > > > > On Wed, Aug 19, 2020 at 10:32 AM Christophe Lyon via Gcc-patches > > > wrote: > > > > > > > >

Re: [PATCH] arm: Fix -mpure-code support/-mslow-flash-data for armv8-m.base [PR94538]

2020-08-24 Thread Christophe Lyon via Gcc-patches
On Mon, 24 Aug 2020 at 11:09, Christophe Lyon wrote: > > On Sat, 22 Aug 2020 at 00:44, Ramana Radhakrishnan > wrote: > > > > On Wed, Aug 19, 2020 at 10:32 AM Christophe Lyon via Gcc-patches > > wrote: > > > > > > armv8-m.base (cortex-m23) has the movt instruction, so we need to > > > disable

Re: [PATCH] arm: Fix -mpure-code support/-mslow-flash-data for armv8-m.base [PR94538]

2020-08-24 Thread Christophe Lyon via Gcc-patches
On Sat, 22 Aug 2020 at 00:44, Ramana Radhakrishnan wrote: > > On Wed, Aug 19, 2020 at 10:32 AM Christophe Lyon via Gcc-patches > wrote: > > > > armv8-m.base (cortex-m23) has the movt instruction, so we need to > > disable the define_split to generate a constant in this case, > > otherwise we get

Re: [PATCH] arm: Fix -mpure-code support/-mslow-flash-data for armv8-m.base [PR94538]

2020-08-21 Thread Ramana Radhakrishnan via Gcc-patches
On Wed, Aug 19, 2020 at 10:32 AM Christophe Lyon via Gcc-patches wrote: > > armv8-m.base (cortex-m23) has the movt instruction, so we need to > disable the define_split to generate a constant in this case, > otherwise we get incorrect insn constraints as described in PR94538. > > We also need to