[PATCH] D130820: [cmake] Add LoongArch to config.guess
Lu Weining via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jul 31 20:21:35 PDT 2022
SixWeining accepted this revision.
SixWeining added a comment.
LGTM. Thanks.
================
Comment at: llvm/cmake/config.guess:925
exit ;;
+ loongarch32:Linux:*:* | loongarch64:Linux:*:* | loongarchx32:Linux:*:*)
+ echo "${UNAME_MACHINE}-unknown-linux-${LIBC}"
----------------
xen0n wrote:
> SixWeining wrote:
> > I'm not sure it exists or not. Let me double check with others internally. Thanks.
> IIRC it was planned to be something like the MIPS n32, but due to the later shift to the RISC-V-like model it should no longer exist. The "LPX32" ABI should never see the light of day.
>
> But again here I'm just updating the script to match the upstream behavior. If `loongarchx32` should be deleted here, then the upstream config.git should get updated too.
I agree with you. Let's remove it after upstream config.git has done so.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130820/new/
https://reviews.llvm.org/D130820
More information about the llvm-commits
mailing list