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

Joel Jones via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 11 10:14:32 PDT 2016


joelkevinjones added a comment.

I'm going to be working on this patch from Ananth. For the AsmParser test, I'm assuming that the file being referred to is: unittests/Support/TargetParserTest.cpp? If so, I have several questions:

1. What is this testing exactly? It just seems to be repeating the information in include/llvm/Support/AArch64TargetParser.def
2. Is simply adding "thunderx" to the "CPU" array initialization sufficient?


Repository:
  rL LLVM

https://reviews.llvm.org/D24540





More information about the llvm-commits mailing list