yronglin wrote: > I think we should implement CWG2947 as written and allow that. CWG2947 has > not yet been accepted, but I don't believe there will be issues there.
I've temporarily added CWG2947 support in the patch. This needs tests for `-E`. > The line separation is not getting maintained: > ```cpp > export module m; int x; > extern "C++" int *y = &x; > ``` > > Output: > ``` > # 1 "<stdin>" > # 1 "<built-in>" 1 > # 1 "<built-in>" 3 > # 665 "<built-in>" 3 > # 1 "<command line>" 1 > # 1 "<built-in>" 2 > # 1 "<stdin>" 2 > export module m; int x;extern "C++" int *y = &x; > ``` Hmm yes, this is a bug in current implementation. I'll fix this later. https://github.com/llvm/llvm-project/pull/107168 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
