GCC: Warnings with syzygy about stockfish HOT 16 CLOSED

Which compiler?

from stockfish.

I also get that.

Apple LLVM version 6.0 (clang-600.0.56) (based on LLVM 3.5svn)
Target: x86_64-apple-darwin14.0.0
Thread model: posix

from stockfish.

gcc version 4.8.2 (Ubuntu 4.8.2-19ubuntu1)

On Sat, Jan 10, 2015 at 7:05 PM, Gary Linscott [email protected]
wrote:

Which compiler?

—
Reply to this email directly or view it on GitHub
#196 (comment)
.

from stockfish.

I only get that when compiling with debug=yes.
gcc version 4.9.2 (Ubuntu 4.9.2-0ubuntu1~14.04)

from stockfish.

gcc is known to suffer from false positives in this area... especially with -O3

from stockfish.

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56273

from stockfish.

Thanks for the link. Sadly it seems they simply don't care (after a failed
first attempt) to fix it.

On Wed, Jan 14, 2015 at 9:09 AM, Joona Kiiski [email protected]
wrote:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56273

—
Reply to this email directly or view it on GitHub
#196 (comment)
.

from stockfish.

Ignore for SF6.

syzygy implementation will be worked on during SF7 anyway.

from stockfish.

Good news. They have fixed the bug in the gcc5 branch :)

from stockfish.

Hi all,

This just for the record, I have recently installed the latest MinGW and, as reported by Felix on FishCooking also, it seems the issue is not fixed/back again, at least with current Stockfish master, you get the same warning. My version of GCC:
C:>GCC --version
GCC (i686-posix-dwarf-rev0, Built by MinGW-W64 project) 5.3.0
Copyright (C) 2015 Free Software Foundation, Inc.

Eelco

from stockfish.

Still unfixed gcc bug:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59124

from stockfish.

this issue can now be closed.

What about making gcc's -Werror (all warnings are error) part of the makefile now ?

from stockfish.

With Marco's syzygy rewrite and gcc-6.1.0 I no longer get this warning

from stockfish.

Fixed by c0bb0415394179e9c771

from stockfish.

I get these warnings under MSVC w/ the new C++ syzygy implementation.
tbprobe.cpp(536): warning C4267: 'initializing': conversion from 'size_t' to 'uint32_t', possible loss of data
tbprobe.cpp(547): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
tbprobe.cpp(582): warning C4244: '=': conversion from 'uint64_t' to '`anonymous-namespace'::Sym', possible loss of data
tbprobe.cpp(983): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data
tbprobe.cpp(1010): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data

from stockfish.

Thanks for reporting. I knew them also by looking at Travis ci output after
each commit.

I don't plan to fix them because they are wrong (gcc and clang don't report
them) and the fix if is a bunch of casts it is very ugly too.

If you are able to fix them without using casts then it's ok.

On Tuesday, November 8, 2016, mstembera [email protected] wrote:

I get these warnings under MSVC w/ the new C++ syzygy implementation.
tbprobe.cpp(536): warning C4267: 'initializing': conversion from 'size_t'
to 'uint32_t', possible loss of data
tbprobe.cpp(547): warning C4267: 'initializing': conversion from 'size_t'
to 'int', possible loss of data
tbprobe.cpp(582): warning C4244: '=': conversion from 'uint64_t' to
'`anonymous-namespace'::Sym', possible loss of data
tbprobe.cpp(983): warning C4267: '=': conversion from 'size_t' to 'int',
possible loss of data
tbprobe.cpp(1010): warning C4267: 'initializing': conversion from 'size_t'
to 'int', possible loss of data

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
#196 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABDGASjhs1J8vesjNkeKpQtkvVy_an7Zks5q8M1JgaJpZM4DQuIH
.

from stockfish.