[llvm-commits] PATCH: ARM_TARGET1 relocation for C++ global constructors

[Kevin Enderby]

I've looked at the patch but I don't know this area.  I think it may have to wait for Jim to say if it is correct for all platforms.

Kev

On Jan 4, 2012, at 11:12 PM, Evan Cheng wrote:

> Has this been reviewed?
> 
> I'm not sure if this patch is correct for all platforms, e.g. iOS. Jim, Kevin, and Owen, do you guys know?
> 
> Evan
> 
> On Dec 21, 2011, at 7:12 AM, James Molloy wrote:
> 
>> Hi,
>>  
>> Attached is a patch to cause the relocations for C++ global/static constructor thunks to be outputted properly on ARM. The ABI states that these thunks should have an R_ARM_TARGET1 relocation type applied to them for portability reasons. Currently they’re emitted as normal global variables so get an R_ARM_ABS32.
>>  
>> I’ve implemented this by creating a new overridable hook, “EmitXXStructor()”, in the AsmPrinter. For all targets this defaults to EmitGlobalConstant as usual, but ARM overrides it to emit a constant with this specific relocation type.
>>  
>> Am I OK to commit?
>>  
>> Cheers,
>>  
>> James
>> <reloc_arm_target1.patch>_______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120105/221e49a9/attachment.html>