[PATCH] D50858: [M680x0] Add ELF and Triple info

Thu Aug 16 12:20:45 PDT 2018

m4yers added a comment.

In https://reviews.llvm.org/D50858#1202964, @lebedev.ri wrote:

> I'm not sure how ELF/relocation stuff should be tested, but i *think* it can also be split into a second review.
> Though
>
>   llvm/test$ find -iname *Relocs*
>
>
> does find some things.

There are tests for relocations in the main diff https://reviews.llvm.org/D50314 `test/CodeGen/M680x0/OBJ/Relocations`, but those require the full backend available. Aside from that I'm not really sure how to test them in a separate diff. For now they are just data copied from the m68k manual.

> Please add appropriate test coverage to unittests/ADT/TripleTest.cpp.

Will do

Repository:
  rL LLVM

https://reviews.llvm.org/D50858