[Mesa-stable] [Mesa-dev] [PATCH 4/7] Travis: add binutils 2.26 for a few more LLVM 3.9 builds

[Jan Vesely]

On Thu, 2017-10-05 at 17:46 +0100, Emil Velikov wrote:
> On 5 October 2017 at 17:28, Jan Vesely <jan.vesely at rutgers.edu> wrote:
> > On Thu, 2017-10-05 at 11:19 +0100, Emil Velikov wrote:
> > > From: Emil Velikov <emil.velikov at collabora.com>
> > > 
> > > Otherwise we error out at link stage as follows:
> > > 
> > > /usr/lib/llvm-3.9/lib/libLLVMAMDGPUCodeGen.a(R600OptimizeVectorRegisters.cpp.o):
> > > unrecognized relocation (0x2a) in section
> > > `.text._ZNK12_GLOBAL__N_119R600VectorRegMerger16getAnalysisUsageERN4llvm13AnalysisUsageE'
> > > /usr/bin/ld: final link failed: Bad value
> > > 
> > > Cc: mesa-stable at lists.freedesktop.org
> > > Cc: Jan Vesely <jan.vesely at rutgers.edu
> > > Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> > > ---
> > > Suspecting that there are due to LLVM/Clang itself being built with the
> > > newer version. Something like that seems to be mentioned below.
> > > AKA packaging bug or we're using the wrong package ;-)
> > > 
> > > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=808205
> > > 
> > > AFAICT it has nothing to do with linking against libclang.so
> > 
> > it does, in a way that linking to dynamic library does not hit this
> > issue.
> > 
> 
> Right, that's how to bypass. I'm thinking about the root cause ;-)
> 
> > > 
> > > Jan any reason you opted to address only one of the LLVM 3.9 builds?
> > > Should we bother with the rest, set it globally, other?
> > 
> > the other builds did not need it. I only hit it when linking to static
> > clang libraries (for whatever reason it decided to use static
> > libraries). I assume you started hitting this when you switched
> > enable_llvm_shared_libs off by default?
> > 
> 
> Yes, anything LLVM (project) related that is static linked exhibits the problem.
> 
> Issue is restricted to 3.9, so I seriously suspect something fishy in
> the code or package.

I think your suspicion that they used binutils 2.26 when building the
package is reasonable. I don't know much about packaging, let alone
ubuntu packaging so I can't really add much here. sorry.

Jan

> 
> -Emil
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/mesa-stable/attachments/20171005/adc3a98c/attachment.sig>