[PATCH] Adding the missing LTO API to lto.exports

Duncan P. N. Exon Smith dexonsmith at apple.com
Tue Jun 23 11:58:13 PDT 2015


LGTM too, thanks for the fix.

> On 2015-Jun-23, at 11:51, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
> 
> LGTM
> 
> On 23 June 2015 at 13:04, Steven Wu <stevenwu at apple.com> wrote:
>> Hi dexonsmith,
>> 
>> lto_codegen_set_should_embed_uselists is introduced in r235943 but not
>> added to lto.exports. Add to export list to expose the API.
>> 
>> http://reviews.llvm.org/D10658
>> 
>> Files:
>>  tools/lto/lto.exports
>> 
>> Index: tools/lto/lto.exports
>> ===================================================================
>> --- tools/lto/lto.exports
>> +++ tools/lto/lto.exports
>> @@ -42,6 +42,7 @@
>> lto_codegen_optimize
>> lto_codegen_compile_optimized
>> lto_codegen_set_should_internalize
>> +lto_codegen_set_should_embed_uselists
>> LLVMCreateDisasm
>> LLVMCreateDisasmCPU
>> LLVMDisasmDispose
>> 
>> EMAIL PREFERENCES
>>  http://reviews.llvm.org/settings/panel/emailpreferences/
>> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits





More information about the llvm-commits mailing list