RE: [patch] clean up pdp11.md a bit

2012-02-21 Thread Paul_Koning
Paul; GCC Patches Subject: [patch] clean up pdp11.md a bit Hello, Just a few cleanups for things I noticed last weekend. * Constraints on define_expand are never used, remove them (most other ports do not have constraints on define_expands either) * Some patterns have no name, makes debug

RE: [patch] clean up pdp11.md a bit

2012-02-16 Thread Paul_Koning
#x27;t all clean yet. So that goes onto the list of things to work on, too. paul -Original Message- From: Steven Bosscher [mailto:stevenb@gmail.com] Sent: Tuesday, February 14, 2012 5:09 PM To: Koning, Paul; GCC Patches Subject: [patch] clean up pdp11.md a bit Hello, Just a fe

Re: [patch] clean up pdp11.md a bit

2012-02-14 Thread Richard Henderson
On 02/14/2012 03:41 PM, Steven Bosscher wrote: > Not sure why this was changed. What I'd like to do, is remove the > divhi3 and movhi3 define_expand and define_insn, and just keep > divmodhi4. You point to s390. I suppose you mean the divmoddi4 > expander there? Yes. Naturally the TImode stuff be

Re: [patch] clean up pdp11.md a bit

2012-02-14 Thread Steven Bosscher
On Tue, Feb 14, 2012 at 11:51 PM, Richard Henderson wrote: > On 02/14/2012 02:08 PM, Steven Bosscher wrote: >> OK for trunk? > > This can wait for stage1. > >> +;; On PDP-11, DIV always produces a quotient and a remainder.  But CSE >> +;; cannot optimize the divmods away because the SET_DESTs are

Re: [patch] clean up pdp11.md a bit

2012-02-14 Thread Richard Henderson
On 02/14/2012 02:08 PM, Steven Bosscher wrote: > OK for trunk? This can wait for stage1. > +;; On PDP-11, DIV always produces a quotient and a remainder. But CSE > +;; cannot optimize the divmods away because the SET_DESTs are SUBREGs. > +; > ;(define_expand "divmodhi4" > ; [(parallel [(set (

[patch] clean up pdp11.md a bit

2012-02-14 Thread Steven Bosscher
Hello, Just a few cleanups for things I noticed last weekend. * Constraints on define_expand are never used, remove them (most other ports do not have constraints on define_expands either) * Some patterns have no name, makes debugging a bit harder * The divmodhi4 expander has been commented out s