Skip to content

Conversation

JelteF
Copy link
Contributor

@JelteF JelteF commented Jun 24, 2020

This doesn't make execution slower for GCC compiles according to
Postgres docs:

--enable-debug

    Compiles all programs and libraries with debugging symbols. This
    means that you can run the programs in a debugger to analyze
    problems. This enlarges the size of the installed executables
    considerably, and on non-GCC compilers it usually also disables
    compiler optimization, causing slowdowns. However, having the
    symbols available is extremely helpful for dealing with any
    problems that might arise. Currently, this option is recommended
    for production installations only if you use GCC. But you should
    always have it on if you are doing development work or running a
    beta version.

The -ggdb and -g3 flags simply tell which type of debug symbols we
want.

This doesn't make execution slower for GCC compiles according to
Postgres docs:

    --enable-debug

        Compiles all programs and libraries with debugging symbols. This
        means that you can run the programs in a debugger to analyze
        problems. This enlarges the size of the installed executables
        considerably, and on non-GCC compilers it usually also disables
        compiler optimization, causing slowdowns. However, having the
        symbols available is extremely helpful for dealing with any
        problems that might arise. Currently, this option is recommended
        for production installations only if you use GCC. But you should
        always have it on if you are doing development work or running a
        beta version.

The `-ggdb` and `-g3` flags simply tell which type of debug symbols we
want.
@JelteF JelteF requested a review from SaitTalhaNisanci June 25, 2020 14:34
@SaitTalhaNisanci
Copy link
Contributor

SaitTalhaNisanci commented Jun 30, 2020

In the README, It would be good to mention that by default debug symbols will be enabled.

I am thinking of testing this PR with #180, the change itself makes sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants