[PATCH] Select new undefined atom rather than old one if other conditions are the same.

Rui Ueyama ruiu at google.com
Thu Nov 14 18:57:33 PST 2013


On Thu, Nov 14, 2013 at 6:41 PM, kledzik at apple.com <kledzik at apple.com>wrote:

>
>
> ================
> Comment at: lib/Core/SymbolTable.cpp:192-194
> @@ +191,5 @@
> +    const UndefinedAtom *newFallback = newUndef->fallback();
> +    bool hasDifferentFallback =
> +        (existingFallback && newFallback &&
> +         existingFallback->name() != newFallback->name());
> +    if (hasDifferentFallback) {
> ----------------
> hasDifferentFallback should also be true if one has a fallback and the
> other does not.
>

If the existing atom does not have a fallback and new one has, it's normal
-- no need to warn that. So the condition would be a bit complicated that.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131114/52a4e879/attachment.html>


More information about the llvm-commits mailing list