search

kkebo / zyphy

Zyphy is (or will be) a fast web browser engine written in Swift.
MIT License
4 stars 1 forks source link

build: remove CMO flags from library targets #110

Closed kkebo closed 1 week ago

kkebo commented 1 month ago

swift-homomorphic-encryption enables CMO only on executable or benchmark targets, not on library targets. I mimics it.

github-actions[bot] commented 1 month ago

Code Metrics Report

main (579897a) #110 (25fc0fd) +/-
Coverage 85.4% 85.6% +0.1%
Details ``` diff | | main (579897a) | #110 (25fc0fd) | +/- | |-----------|----------------|----------------|-------| + | Coverage | 85.4% | 85.6% | +0.1% | | Files | 9 | 9 | 0 | | Lines | 1595 | 1580 | -15 | - | Covered | 1363 | 1353 | -10 | ```

Reported by octocov

github-actions[bot] commented 1 week ago

Code Metrics Report

main (a656fc0) #110 (38f42d1) +/-
Coverage 85.4% 85.7% +0.3%
Details ``` diff | | main (a656fc0) | #110 (38f42d1) | +/- | |-----------|----------------|----------------|-------| + | Coverage | 85.4% | 85.7% | +0.3% | | Files | 10 | 10 | 0 | | Lines | 1594 | 1582 | -12 | - | Covered | 1362 | 1357 | -5 | ```

Reported by octocov

kkebo commented 1 week ago

If I remove CMO flags from library targets, the benchmark results will be slower. So I decided not to merge this.