[Review] Change TargetLoweringObjectFile ownership

Aditya Nandakumar aditya_nandakumar at apple.com
Fri Oct 17 16:56:11 PDT 2014


Moved to Phabricator
http://reviews.llvm.org/D5850 <http://reviews.llvm.org/D5850> 

Thanks
Aditya

> On Oct 17, 2014, at 3:56 PM, Aditya Nandakumar <aditya_nandakumar at apple.com> wrote:
> 
> <ChangeTLOFOwnership.patch>
> 
> 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._______________________________________________
> 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/20141017/d90fcbc1/attachment.html>


More information about the llvm-commits mailing list