[Review] Change TargetLoweringObjectFile ownership

Aditya Nandakumar aditya_nandakumar at apple.com
Fri Oct 17 15:56:42 PDT 2014


A non-text attachment was scrubbed...
Name: ChangeTLOFOwnership.patch
Type: application/octet-stream
Size: 16806 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141017/50cf8ac0/attachment.obj>
-------------- next part --------------


Hi All

This patch changes the ownership of TLOF from TargetLoweringBase to TargetMachine so that different subtargets could share the TLOF effectively.
I have made the change to X86 only currently and it passes make check. I could update the other targets after feedback.

Please review.

Thanks
Aditya

PS: The additional lines which seem like no change are actually due to my editor stripping trailing whitespaces on save. I could disable that if required.


More information about the llvm-commits mailing list