AaronBallman wrote:

> I thought there was agreement that at least for C the literals have type int,

That's correct; this patch gets started on addressing that issue by updating 
the way we expand the embedded contents but isn't intended to fix the type 
issue yet (that will be a follow-up patch). When we do the follow-up, we should 
ensure the embedded elements are of type `int` (for both C and C++; we don't 
want different types coming out of the preprocessor depending on language mode 
as that will make life harder for mixed C and C++ code bases).

https://github.com/llvm/llvm-project/pull/97274
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to