[PATCH] D109950: [Clang] Enable IC/IF mode for __ibm128

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 1 18:46:40 PDT 2021


rjmccall added a comment.

Thanks.  @hubert.reinterpretcast, @qiucf, can you verify that other compilers for PPC follow the logic for `TF` / `TC` that we now have with Elizabeth's patch?  There are three different type specifiers (`long double`, `__ibm128`, and `float128_t`) which represent formally distinct types, and IIUC there are a bunch of different flags and target options that change the meaning of `long double` and/or disable/enable `__ibm128` and `float128_t`.  We care about exactly which type is produced by the mode attribute, so you'll have to do something which is sensitive to the exact formal type, like `_Generic` or a C++ template or doing a pointer conversion without a cast; checking code generation will only tell us the underlying format.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109950



More information about the cfe-commits mailing list