[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