aaron.ballman added inline comments.

================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:8422
 
+def err_int_to_float_bit_int_max_size : Error<
+  "cannot convert '_BitInt' operands of more than %0 bits to floating point">;
----------------
mgehre-amd wrote:
> erichkeane wrote:
> > Can you explain the issue here?  This is supposed to be well-defined 
> > behavior.
> I saw `Assertion LC != RTLIB::UNKNOWN_LIBCALL && "Don't know how to expand 
> this SINT_TO_FP!"' failed.` in the backend. I think we would need to add 
> library calls for floating to bitint (and vice versa) to the bitint library 
> to enable that.
Good catch! I think we'll need to solve that before we can enable wide 
bit-width support (users are going to expect assignment and initialization to 
not crash as those are basic builtin operators). FWIW, this is a reproducer 
from Clang 13 where we still allowed large widths: 
https://godbolt.org/z/hvzWq1KTK


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122234

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

Reply via email to