[PATCH] D18986: [ThinLTO] Prevent importing of "llvm.used" values

[Mehdi AMINI via llvm-commits]

joker.eph added a comment.

In http://reviews.llvm.org/D18986#404380, @tejohnson wrote:

> In http://reviews.llvm.org/D18986#404324, @joker.eph wrote:
>
> > Also, could we promote and rename while keeping a private alias with the old name for these globals?
>
>
> No, because we could import inline assembly from two different modules, each containing a reference to a local 'myvar', and the two imported uses within the imported inline assembly need to each see the correct corresponding promoted global.


There are two things:

1. "llvm.used" symbols (that are referred *from* inline assembly)
2. Functions that contains inline assembly themselves.

I was referring to the first one while you are mentioning the second ones right?


http://reviews.llvm.org/D18986