[llvm-commits] [llvm] r167864 - in /llvm/trunk: include/llvm/Support/ELF.h lib/Target/PowerPC/MCTargetDesc/PPCELFObjectWriter.cpp

[Ulrich Weigand]

Chandler Carruth <chandlerc at google.com> wrote on 13.11.2012 21:05:38:

> > URL: http://llvm.org/viewvc/llvm-project?rev=167864&view=rev
> > Log:
> > Add (some) PowerPC TLS relocation types to ELF.h and
> > generate them from PPCELFObjectWriter::getRelocTypeInner
> > as appropriate.
>
> Is this fixing a JIT test that is currently failing on PPC hosts? If
> so, please mention it with the commit so folks understand that is how
> it is being tested. If not, a new test case to cover this seems
> useful.

Well, it fixes the TLS tests in test-suite:
SingleSource/UnitTests/Threads/2010-12-08-tls
SingleSource/UnitTests/Threads/tls
when using clang with the integrated assembler on PowerPC.

But for this particular patch, it may be feasible to construct a test
even without having the asm parser in place, something along the lines
of test/MC/PowerPC/ppc64-relocs-01.ll.  I'll work on that.

Thanks,
Ulrich