[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