[PATCH] D106074: [AIX] Emit unsupported 128-bit long double option for AIX

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 19 09:07:29 PDT 2021


hubert.reinterpretcast added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:4845
+  if (Arg *A = Args.getLastArg(options::OPT_mlong_double_128)) {
+    // AIX doesn't support 128-bit long double yet.
+    if (Triple.isOSAIX())
----------------
cebowleratibm wrote:
> It's not that AIX doesn't support 128-bit long double.  The xlc compiler has -qldbl128, which links in different libraries for 128-bit.  The XL compilers use -lc128 and -lC128 so there are binary compatibility concerns that need to be investigated.
> 
> I suggest this comment be updated accordingly.
Note also to update the commit message.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106074



More information about the cfe-commits mailing list