[llvm-commits] [llvm] r68037 - /llvm/trunk/include/llvm/Constant.h

[Anton Korobeynikov]

Hi, Dan

> The rest of the comment seems to support that it's
> redundant. However, many optimizations that apply to
> symbols with internal and private linkage also apply to
> symbols with hidden or protected visibility and a
> definition in the same module, so the intent here
> isn't clear.
Here I tried to mimic gcc's behaviour. It does not check for
visibility at all during computation of relocations. I don't know for
sure, whether adding visibility stuff here would break things or not,
that's why I tried to be as conservative as possible.

-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University