Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 643712 - app-editors/neovim built with CMAKE_BUILD_TYPE=RelWithDebInfo makes neovim unnecessarily slow
Summary: app-editors/neovim built with CMAKE_BUILD_TYPE=RelWithDebInfo makes neovim un...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Vim Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-01-06 15:15 UTC by pa.lacaze
Modified: 2018-01-07 16:37 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description pa.lacaze 2018-01-06 15:15:27 UTC
Neovim felt quite slow compared to vim, it appears that every ebuild in portage from 25 october 2015 on has been passing CMAKE_BUILD_TYPE=RelDebWithInfo to CMake instead of 'Release' or 'Gentoo' build type.

Neovim's CMakeLists.txt overrides RelWithDebInfo compiler optimization flag with -Og whenever available from the usual -O2. Despite the claims of -Og being generating quite optimized binaries, it is sadly not the case in my experience (and even worse, -Og is often slower than -O2 to compile).

In practice, replacing RelDebWithInfo with Release has led to a 3 fold improvement in vim (hot cache) startup time. I expect editing experience to be better too.

Would you consider making the switch in the ebuilds?
Comment 1 Patrice Clement gentoo-dev 2018-01-07 16:35:56 UTC
Sounds good to me. I'm going to put in the change in a second.
Comment 2 Larry the Git Cow gentoo-dev 2018-01-07 16:37:15 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=42d0016301e042830b684755ab94048977043821

commit 42d0016301e042830b684755ab94048977043821
Author:     Patrice Clement <monsieurp@gentoo.org>
AuthorDate: 2018-01-07 16:03:59 +0000
Commit:     Patrice Clement <monsieurp@gentoo.org>
CommitDate: 2018-01-07 16:37:10 +0000

    app-editors/neovim: switch CMAKE_BUILD_TYPE to Release.
    
    Closes: https://bugs.gentoo.org/643712
    Package-Manager: Portage-2.3.13, Repoman-2.3.3

 app-editors/neovim/neovim-0.2.0-r1.ebuild | 4 ++--
 app-editors/neovim/neovim-0.2.2-r1.ebuild | 4 ++--
 app-editors/neovim/neovim-9999.ebuild     | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)