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

concurrencpp: support some platform #4834

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Conversation

star-hengxing
Copy link
Contributor

No description provided.

@star-hengxing star-hengxing requested a review from waruqi August 3, 2024 10:55
@star-hengxing
Copy link
Contributor Author

🤔on_check failed with

error: @programdir\core\main.lua:329: @programdir\core\package\package.lua:2420: package(concurrencpp): links not found!

@waruqi
Copy link
Member

waruqi commented Aug 4, 2024

先放着吧 等下周我看下

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


Let it go for now. I'll take a look at it next week.

@waruqi
Copy link
Member

waruqi commented Aug 4, 2024

🤔on_check failed with

error: @programdir\core\main.lua:329: @programdir\core\package\package.lua:2420: package(concurrencpp): links not found!

这个我修了,改成仅仅在 on_test 才做检测。。

void test() {
auto x = std::hardware_destructive_interference_size;
}
]]}, {configs = {languages = "c++20"}}), "package(concurrencpp) Require at least C++20.")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不过这里似乎还是不太准,我 macos 上测试

8C454D9092B3AAD9C2C3C0B4.cpp:7:31: error: no member named 'hardware_destructive_interference_size
' in namespace 'std'
                auto x = std::hardware_destructive_interference_size;
                         ~~~~~^

但是去掉这个 最后安装编译 on_test 都能过。

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.

3 participants