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

Chris Bowler via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 19 08:24:01 PDT 2021


cebowleratibm requested changes to this revision.
cebowleratibm added inline comments.
This revision now requires changes to proceed.


================
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())
----------------
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.


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