[PATCH] D43027: [ThinLTO] Skip BlockAddresses while replacing uses in function import

[via llvm-commits]

How do I generate yams summary file? 
Tests for -lowertypetests-write-summary always read an input summary.
How do I generate the input?

> On Feb 7, 2018, at 12:17 PM, Peter Collingbourne via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> pcc added a comment.
> 
> In https://reviews.llvm.org/D43027#1000808, @tejohnson wrote:
> 
>> I'd like @pcc to comment. I noticed there are a number of other places in this file where we will still be calling replaceAllUsesWith on a Function. How do we know which should use which method?
> 
> 
> We need to use `replaceUsesExceptBlockAddr` if the function that we are replacing is a function definition. As far as I can tell, in all other cases we are replacing function declarations.
> 
> 
> 
> ================
> Comment at: llvm/test/ThinLTO/X86/blockaddr-import.ll:2
> +; RUN: opt -thinlto-bc %s -o %t1.bc
> +; RUN: llvm-lto2 run %t1.bc -r=%t1.bc,o,x -r=%t1.bc,l,plx -r=%t1.bc,m,plx -r=%t1.bc,d,plx -o %t2
> +
> ----------------
> Can this be written as an IR-level test of the pass? See for example `llvm/test/Transforms/LowerTypeTests/import-icall.ll`.
> 
> 
> https://reviews.llvm.org/D43027
> 
> 
>