Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compile private C++20 modules #3125

Merged
merged 7 commits into from
Dec 2, 2022
Merged

Conversation

Arthapz
Copy link
Member

@Arthapz Arthapz commented Dec 1, 2022

We forgot to build privates modules

@waruqi
Copy link
Member

waruqi commented Dec 1, 2022

please remove empty dummy.cpp, it's unused.

@waruqi
Copy link
Member

waruqi commented Dec 1, 2022

We also need to consider incremental compilation. It does not work for this test case. But other tests work.

for clang

ruki-2:private_module ruki$ xmake 
[  0%]: generating.cxx.module.deps src/use.mpp
[  0%]: generating.cxx.module.deps src/dep1.mpp
[  0%]: generating.cxx.module.deps src/dep2.mpp
[ 14%]: generating.cxx.module.bmi dep1
[ 28%]: generating.cxx.module.bmi dep2
[ 42%]: build.cxx.module src/use.mpp
[ 71%]: archiving.release libprivate_module.a
[100%]: build ok!
ruki-2:private_module ruki$ xmake
[  0%]: generating.cxx.module.deps src/use.mpp
[ 42%]: build.cxx.module src/use.mpp
[ 71%]: archiving.release libprivate_module.a
[100%]: build ok!

@Arthapz
Copy link
Member Author

Arthapz commented Dec 1, 2022

We also need to consider incremental compilation. It does not work for this test case. But other tests work.

for clang

ruki-2:private_module ruki$ xmake 
[  0%]: generating.cxx.module.deps src/use.mpp
[  0%]: generating.cxx.module.deps src/dep1.mpp
[  0%]: generating.cxx.module.deps src/dep2.mpp
[ 14%]: generating.cxx.module.bmi dep1
[ 28%]: generating.cxx.module.bmi dep2
[ 42%]: build.cxx.module src/use.mpp
[ 71%]: archiving.release libprivate_module.a
[100%]: build ok!
ruki-2:private_module ruki$ xmake
[  0%]: generating.cxx.module.deps src/use.mpp
[ 42%]: build.cxx.module src/use.mpp
[ 71%]: archiving.release libprivate_module.a
[100%]: build ok!

now it should work

@waruqi waruqi merged commit 427f36f into xmake-io:dev Dec 2, 2022
@Arthapz Arthapz deleted the compile-private-modules branch February 3, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants