[llvm-commits] [llvm] r56341 - in /llvm/trunk: include/llvm/ include/llvm/Transforms/ lib/Transforms/IPO/ test/Analysis/GlobalsModRef/ test/Transforms/AddReadAttrs/

Duncan Sands baldrick at free.fr
Sun Sep 21 02:10:59 PDT 2008

On Saturday 20 September 2008 20:00:38 Anton Korobeynikov wrote:
> > I was following the logic in resolveAliasedGlobal (which
> > shouldn't resolve aliases if they are not definitive, i.e.
> > may be overridden).  Perhaps Anton can comment.  Anyway,
> > hopefully PR2742 will take care of all such problems.
> If this applies only to alias - then check for weak linkage only is
> correct: aliases can have only external, weak or internal linkage.
> resolveAliasedGlobal() walks through the alising chain until it will
> find weak alias (or non-alias).

I see.  For functions, which linkage types mean that the body
(definition) cannot be relied upon as being correct?  I'll add
a method (isDefinitive?) for this.



More information about the llvm-commits mailing list