[PATCH] D142688: [Clang][Driver] Handle LoongArch multiarch tuples

Xi Ruoyao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 16 01:08:07 PDT 2023


xry111 added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Arch/LoongArch.cpp:66
+  default:
+    return IsLA32 ? "ilp32d" : "lp64d";
+  }
----------------
SixWeining wrote:
> Better to be `f`? (Probably most 32-bit hardwares do not support double-float? But I'm not sure about this.)
The ISA manual says 64-bit float support is not limited by 32-bit basic ISA support (except the moving instructions accessing 64-bit GPR).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142688



More information about the cfe-commits mailing list