[llvm-commits] [PATCH] Lazily Link functions

Duncan Sands baldrick at free.fr
Wed Oct 26 23:22:39 PDT 2011


Hi Tanya,

> Hmm.. if I am understanding all these different linkage types correctly,

yeah, there are so many and often obscure - I'm not sure they are even all used
in practice...

  then I don't think it should be a problem to change this to:
> if (SF->hasLocalLinkage() || SF->hasLinkOnceLinkage() || SF.hashasAvailableExternallyLinkage())

how about introducing a helper function in GlobalValue for testing this, eg:
"notNeededIfUnusedLinkage", and use this in both places?

Ciao, Duncan.



More information about the llvm-commits mailing list