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.

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 #31 set -O2 while Compiling #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix #31 set -O2 while Compiling #46

wants to merge 1 commit into from

Conversation

zccold
Copy link

@zccold zccold commented Sep 17, 2019

原来的配置文件中没有对c++开启编译优化，开启后性能提升到正常水平（8倍左右）

@zccold
Copy link
Author

zccold commented Sep 17, 2019

ci是java11的，这个nar-maven貌似不支持，如果需要必须过ci那只能先解决那个问题了……
自测用jdk8没有问题

@zccold zccold force-pushed the master branch 4 times, most recently from e0935fc to fdb9523 Compare September 23, 2019 15:55
@Oneplus
Copy link
Member

Oneplus commented Oct 17, 2019

@zccold 非常感谢pr。实在抱歉，我现在没有时间维护这个项目。但看起来好像是只要在ci里指定jdk8就行了。如果这个可行，能不能再交一个ci的pr。非常感谢！

@zccold
Copy link
Author

zccold commented Oct 17, 2019

@Oneplus 不太熟悉Travis，实际上我上上周按照文档弄了半天，用各种方式都没能使得指定jdk版本生效。。。。。有时间我再看一下吧

@Oneplus
Copy link
Member

Oneplus commented Oct 17, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zccold @Oneplus