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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
