-
Notifications
You must be signed in to change notification settings - Fork 44
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
Feat. req. Оптимизировать флаги компиляции #89
Comments
А что конкретно вам из этого нужно? Какие-то видюшки тормозят или что-то другое тормозит? |
@deemru Есессно на ваше усмотрение, но выше в форке произведена общая оптимизация кода компилятором + использование AVX, AVX2 инструкций и т.д. (No Sync, compiler clang (+lld_linker +thinlto +pgo +avx2 +fma +compiler_optimizations +linker_optimizations) |
Основной аргумент: непонятно, на что это повлияло -- никаких метрик было/стало не приведено. Также, использование продвинутых процессорных инструкций ведёт к отказу работы на процессорах (а также виртуализированных процессорах), где они не поддерживаются. Текущими более менее общими считаются инструкции x86-64-v2: То есть, необходимо выпускать на каждую оптимизированную сборку ещё одну, для всех остальных, но при этом смотри основной аргумент выше. |
Имеем два результата с разницей в 11 месяцев, 427 и 570 единиц. По тексту непонятно, что речь идёт об одной и той же версии Chromium, которая отличается только флагами оптимизации (например, такой прирост могли дать улучшения в v8 за эти же 11 месяцев), но даже если примем эти цифры как абсолютно чистый тест, прирост в ~25% это не столь существенно для отказа от старых архитектур: #89 (comment) |
@deemru , @crypto-das , @aitsvet
https://github.com/RobRich999/Chromium_Clang
The text was updated successfully, but these errors were encountered: