[Patch] Add TargetLibraryInfo in LTO passes builder

Yi Jiang yjiang at apple.com
Wed Dec 11 17:42:02 PST 2013


Thank you for the review. Checked in r197105.
On Dec 10, 2013, at 10:51 PM, Bill Wendling <isanbard at gmail.com> wrote:

> On Dec 9, 2013, at 4:39 PM, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
> 
>> On Dec 9, 2013, at 3:52 PM, Yi Jiang <yjiang at apple.com> wrote:
>> 
>>> Thank you for the clarification. Here is the patch with test case. 
>>> <lto.libinfo.patch>
>> 
>> This looks right to me.
>> 
>> Bill, you’re listed as code owner for LTO.  Was there a good reason TargetLibraryInfo wasn’t being set before?
> 
> Not to my knowledge. The patch looks fine to me.
> 
> -bw
> 





More information about the llvm-commits mailing list