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.

Sign up for GitHub

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

Jump to bottom

Fix issue#2954. #2967

Closed

CCCptH wants to merge 1 commit into xmake-io:dev from CCCptH:issue-2954

CCCptH commented Oct 24, 2022

Character : can not exist in path in windows system. So when using clang to build project that contains module partitions, the filename of bmifile it generates should not contains :
Change bmifile filename according to https://clang.llvm.org/docs/StandardCPlusPlusModules.html#global-module-fragment..

The file name of BMIs should end with .pcm. The file name of the BMI of a primary module interface unit should be module_name.pcm. The file name of BMIs of module partition unit should be module_name-partition_name.pcm.


          Fix issue#2954.

968f1ff

Change bmifile filename according to https://clang.llvm.org/docs/StandardCPlusPlusModules.html#global-module-fragment.

waruqi changed the base branch from master to dev

October 24, 2022 10:03

Member

waruqi commented Oct 24, 2022

waruqi closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet