[PATCH] D121445: [Clang][CSKY] Add the CSKY target and compiler driver

[Renato Golin via Phabricator via cfe-commits]

rengolin added a comment.

I don't know enough about your toolchain requirements, but this looks good to me.

Please check the clang-format warnings. If you did pass clang-format, perhaps you need to upgrade to a newer one?

I won't approve just yet, to let other people review it also.



================
Comment at: clang/test/Driver/csky-arch.c:26
+
+// RUN: %clang -target csky-unknown-elf -march=csky -### %s \
+// RUN: -fsyntax-only 2>&1 | FileCheck -check-prefix=CSKY %s
----------------
I don't think this is doing what you expect it to do.

Depending on the output, you can still match all CHECK lines and not have the output you want.

To avoid issues, we usually separate tests that must pass (positive tests) like the lines above 24, from tests that must fail (negative tests) like the lines below.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121445/new/

https://reviews.llvm.org/D121445