Skip to content

Conversation

greenaddress
Copy link

No description provided.

@jkauffman1
Copy link
Contributor

utack 0d06449

@NickeZ
Copy link
Contributor

NickeZ commented Mar 30, 2020

nack

$ CFLAGS="-Os" ../configure
...
  CFLAGS                  = -Os -W -std=c89 -pedantic -Wall -Wextra -Wcast-align -Wnested-externs -Wshadow -Wstrict-prototypes -Wno-unused-function -Wno-long-long -Wno-overlength-strings -fvisibility=hidden -O2
  CPPFLAGS                =
  LDFLAGS                 =  -O2 -Wl,-z,relro

It still looks like -O2 is added at the end, which will override -Os.

@greenaddress
Copy link
Author

greenaddress commented Mar 30, 2020

@NickeZ fair enough & good catch thanks

Better now?

@NickeZ
Copy link
Contributor

NickeZ commented Mar 30, 2020

tACK, seems correct.

@greenaddress
Copy link
Author

for reference bitcoin-core/secp256k1#700 (comment)

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.

3 participants