Fznamznon added a comment.

In D74387#1974981 <https://reviews.llvm.org/D74387#1974981>, @jdoerfert wrote:

> As I mentioned before. As long as the type is not "used" you can treat it as 
> a sequence of bytes just as well. So we can lower `__float128` to `char [16]` 
> with the right alignment. SPIRV will never see unsupported types and the code 
> works because we never access it as `float128` anyway. WDYT?


Yes, it can work for SYCL without additional diagnostics if it is possible to 
replace `__float128` with `char [16]` everywhere (including struct definitions 
and so on) in the resulting LLVM IR module.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74387



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

Reply via email to