[PATCH] D158778: [CUDA] Propagate __float128 support from the host.

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 29 11:09:45 PDT 2023


tra added a subscriber: jhuber6.
tra added a comment.

In D158778#4624408 <https://reviews.llvm.org/D158778#4624408>, @ABataev wrote:

> Just checks removal should be fine

Looks like OpenMP handles long double and __float128 differently -- it always insists on using the host's FP format for both.
https://github.com/llvm/llvm-project/blob/d037445f3a2c6dc1842b5bfc1d5d81988c2f223d/clang/lib/AST/ASTContext.cpp#L1674

This creates a divergence between what clang thinks and what LLVM can handle.
I'm not quite sure how it's supposed to work with NVPTX or AMDGPU, where we demote those types to double and can't generate code for the actual types.

@jhuber6 what does OpenMP expect to happen for those types on the GPU side?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158778



More information about the cfe-commits mailing list