[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.
Thanks,
Duncan.
More information about the llvm-commits
mailing list