search

Alex313031 / Mercury

Firefox fork with compiler optimizations and patches from Librewolf, Waterfox, and GNU IceCat.
https://thorium.rocks/mercury
Mozilla Public License 2.0
1.22k stars 28 forks source link

more rustflags opts #168

Closed SashaXser closed 7 months ago

SashaXser commented 8 months ago

Added the tune-cpu option as well as target-cpu for macos-arm64. I'll leave it as a draft for now

gz83 commented 7 months ago

Have you done any performance comparison testing? Sometimes adding too many optimization parameters can be counterproductive.

In addition, some of the parameters you added may not be appropriate, and there are also wrong parameters.

SashaXser commented 7 months ago

Have you done any performance comparison testing? Sometimes adding too many optimization parameters can be counterproductive.

I can't run the tests right now, can you run the tests for me? In addition, some of the parameters you added may not be appropriate, and there are also wrong parameters.

mtune is used in Thorium, and tune-cpu requires Rust Nightly, similar changes are presented in https://github.com/Eclipse-Community/r3dfox/pull/42

gz83 commented 7 months ago

Sorry, I don't have any resources right now to test the performance differences

SashaXser commented 7 months ago

If you don't like it, close it. I don't want to do anything else.