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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to