[llvm-commits] [RFC] Patch 1/3 for the native code compiler of Erlang

[Yiannis Tsiouris]

On 05/14/2012 01:31 AM, Yiannis Tsiouris wrote:
> On 05/13/2012 07:51 PM, Jakob Stoklund Olesen wrote:
>> On May 13, 2012, at 8:32 AM, Yiannis Tsiouris wrote:
>>> Ping.
>>>
>>> Is there anything wrong with this patch?
>> The patch doesn't apply cleanly, possibly because of the missing newline at the end of your test case.
>>
>> Here you check for a NULL Function:
>>
>>      const Function *F = MF->getFunction();                                      
>>      ghcCall = (F ? F->getCallingConv() == CallingConv::GHC : false);            
>> +    hipeCall = (F ? F->getCallingConv() == CallingConv::HiPE : false);          
>>
>> Here you don't:
>>
>> +    if (MF.getFunction()->getCallingConv() == CallingConv::HiPE)                
>> +      return &X86::GR32RegClass;                                                
>>
>> /jakob
> I attatch an updated patch for the Calling Convention.
Ping.
Is there something wrong with the new CC patch?

With best regards,
Yiannis

-- 
Yiannis Tsiouris
Ph.D. student,
Software Engineering Laboratory,
National Technical University of Athens
WWW: http://www.softlab.ntua.gr/~gtsiour

-------------- next part --------------
A non-text attachment was scrubbed...
Name: hipe-calling-conv.patch
Type: text/x-diff
Size: 12282 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120531/a2d598d0/attachment.patch>