igorkudrin wrote:

> If we think the argument is always supposed to fit into 64 bits, we should do 
> the truncation in Sema, as a conversion to uint64_t or something like that. 
> The generated code is basically the same, but the implicit conversion in Sema 
> will generate a -Wconversion warning. (Which is not on by default, but better 
> than nothing.)

Are you suggesting calling `Sema::CheckImplicitConversion()` explicitly? It 
seems `Sema::ImpCastExprToType()` does not produce the diagnostic.

https://github.com/llvm/llvm-project/pull/197620
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to