Re: [PATCH] c++: Handle RAW_DATA_CST in modules.cc [PR119076]

2025-03-11 Thread Jason Merrill
On 3/4/25 4:02 AM, Jakub Jelinek wrote: Hi! The following testcases (one with #embed, one with large initializer turned into RAW_DATA_CST) show that I forgot to handle RAW_DATA_CST in module streaming. Similar to the PCH case we need to stream out RAW_DATA_CST with NULL RAW_DATA_OWNER (i.e. a t

C++ patch ping (Re: [PATCH] c++: Handle RAW_DATA_CST in modules.cc [PR119076])

2025-03-11 Thread Jakub Jelinek
Hi! On Tue, Mar 04, 2025 at 10:02:58AM +0100, Jakub Jelinek wrote: > The following testcases (one with #embed, one with large initializer > turned into RAW_DATA_CST) show that I forgot to handle RAW_DATA_CST in > module streaming. > > Similar to the PCH case we need to stream out RAW_DATA_CST wit

[PATCH] c++: Handle RAW_DATA_CST in modules.cc [PR119076]

2025-03-04 Thread Jakub Jelinek
Hi! The following testcases (one with #embed, one with large initializer turned into RAW_DATA_CST) show that I forgot to handle RAW_DATA_CST in module streaming. Similar to the PCH case we need to stream out RAW_DATA_CST with NULL RAW_DATA_OWNER (i.e. a tree which has data owned by libcpp buffer)