[PATCH] D112906: [PowerPC] Emit warning for ieeelongdouble on older GNU toolchain

Tue Feb 8 08:55:37 PST 2022

qiucf added a comment.

In D112906#3305014 <https://reviews.llvm.org/D112906#3305014>, @jwakely wrote:

> In D112906#3304925 <https://reviews.llvm.org/D112906#3304925>, @qiucf wrote:
>
>> Is that because clang lacks something required by this feature? (for example. clang-12 doesn't have `__ibm128` and many builtins) If so, clang-14 should have these fixed.
>
> When I implemented the libstdc++ changes Clang didn't have any of your work yet. Now that everything seems to be present, and `__LONG_DOUBLE_IEEE128__` and `__LONG_DOUBLE_IBM128__` are defined as appropriate, I think I can enable it for clang. I'll look into that tomorrow.

Thanks! I verified latest LLVM/Clang with latest libstdc++ in some programs and they look okay. If you found any more issues, I still have time to get them fixed in clang 14 rc2.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112906