On Fri, Jun 26, 2015 at 09:42:06AM +0200, Uros Bizjak wrote:
> Hello!
>
> > > > [ I see i386 also does PUT_CODE in a few more splitters, hrm. ]
> > >
> > > I think the splitters need to be fixed but I also think that
> > > shallow_copy_rtx should
> > > be enough, no? combine will not actually end
Hello!
> > > [ I see i386 also does PUT_CODE in a few more splitters, hrm. ]
> >
> > I think the splitters need to be fixed but I also think that
> > shallow_copy_rtx should
> > be enough, no? combine will not actually end up generating more references
> > to the original RTX so we won't end up w
On Thu, Jun 25, 2015 at 11:59:28AM +0200, Richard Biener wrote:
> On Thu, Jun 25, 2015 at 7:11 AM, Segher Boessenkool
> wrote:
> > On Wed, Jun 24, 2015 at 09:40:28PM -0600, Jeff Law wrote:
> >> On 06/24/2015 05:29 PM, Segher Boessenkool wrote:
> >> >A few define_split's in the i386 backend modify
On Thu, Jun 25, 2015 at 7:11 AM, Segher Boessenkool
wrote:
> On Wed, Jun 24, 2015 at 09:40:28PM -0600, Jeff Law wrote:
>> On 06/24/2015 05:29 PM, Segher Boessenkool wrote:
>> >A few define_split's in the i386 backend modify RTL in place. This does
>> >not work. This patch fixes all cases that do
On Wed, Jun 24, 2015 at 09:40:28PM -0600, Jeff Law wrote:
> On 06/24/2015 05:29 PM, Segher Boessenkool wrote:
> >A few define_split's in the i386 backend modify RTL in place. This does
> >not work. This patch fixes all cases that do PUT_MODE on existing RTL.
> > * config/i386/i386.md (variou
On 06/24/2015 09:40 PM, Jeff Law wrote:
On 06/24/2015 05:29 PM, Segher Boessenkool wrote:
A few define_split's in the i386 backend modify RTL in place. This does
not work. This patch fixes all cases that do PUT_MODE on existing RTL.
Bootstrapped and tested; no regressions. Is this okay for t
On 06/24/2015 05:29 PM, Segher Boessenkool wrote:
A few define_split's in the i386 backend modify RTL in place. This does
not work. This patch fixes all cases that do PUT_MODE on existing RTL.
Bootstrapped and tested; no regressions. Is this okay for trunk?
Hrm, this wants the testcase in th
A few define_split's in the i386 backend modify RTL in place. This does
not work. This patch fixes all cases that do PUT_MODE on existing RTL.
Bootstrapped and tested; no regressions. Is this okay for trunk?
Hrm, this wants the testcase in that PR added I suppose. Will send
it separately.
S
