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

Lu Weining via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 7 00:57:00 PST 2022


SixWeining added a comment.

In D115857#3225040 <https://reviews.llvm.org/D115857#3225040>, @xen0n wrote:

> As previously discussed, we might want to wait until the gcc port is merged, to avoid code churn regarding triple names here.

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

We can even add some dummy triple test like this, right?

  T = Triple("loongarch64-unknown-darwin");
  EXPECT_EQ(Triple::loongarch64, T.getArch());
  EXPECT_EQ(Triple::UnknownVendor, T.getVendor());
  EXPECT_EQ(Triple::Darwin, T.getOS());
  EXPECT_EQ(Triple::UnknownEnvironment, T.getEnvironment());


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115857



More information about the llvm-commits mailing list