[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