cryptozeny / sugarchain-v0.16.3

OUTDATED! This repo is moved to:
https://github.com/sugarchain-project/sugarchain
MIT License
0 stars 1 forks source link

Sugarchain: fix: Yespower Makefile #50

Closed cryptozeny closed 5 years ago

cryptozeny commented 5 years ago

changes:

TODO:

On Thu, Feb 28, 2019 at 01:23:25AM +0900, cryptozeny cryptozeny wrote:
> "-msse2" option is already implied on x86-64? So 64-bit build doesn't need
> that CFLAGS?

Correct.

> Please check below list
> 
> ===
> ARM (no such function)
> -Wall -O2 -fomit-frame-pointer
> 
> Win32 (need -msse2)
> -Wall *-msse2* -O2 -fomit-frame-pointer
> 
> Linux32 (need -msse2)
> -Wall *-msse2* -O2 -fomit-frame-pointer
> 
> Win64 (already implied so not needed)
> -Wall -O2 -fomit-frame-pointer
> 
> Linux64 (already implied so not needed)
> -Wall -O2 -fomit-frame-pointer
> 
> macOS (already implied so not needed)
> -Wall -O2 -fomit-frame-pointer
> ===
> 
> Am I right?

This looks OK to me.

Alexander
cryptozeny commented 5 years ago

Tested ACK