[PATCH] D78513: [hip] Claim builtin type `__float128` supported if the host target supports it.

Michael Liao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 20 14:38:43 PDT 2020


hliao added a comment.

In D78513#1993115 <https://reviews.llvm.org/D78513#1993115>, @yaxunl wrote:

> Currently if instructions of float128 get to amdgpu backend, are we going to crash?


As `Float128Format` is re-defined as `double`, there won't be any issue in the backend. But, it won't function as the developer expects. That's quite similar to `long double` in clang.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78513





More information about the cfe-commits mailing list