Fwd: NDS32 V3 backend

Shiva Chen via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 16 17:52:26 PST 2017


Hi Joerg and Tobias,

Thanks for the comment.
I reorganize the patches and hook up the printer as soon as possible.
Before hooking up the printer, only necessary preliminary target class
(RegisterInfo/InstrInfo/...) have been added.
After adding the printer, each patch would add certain instructions
with  testcases.

You could find the new patches here:
* <https://reviews.llvm.org/differential/?authors=shiva0217>

Does the new patches ok for reviewing ?
Please let me know if i still missing something.

Thanks,
Shiva

2017-01-19 20:21 GMT+08:00 Joerg Sonnenberger via llvm-commits
<llvm-commits at lists.llvm.org>:
> On Thu, Jan 19, 2017 at 05:05:57PM +0800, Shiva Chen wrote:
>> Hi Tobias and Joerg,
>>
>> To my understanding, the test cases in test/CodeGen could run by lit
>> until InstPrinter have been implement.
>> So, should I add the CodeGen test cases in the
>> [NDS32 17/22] add NDS32 Instruction Printer  patch ?
>> Or before the InstPrinter have been implement, there're still other
>> testcase I should add ?
>
> This is the reason why I would hook up the instruction printer as soon
> as possible. It makes testing the functionality of everything else much
> easier.
>
> Joerg
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list