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

2012-02-21 Thread Paul_Koning
By the way, the "compile" subset of the testsuite works for pdp11; it has some errors which still need cleanup but a large fraction works. paul -Original Message- From: Steven Bosscher [mailto:stevenb@gmail.com] Sent: Tuesday, February 14, 2012 5:09 PM To: Koning, Paul; GCC

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

2012-02-16 Thread Paul_Koning
These look fine. I'll defer to others on whether it should wait to Phase 1. I had tried to make divmod work but never figured out the reason why it did not. Thanks for answering that question. As for the subregs that Richard commented on -- I will gladly admit that this target isn't all clean

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 (