Skip to content
This repository has been archived by the owner on Dec 17, 2024. It is now read-only.

32-bit platform support #2

Closed
arkpar opened this issue Mar 10, 2015 · 1 comment · Fixed by #6
Closed

32-bit platform support #2

arkpar opened this issue Mar 10, 2015 · 1 comment · Fixed by #6

Comments

@arkpar
Copy link

arkpar commented Mar 10, 2015

dag_sizes in C implementation overflows with 32-bit size_t type. There might be other issues as well, see cpp-ethereum issue

@jorisbontje
Copy link

getting the following build warnings:

# github.com/ethereum/ethash
In file included from Godeps/_workspace/src/github.com/ethereum/ethash/src/libethash/internal.c:30:0,
                 from Godeps/_workspace/src/github.com/ethereum/ethash/ethash.go:16:
Godeps/_workspace/src/github.com/ethereum/ethash/src/libethash/data_sizes.h:129:9: warning: large integer implicitly truncated to unsigned type [-Woverflow]
         4303349632U, 4311743104U, 4320130432U, 4328521088U, 4336909184U,
         ^
Godeps/_workspace/src/github.com/ethereum/ethash/src/libethash/data_sizes.h:129:9: warning: large integer implicitly truncated to unsigned type [-Woverflow]
Godeps/_workspace/src/github.com/ethereum/ethash/src/libethash/data_sizes.h:129:9: warning: large integer implicitly truncated to unsigned type [-Woverflow]
Godeps/_workspace/src/github.com/ethereum/ethash/src/libethash/data_sizes.h:129:9: warning: large integer implicitly truncated to unsigned type [-Woverflow]
Godeps/_workspace/src/github.com/ethereum/ethash/src/libethash/data_sizes.h:129:9: warning: large integer implicitly truncated to unsigned type [-Woverflow]
Godeps/_workspace/src/github.com/ethereum/ethash/src/libethash/data_sizes.h:130:9: warning: large integer implicitly truncated to unsigned type [-Woverflow]
         4345295488U, 4353687424U, 4362073472U, 4370458496U, 4378852736U,

mkrufky added a commit to mkrufky/ethereum-ethash that referenced this issue Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants