Redefine error in "accumulator.h" (commit c52868e)

jhonnold / berserk

UCI Chess Engine written in C

GNU General Public License v3.0

200 stars 30 forks source link

Redefine error in "accumulator.h" (commit c52868e) #479

Closed Victor-Ch closed 1 year ago

Victor-Ch commented 1 year ago

In header accumulator.h:

43 #define UNROLL 128 44 #define UNROLL 16

Obviously it should be:

43 #define UNROLL 128 44 #define NUM_REGS 16

jherrera80 commented 1 year ago

Also, changes in dad64e04d0392b906a6cd7a5f48aac8082c1b159 seem to have been lost.

jhonnold commented 1 year ago

Thank you for this catch Victor. If you'd like to submit a PR for this fix, I'd gladly accept it.

@jherrera80 They're lost as the binary cannot support non-vectorized code since this commit. I will work my way back to it, but it is not a top priority to support those between major releases.

jhonnold commented 1 year ago

Closing as this was fixed in #482