[PATCH] D24540: [AArch64] Add Cavium ThunderX subtarget support.

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 18 05:05:48 PDT 2016


rengolin accepted this revision.
rengolin added a reviewer: rengolin.
rengolin added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D24540#572750, @jmolloy wrote:

> Hi,
>
> This generally looks good to me - I agree that the targetparser testing is pretty tautological. Eric, do you have input on whether that's needed?


We also have some parsing tests to make sure we're not loosing anything, but I agree, the number of relevant tests there is not great.

I think this can be a task for a separate patch.

cheers,
--renato


Repository:
  rL LLVM

https://reviews.llvm.org/D24540





More information about the llvm-commits mailing list