[patch] Clarify and verify what llvm.used is

John McCall rjmccall at apple.com
Tue Apr 16 16:40:10 PDT 2013


On Apr 16, 2013, at 4:18 PM, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
> When reviewing my previous patch to GlobalOpt Duncan raised some
> questions about the semantics of llvm.used. We should probably clarify
> the documentation and have the verifier check it before anything else.
> 
> The attached patch makes it explicit that aliases can be put in
> llvm.used and changes the verifier to check llvm.used only has
> functions, variables and aliases.
> 
> Duncan, are you OK with this definition? Is the patch OK?

This looks fine to me, but did you intend to post this to llvm-commits, too?

John.



More information about the cfe-commits mailing list