[clang] [clang] Inject tokens containing #embed back into token stream (PR #97274)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 16 05:08:49 PDT 2024


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


More information about the cfe-commits mailing list