[PATCH] D110514: MIPS: Triple::setArch as R6 if MipsSubArch_r6

Simon Atanasyan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 27 14:44:48 PDT 2021


atanasyan added a comment.

If we construct `T = Triple("mipsisa64r6el-unknown-linux-gnuabi64")` what is result of the `T.getArchName()` call (without your patch)? If `getArchName()` returns `mipsisa64r6el` we need to investigate what is the reason for redundant `setArch()` call somewhere in the clang driver code. If `getArchName()` does not return `mipsisa64r6el` we need to fix triple parsing. One more question - why other targets which has "sub architectures" does not need to extend the `Triple::setArch` method?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110514



More information about the llvm-commits mailing list