[PATCH] D115857: [LoongArch 1/6] Add triples loongarch{32, 64} for the upcoming LoongArch target

[Renato Golin via Phabricator via llvm-commits]

rengolin added a comment.

In D115857#3226913 <https://reviews.llvm.org/D115857#3226913>, @SixWeining wrote:

> Those triples used in this patch are only for triple-parsing test and I think it doesn't affect triple definitions by gcc.

Agreed. `target.unknown.unknown` should always work, regardless of what GCC does.

Those tests only check if the target is valid, completely ignoring the rest, which given what this patch introduces (just the target), it's a fair test.

Once GCC agrees on a set of triples, we can add them here again with a new patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115857