Skip to content
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.

Sign up for GitHub

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

Allow overriding default flags #700

Merged
merged 4 commits into from
Mar 20, 2020
Merged

Allow overriding default flags #700

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
613c34c
Remove test in configure.ac because it doesn't have an effect
jonasnick Dec 17, 2019
ecba813
Append instead of Prepend user-CFLAGS to default CFLAGS allowing the …
jonasnick Dec 17, 2019
83fb1bc
Remove -O2 from default CFLAGS because this would override the -O3 fl…
jonasnick Dec 17, 2019
ca739cb
Compile with optimization flag -O2 by default instead of -O3
jonasnick Feb 19, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions configure.ac
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ AH_TOP([#ifndef LIBSECP256K1_CONFIG_H])
AH_TOP([#define LIBSECP256K1_CONFIG_H])
AH_BOTTOM([#endif /*LIBSECP256K1_CONFIG_H*/])
AM_INIT_AUTOMAKE([foreign subdir-objects])

# Set -g (but not -O2 because this would override -O3 which we're adding later)
# if CFLAGS are not already set (see PROG_CC in the Autoconf manual)
: ${CFLAGS="-g"}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couldn't we get rid of this entirely then because Autoconf's default is -g -O2?

If your answer is, maybe keep this variant still as a branch somewhere until we have further Concept ACKs on the idea of switching to -O2 :p

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we remove this then if $CFLAGS aren't already set they will be set to -O2 ... -O2. The comment additionally serves as a hint to how autoconf works :) because I'd rather set : ${CFLAGS=""} in the future, because anything else doesn't make much sense to me.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay yes. Also, autoconf sets -O2 apparently only for gcc and not for other compilers according to its manual... So we need to keep the -O2 further below anyway.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

83fb1bc

This causes multiple rm: conftest.dSYM: is a directory warnings on macOS (#896).

LT_INIT

dnl make the compilation flags quiet unless V=1 is used
Expand Down