Skip to content

Use link time optimization for CI release build #877

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
Merged
merged 10 commits into from
Jan 21, 2025
Merged

Use link time optimization for CI release build #877

merged 10 commits into from
Jan 21, 2025

Conversation

TonyXiang8787
Copy link
Member

Use Link Time Optimization for CI release build for all targets. This is achieved by setting CMAKE_INTERPROCEDURAL_OPTIMIZATION in the preset.

In the user build, the C-API always enables LTO.

@TonyXiang8787
use link time optimization for CI release build
d54b032 
Signed-off-by: Tony Xiang <tony.xiang@alliander.com>
@TonyXiang8787
try to turn off odr violation check in address santitizer
108dfc3 
Signed-off-by: Tony Xiang <tony.xiang@alliander.com>
@TonyXiang8787
try define separately
204db40 
Signed-off-by: Tony Xiang <tony.xiang@alliander.com>
@TonyXiang8787
format
e916cd0 
Signed-off-by: Tony Xiang <tony.xiang@alliander.com>
@TonyXiang8787 TonyXiang8787 marked this pull request as draft January 21, 2025 09:03
TonyXiang8787 and others added 4 commits January 21, 2025 14:36
@TonyXiang8787
Merge branch 'main' into feature/use-lto-for-ci-release
c338e0d
@TonyXiang8787
try to enable LTO for clang-tidy
923adff 
Signed-off-by: Tony Xiang <tony.xiang@alliander.com>
@TonyXiang8787
Revert "try to enable LTO for clang-tidy"
e60dac1 
This reverts commit 923adff.
@TonyXiang8787
DCO Remediation Commit for Tony Xiang <tony.xiang@alliander.com>
96f1626 
I, Tony Xiang <tony.xiang@alliander.com>, hereby add my Signed-off-by to this commit: e60dac1

Signed-off-by: Tony Xiang <tony.xiang@alliander.com>
@TonyXiang8787 TonyXiang8787 marked this pull request as ready for review January 21, 2025 14:30
@TonyXiang8787 TonyXiang8787 marked this pull request as draft January 21, 2025 14:31
@TonyXiang8787
fix other lambda issues
8540595 
Signed-off-by: Tony Xiang <tony.xiang@alliander.com>
@TonyXiang8787 TonyXiang8787 marked this pull request as ready for review January 21, 2025 14:52
@TonyXiang8787
generic include all
3ef4445 
Signed-off-by: Tony Xiang <tony.xiang@alliander.com>
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@TonyXiang8787 TonyXiang8787 added this pull request to the merge queue Jan 21, 2025
Merged via the queue into main with commit a4be26e Jan 21, 2025
28 checks passed
@TonyXiang8787 TonyXiang8787 deleted the feature/use-lto-for-ci-release branch January 21, 2025 15:41
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo changed the title use link time optimization for CI release build Use link time optimization for CI release build May 13, 2025
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo added the dependencies Pull requests that update a dependency file label May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mgovers mgovers mgovers approved these changes

Assignees

@TonyXiang8787 TonyXiang8787

Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TonyXiang8787 @mgovers @Jerry-Jinfeng-Guo