Coldfire Diffs

Coldfire Diffs

[Joel Sherrill]

Below my signature are some diffs from the win32 cdk which may or may not
be included in current egcs snapshots.  There are about 200 lines of diffs
to about 5 files.  I am pretty sure some are already included in egcs
snapshots.  

David Fiddes <D.J.Fiddes@hw.ac.uk> uses these to build a Coldfire toolset
he has been graciously distributing to those interested in the Coldfire. 
At my request, he sent these to me so I could try to helkp get them into
the egcs tree.

Could someone knowledgeable about the Coldfire please check these out?

Thanks.

--joel
Joel Sherrill                    Director of Research & Development
joel@OARcorp.com                 On-Line Applications Research
Ask me about RTEMS: a free RTOS  Huntsville AL 35805
   Support Available             (205) 722-9985

begin 604 coldfire.dif
M9&EF9B`M<B!C9&LO8V]N9FEG+6UL+FEN(&-D:RUO<FEG+V-O;F9I9RUM;"YI
M;@HR-#DL,C4Y9#(T.`H\("`@("`@("`@:68@6R!X)&5N86)L95]M-3(P,"`]
M('AN;R!="CP@("`@("`@("!T:&5N"CP@("`@("`@("`@(&]L9%]M=6QT:61I
M<G,](B1[;75L=&ED:7)S?2(*/"`@("`@("`@("`@;75L=&ED:7)S/2(B"CP@
M("`@("`@("`@(&9O<B!X(&EN("1[;VQD7VUU;'1I9&ER<WT[(&1O"CP@("`@
M("`@("`@("`@8V%S92`B)'@B(&EN"CP@("`@("`@("`@("`@("`J;34R,#`J
M("D@.B`[.PH\("`@("`@("`@("`@("`@*BD@;75L=&ED:7)S/2(D>VUU;'1I
M9&ER<WT@)'MX?2(@.SL*/"`@("`@("`@("`@("!E<V%C"CP@("`@("`@("`@
M(&1O;F4*/"`@("`@("`@(&9I"F1I9F8@+7(@8V1K+V=C8R]C;VYF:6<O;38X
M:R]M-CAK+FUD(&-D:RUO<FEG+V=C8R]C;VYF:6<O;38X:R]M-CAK+FUD"C<U
M,F,W-3(*/"`H9&5F:6YE7VEN<VX@(F)T<W1?,2(*+2TM"CX@*&1E9FEN95]I
M;G-N("(B"C<V,&,W-C`*/"`H9&5F:6YE7VEN<VX@(F)T<W1?,B(*+2TM"CX@
M*&1E9FEN95]I;G-N("(B"C<W,F,W-S(*/"`H9&5F:6YE7VEN<VX@(F)T<W1?
M,R(*+2TM"CX@*&1E9FEN95]I;G-N("(B"C<X,F,W.#(*/"`H9&5F:6YE7VEN
M<VX@(F)T<W1?-"(*+2TM"CX@*&1E9FEN95]I;G-N("(B"C<Y-&,W.30*/"`H
M9&5F:6YE7VEN<VX@(F)T<W1?-2(*+2TM"CX@*&1E9FEN95]I;G-N("(B"C<Y
M.&,W.3@*/"`@("(H=6YS:6=N960I($E.5%9!3"`H;W!E<F%N9'-;,5TI(#P@
M."`F)B`A5$%21T547S4R,#`B"BTM+0H^("`@(BAU;G-I9VYE9"D@24Y45D%,
M("AO<&5R86YD<ULQ72D@/"`X(@HX,#5C.#`U"CP@*&1E9FEN95]I;G-N(")B
M='-T7S8B"BTM+0H^("AD969I;F5?:6YS;B`B(@HY.31D.3DS"CP@.SL[($-(
M04Y'140*.3DV+#DY-V,Y.34L.3DV"CP@("!;*'-E="`H;6%T8VA?;W!E<F%N
M9#I122`P(")G96YE<F%L7V]P97)A;F0B("(]9"PJ82QM(BD*/"`)*&UA=&-H
M7V]P97)A;F0Z44D@,2`B9V5N97)A;%]O<&5R86YD(B`B9&UI*F$L9&UI*F$L
M9"IA(BDI70HM+2T*/B`@(%LH<V5T("AM871C:%]O<&5R86YD.E%)(#`@(F=E
M;F5R86Q?;W!E<F%N9"(@(CUD/#XL9&TB*0H^(`DH;6%T8VA?;W!E<F%N9#I1
M22`Q(")G96YE<F%L7V]P97)A;F0B(")D;6DL9#P^(BDI70HQ,#`T+#$P,C!C
M,3`P,PH\("`@(B%405)'151?-3(P,"(*/"`@("(J"CP@>PH\("`@:68@*&]P
M97)A;F1S6S%=(#T](&-O;G-T,%]R='@*/"`@("`@("`O*B!C;'(@:6YS;G,@
M;VX@-C@P,#`@<F5A9"!B969O<F4@=W)I=&EN9RX*/"`@("`@("`@("!4:&ES
M(&ES;B=T('-O(&]N('1H92`V.#`Q,"P@8G5T('=E(&AA=F4@;F\@5$%21T54
M7S8X,#$P+B`@*B\*/"`@("`@("`F)B`H*%1!4D=%5%\V.#`R,"!\?"!405)'
M151?-3(P,"D*/"`@("`@("`@("`@?'P@(2A'151?0T]$12`H;W!E<F%N9'-;
M,%TI(#T]($U%32`F)B!-14U?5D],051)3$5?4"`H;W!E<F%N9'-;,%TI*2DI
M"CP@("`@(')E='5R;B!<(F-L<B4N8B`E,%PB.PH\("`@<F5T=7)N(%PB;6]V
M924N8B`E,2PE,%PB.PH\('TB*0H\(`H\(#L[.R!#2$%.1T5$"CP@*&1E9FEN
M95]I;G-N(")M;W9S=')I8W1Q:5\U,C`P(@H\("`@6RAS970@*'-T<FEC=%]L
M;W=?<&%R="`H;6%T8VA?;W!E<F%N9#I122`P(")G96YE<F%L7V]P97)A;F0B
M("(K9"QM(BDI"CP@"2AM871C:%]O<&5R86YD.E%)(#$@(F=E;F5R86Q?;W!E
M<F%N9"(@(F1M;BQD(BDI72`*/"`@(")405)'151?-3(P,"(*+2TM"CX@("`B
M(@HQ,C$R+#$R-3-C,3$Y-0H\("`@(B%405)'151?-C@X.#$@)B8@(51!4D=%
M5%\U,C`P(@H\("`@(BH*/"!["CP@("!I9B`H1E!?4D5'7U`@*&]P97)A;F1S
M6S!=*2D*/"`@("`@>PH\("`@("`@(&EF("A&4%]214=?4"`H;W!E<F%N9'-;
M,5TI*0H\(`ER971U<FX@7")F;6]V924N>"`E,2PE,%PB.PH\("`@("`@(&EF
M("A214=?4"`H;W!E<F%N9'-;,5TI*0H\(`E["CP@"2`@<G1X('AO<&5R86YD
M<ULR73L*/"`)("!X;W!E<F%N9'-;,5T@/2!G96Y?<G1X("A214<L(%-);6]D
M92P@4D5'3D\@*&]P97)A;F1S6S%=*2`K(#(I.PH\(`D@(&]U='!U=%]A<VU?
M:6YS;B`H7")M;W9E)2YL("4Q+"4M7"(L('AO<&5R86YD<RD["CP@"2`@>&]P
M97)A;F1S6S%=(#T@9V5N7W)T>"`H4D5'+"!326UO9&4L(%)%1TY/("AO<&5R
M86YD<ULQ72D@*R`Q*3L*/"`)("!O=71P=71?87-M7VEN<VX@*%PB;6]V924N
M;"`E,2PE+5PB+"!X;W!E<F%N9',I.PH\(`D@(&]U='!U=%]A<VU?:6YS;B`H
M7")M;W9E)2YL("4Q+"4M7"(L(&]P97)A;F1S*3L*/"`)("!R971U<FX@7")F
M;6]V924N>"`E*RPE,%PB.PH\(`E]"CP@("`@("`@:68@*$=%5%]#3T1%("AO
M<&5R86YD<ULQ72D@/3T@0T].4U1?1$]50DQ%*0H\("`@("`@("`@<F5T=7)N
M(%PB9FUO=F4E+G@@)3$L)3!<(CL*/"`@("`@("!R971U<FX@7")F;6]V924N
M>"`E9C$L)3!<(CL*/"`@("`@?0H\("`@:68@*$907U)%1U]0("AO<&5R86YD
M<ULQ72DI"CP@("`@('L*/"`@("`@("!I9B`H4D5'7U`@*&]P97)A;F1S6S!=
M*2D*/"`@("`@("`@('L*/"`@("`@("`@("`@;W5T<'5T7V%S;5]I;G-N("A<
M(F9M;W9E)2YX("5F,2PE+5P[;6]V924N;"`E*RPE,%PB+"!O<&5R86YD<RD[
M"CP@("`@("`@("`@(&]P97)A;F1S6S!=(#T@9V5N7W)T>"`H4D5'+"!326UO
M9&4L(%)%1TY/("AO<&5R86YD<ULP72D@*R`Q*3L*/"`@("`@("`@("`@;W5T
M<'5T7V%S;5]I;G-N("A<(FUO=F4E+FP@)2LL)3!<(BP@;W!E<F%N9',I.PH\
M("`@("`@("`@("!O<&5R86YD<ULP72`](&=E;E]R='@@*%)%1RP@4TEM;V1E
M+"!214=.3R`H;W!E<F%N9'-;,%TI("L@,2D["CP@("`@("`@("`@(')E='5R
M;B!<(FUO=F4E+FP@)2LL)3!<(CL*/"`@("`@("`@('T*/"`@("`@("!E;'-E
M"CP@("`@("`@("!R971U<FX@7")F;6]V924N>"`E9C$L)3!<(CL*/"`@("`@
M?0H\("`@<F5T=7)N(&]U='!U=%]M;W9E7V1O=6)L92`H;W!E<F%N9',I.PH\
M('T*/"`B*0H\(#L[.R!#2$%.1T5$"CP@*&1E9FEN95]I;G-N(")M;W9F7S4R
M,#`B"CP@("!;*'-E="`H;6%T8VA?;W!E<F%N9#I81B`P(")N;VYI;6UE9&EA
M=&5?;W!E<F%N9"(@(CUR;2QR9BPF<F8\/B(I"CP@"2AM871C:%]O<&5R86YD
M.EA&(#$@(FYO;FEM;65D:6%T95]O<&5R86YD(B`B<F8L;2QR;V8\/B(I*5T*
M/"`@("(A5$%21T547S8X.#@Q("8F(%1!4D=%5%\U,C`P(@HM+2T*/B`@("(A
M(%1!4D=%5%\V.#@X,2(*,C(V."PR,C<S8S(R,3`*/"`@("`@("`@('T*/"`@
M("`@("!I9B`H5$%21T547S4R,#`@)B8@(41!5$%?4D5'7U`H;W!E<F%N9'-;
M,%TI*0H\("`@("`@("`@>PH\("`@("`@("`@("`@(&]P97)A;F1S6S%=(#T@
M9V5N7W)T>"`H4D5'+"!326UO9&4L(#`I.PH\("`@("`@("`@("`@(')E='5R
M;B!<(FUO=F4E+FP@)3(L)3%<.V%D9"4N;"`E,2PE,%PB.PH\("`@("`@("`@
M?0HM+2T*/B`)?0HR,C<X+#(R-SED,C(Q-`H\(`H\(`HV,3(R+#8Q-S=C-C`U
M-PH\("`@(B%405)'151?-3(P,"(*/"`@("(J"CP@(VEF9&5F($%335]21515
M4DY?0T%315]*54U0"CP@($%335]215154DY?0T%315]*54U0.PH\("-E;'-E
M"CP@(VEF9&5F(%-'4PH\("-I9F1E9B!!4TU?3U544%547T-!4T5?3$%"14P*
M/"`@(&EF("A405)'151?-3(P,"D@"CP@("`@(')E='5R;B!<(F5X="4N;"`E
M,%P[:FUP(#8H)25P8RPE,"YL*5PB.PH\("`@96QS90H\("`@("!R971U<FX@
M7")J;7`@-B@E)7!C+"4P+G<I7"(["CP@(V5L<V4*/"`@(&EF("A405)'151?
M-3(P,"D*/"`@("`@>PH\("-I9F1E9B!#4D13"CP@("`@("`@<F5T=7)N(%PB
M97AT)2YL("4P7#MJ;7`@,BAP8RPE,"YL*5PB.PH\("-E;'-E"CP@("`@("`@
M<F5T=7)N(%PB97AT;"`E,%P[:FUP(#(H)25P8RPE,"YL*5PB.PH\("-E;F1I
M9B`@+RH@96YD("%#4D13("HO"CP@("`@('T*/"`@(&5L<V4*/"`@("`@>PH\
M("-I9F1E9B!#4D13"CP@("`@("`@<F5T=7)N(%PB:FUP(#(H<&,L)3`N=RE<
M(CL*/"`C96QS90H\("`@("`@(')E='5R;B!<(FIM<"`R*"4E<&,L)3`N=RE<
M(CL*/"`C96YD:68@("\J(&5N9"`A0U)$4R`J+PH\("`@("!]"CP@(V5N9&EF
M"CP@(V5L<V4@+RH@;F]T(%-'4R`J+PH\("`@:68@*%1!4D=%5%\U,C`P*0H\
M("`@("!["CP@(VEF9&5F($U/5$]23TQ!"CP@("`@("`@<F5T=7)N(%PB97AT
M)2YL("4P7#MJ;7`@*#(L<&,L)3`N;"E<(CL*/"`C96QS90H\("`@("`@(')E
M='5R;B!<(F5X=&P@)3!<.VIM<"!P8T`H,BPE,#IL*5PB.PH\("-E;F1I9@H\
M("`@("!]"CP@("!E;'-E"CP@("`@('L*/"`C:69D968@34]43U)/3$$*/"`@
M("`@("!R971U<FX@7")J;7`@*#(L<&,L)3`N=RE<(CL*/"`C96QS90H\("`@
M("`@(')E='5R;B!<(FIM<"!P8T`H,BPE,#IW*5PB.PH\("-E;F1I9@H\("`@
M("!]"CP@(V5N9&EF"CP@(V5N9&EF"CP@(BD*/"`*/"`H9&5F:6YE7VEN<VX@
M(FIU;7!?=&]?=F%R:6%B;&5?-3(P,"(*/"`@(%LH<V5T("AP8RD*/"`)*'!L
M=7,Z4TD@*'!C*0H\(`D)("AS:6=N7V5X=&5N9#I322`H;6%T8VA?;W!E<F%N
M9#I(22`P(")R96=I<W1E<E]O<&5R86YD(B`B9"(I*2DI"CP@("`@*'5S92`H
M;&%B96Q?<F5F("AM871C:%]O<&5R86YD(#$@(B(@(B(I*2E="CP@("`B5$%2
M1T547S4R,#`B"BTM+0H^("`@(B(*3VYL>2!I;B!C9&LO9V-C+V-O;F9I9R]M
M-CAK.B!M:S8X:RYM9"YO<FEG"F1I9F8@+7(@8V1K+V=C8R]C;VYF:6<O;38X
M:R]T+6TV.&MB87)E(&-D:RUO<FEG+V=C8R]C;VYF:6<O;38X:R]T+6TV.&MB
M87)E"C(R+#(S8S(R"CP@(TU53%1)3$E"7T]05$E/3E,@/2!M-C@P,#`O;38X
M,#(P(&TV.#@X,2]M<V]F="UF;&]A=`H\($U53%1)3$E"7T]05$E/3E,@/2!M
M-C@P,#`O;38X,#(P+VTU,C`P(&TV.#@X,2]M<V]F="UF;&]A=`HM+2T*/B!-
M54Q424Q)0E]/4%1)3TY3(#T@;38X,#`P+VTV.#`R,"!M-C@X.#$O;7-O9G0M
M9FQO870*9&EF9B`M<B!C9&LO;&EB:6)E<G1Y+TUA:V5F:6QE+FEN(&-D:RUO
M<FEG+VQI8FEB97)T>2]-86ME9FEL92YI;@HV,BPV,V,V,@H\("-)3E-404Q,
M(#T@:6YS=&%L;"`M8PH\($E.4U1!3$P@/2!I;G-T86QL"BTM+0H^($E.4U1!
M3$P@/2!I;G-T86QL("UC"F1I9F8@+7(@8V1K+VQI8G-T9&,K*R]-86ME9FEL
M92YI;B!C9&LM;W)I9R]L:6)S=&1C*RLO36%K969I;&4N:6X*-#0L-#=C-#0L
M-#4*/"`C24]?1$E2("`@(#T@)"A-54Q424)524Q$5$]0*2XN+VQI8FEO"CP@
M(TQ)0DE"15)465]$25(@/2`D*$U53%1)0E5)3$143U`I+BXO;&EB:6)E<G1Y
M"CP@24]?1$E2("`@(#T@+BXO;&EB:6\*/"!,24))0D525%E?1$E2(#T@+BXO
M;&EB:6)E<G1Y"BTM+0H^($E/7T1)4B`@("`]("0H355,5$E"54E,1%1/4"DN
M+B]L:6)I;PH^($Q)0DE"15)465]$25(@/2`D*$U53%1)0E5)3$143U`I+BXO
*;&EB:6)E<G1Y"DQ)
`
end

Re: Coldfire Diffs

[Jeffrey A Law]

  In message <Pine.BSF.3.96.970919134654.21863C-100000@vespucci.advicom.net>Joel wrote:
  > 
  > Below my signature are some diffs from the win32 cdk which may or may not
  > be included in current egcs snapshots.  There are about 200 lines of diffs
  > to about 5 files.  I am pretty sure some are already included in egcs
  > snapshots.  
  > 
  > David Fiddes <D.J.Fiddes@hw.ac.uk> uses these to build a Coldfire toolset
  > he has been graciously distributing to those interested in the Coldfire. 
  > At my request, he sent these to me so I could try to helkp get them into
  > the egcs tree.
Unfortunately, the changes are in old-style diff format, which makes them
extremely difficult to read or apply.  So if my analysis is incorrect, it
may be due to not being able to 100% be sure exactly what patterns the
changes apply to.  It's also worth noting there are no testcases or explanation
of why the patches are needed, which makes it hard to analyze the changes.


What's the point behind the config-ml.in change?  Shouldn't that be handled
in the m68k target files?

Providing names to the various patterns may be useful, but isn't strictly
necessary as far as I can tell.  If you want to add names for them, please
prefix them with a "*" so as not to cause confusion with named patterns that
have a particular purpose.  Submit this as a separate patch since it is 
independent of coldfire support.

The one actual bugfix for the btst patterns is already in egcs.

I believe your change to the movqi pattern for the coldfire is incorrect
as we had it in our toolchain once and it was failing.  I think egcs has
a correct set of constraints for the movqi pattern on the coldfire.

I also believe your changes to movstrictqi have also been overtaken by
events and are no longer necessary.

Similarly for the XFmode move insn changes and jump table pattern changes.

We've also got the appropriate multilib changes for t-m68kbare.

So, basically I don't see anything in this patch that hasn't already been
addressed in the egcs sources.

So I guess the thing to do is for you to determine if egcs does the right
thing for the code which caused you to make the various changes to the m68k.md
file.  If egcs is still generating incorrect code we'll have to analyze it
further using whatever testcode is causing the compiler to generate incorrect
code.

Thanks,
jeff