2db239acd1
The import and include problem is a long-standing issue with the use of C++20 modules. This patch tried to improve this by skipping parsing class and functions if their declaration is already defined in modules. The scale of the patch itself is small as the patch reuses previous optimization. Maybe we can skip parsing other declarations in the future. But the patch itself should be good.
42 lines
1.5 KiB
C++
42 lines
1.5 KiB
C++
// RUN: rm -rf %t
|
|
// RUN: mkdir %t
|
|
// RUN: split-file %s %t
|
|
//
|
|
// RUN: %clang_cc1 -std=c++20 %t/foo.cppm -I%t -emit-module-interface -o %t/foo.pcm
|
|
// RUN: %clang_cc1 -fprebuilt-module-path=%t -std=c++20 %t/use.cpp -fsyntax-only -verify
|
|
|
|
// RUN: %clang_cc1 -std=c++20 %t/foo.cppm -I%t -emit-reduced-module-interface -o %t/foo.pcm
|
|
// RUN: %clang_cc1 -fprebuilt-module-path=%t -std=c++20 %t/use.cpp -fsyntax-only -verify
|
|
|
|
//--- foo.cppm
|
|
export module foo;
|
|
export template <typename T = int>
|
|
T v;
|
|
|
|
export template <int T = 8>
|
|
int v2;
|
|
|
|
export template <typename T>
|
|
class my_array {};
|
|
|
|
export template <template <typename> typename C = my_array>
|
|
int v3;
|
|
|
|
//--- use.cpp
|
|
import foo;
|
|
template <typename T = int>
|
|
T v; // expected-error {{declaration of 'v' in the global module follows declaration in module foo}}
|
|
// expected-note@foo.cppm:3 {{previous declaration is here}}
|
|
|
|
template <int T = 8>
|
|
int v2; // expected-error {{declaration of 'v2' in the global module follows declaration in module foo}}
|
|
// expected-note@foo.cppm:6 {{previous declaration is here}}
|
|
|
|
template <typename T>
|
|
class my_array {}; // expected-error {{declaration of 'my_array' in the global module follows declaration in module foo}}
|
|
// expected-note@foo.cppm:9 {{previous declaration is here}}
|
|
|
|
template <template <typename> typename C = my_array>
|
|
int v3; // expected-error {{declaration of 'v3' in the global module follows declaration in module foo}}
|
|
// expected-note@foo.cppm:12 {{previous declaration is here}}
|