[LLVMdev] Linux/ppc backend

[Nicolas Geoffray]

Hi Chris,

Chris Lattner wrote:
> Sorry for the delay,
>
>   
No problem. Plus the reviewing may have taken some time. So thx a lot
for committing.  I talked to Jim who said
he wanted to commit his changes before mine -- I hope everything's Ok.
> I applied the patches after some cleanups.  Please keep code within 80 
> columns, please don't use nested ?: expressions without parens, and please 
> be careful about indentation.  
Alright, sorry I forgot that. I'll be careful next time. Btw, if i have 
some bug fixes
to do, what should I do? Ask for a write access? Or just do an RFC and 
waiting for
someone to commit?
> Please verify that mainline CVS has 
> everything you think it should.
>   
I just verified and launched some compilations. Everything seems OK.
> The one hunk I didn't apply was this one:
>
> @@ -1392,12 +1418,13 @@
>       case MVT::f32:
>       case MVT::f64:
> -      if (isVarArg && isPPC64) {
> +      if (isVarArg || isPPC64) {
>           // Float varargs need to be promoted to double.
>           if (Arg.getValueType() == MVT::f32)
>             Arg = DAG.getNode(ISD::FP_EXTEND, MVT::f64, Arg);
>
> This changes the darwin/ppc ABI.  It's not clear to me that this was 
> intended, so I just left it out.
>   
Ah yes, I changed this and forgot to mention it (it deserves a different 
commit - it's, as you
noticed not, directly related to the linux/ppc abi).
I think it should be a "or" instead of an "and" :
floats are promoted to double in a vararg call wether you're on ppc64 or 
ppc32. Maybe i'm missing
something, anyone can confirm?
> Thanks a lot for these patches.  How well does linux/ppc work now?
>
>   
Thanks to you for reviewing and committing.
I would like to say it works great :) but I know there is one bug
I have to found because I sometimes seem to get an infinite loop.

If others have the opportunity to test it, please give us some feedback.

Cheers,
Nicolas