{"action":"created","issue":{"url":"https://api.github.com/repos/yaneurao/YaneuraOu/issues/50","repository_url":"https://api.github.com/repos/yaneurao/YaneuraOu","labels_url":"https://api.github.com/repos/yaneurao/YaneuraOu/issues/50/labels{/name}","comments_url":"https://api.github.com/repos/yaneurao/YaneuraOu/issues/50/comments","events_url":"https://api.github.com/repos/yaneurao/YaneuraOu/issues/50/events","html_url":"https://github.com/yaneurao/YaneuraOu/pull/50","id":241126891,"number":50,"title":"GCC 7.1.0 での警告に対する修正","user":{"login":"mizar","id":68691,"avatar_url":"https://avatars0.githubusercontent.com/u/68691?v=3","gravatar_id":"","url":"https://api.github.com/users/mizar","html_url":"https://github.com/mizar","followers_url":"https://api.github.com/users/mizar/followers","following_url":"https://api.github.com/users/mizar/following{/other_user}","gists_url":"https://api.github.com/users/mizar/gists{/gist_id}","starred_url":"https://api.github.com/users/mizar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mizar/subscriptions","organizations_url":"https://api.github.com/users/mizar/orgs","repos_url":"https://api.github.com/users/mizar/repos","events_url":"https://api.github.com/users/mizar/events{/privacy}","received_events_url":"https://api.github.com/users/mizar/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-07-07T00:51:12Z","updated_at":"2017-07-07T12:52:32Z","closed_at":"2017-07-07T06:08:51Z","pull_request":{"url":"https://api.github.com/repos/yaneurao/YaneuraOu/pulls/50","html_url":"https://github.com/yaneurao/YaneuraOu/pull/50","diff_url":"https://github.com/yaneurao/YaneuraOu/pull/50.diff","patch_url":"https://github.com/yaneurao/YaneuraOu/pull/50.patch"},"body":"`gcc version 7.1.0 (Rev2, Built by MSYS2 project)` にて以下のような警告が発生しています。\r\n\r\n```\r\nbitboard.h:37:77: warning: '*((void*)(&<anonymous>)+16).Bitboard::<anonymous>.Bitboard::<unnamed union>::m' may be used uninitialized in this function [-Wmaybe-uninitialized]\r\n  Bitboard& operator = (const Bitboard& rhs) { _mm_store_si128(&this->m, rhs.m); return *this; }\r\n                                                                         ~~~~^\r\neval/kppt_evalsum.h:59:33: warning: '*((void*)(&<anonymous>)+384).Eval::EvalSum::<anonymous>.Eval::EvalSum::<unnamed union>::mm' may be used uninitialized in this function [-Wmaybe-uninitialized]\r\n     _mm256_store_si256(&mm, rhs.mm);\r\n                             ~~~~^~\r\neval/kppt_evalsum.h:68:35: warning: '*((void*)(&<anonymous>)+384).Eval::EvalSum::<anonymous>.Eval::EvalSum::<unnamed union>::m[0]' may be used uninitialized in this function [-Wmaybe-uninitialized]\r\n     _mm_store_si128(&m[0], rhs.m[0]);\r\n                            ~~~~~~~^\r\neval/kppt_evalsum.h:69:35: warning: '*((void*)(&<anonymous>)+384).Eval::EvalSum::<anonymous>.Eval::EvalSum::<unnamed union>::m[1]' may be used uninitialized in this function [-Wmaybe-uninitialized]\r\n     _mm_store_si128(&m[1], rhs.m[1]);\r\n                            ~~~~~~~^\r\n```\r\n\r\n```\r\nextra/mate/mate1ply_without_effect.cpp: In function 'Move is_mate_in_1ply_imp(const Position&) [with Color Us = (Color)0]':\r\nextra/mate/../../extra/config.h:412:30: warning: this statement may fall through [-Wimplicit-fallthrough=]\r\nextra/mate/mate1ply_without_effect.cpp: In function 'Move is_mate_in_1ply_imp(const Position&) [with Color Us = (Color)1]':\r\nextra/mate/../../extra/config.h:412:30: warning: this statement may fall through [-Wimplicit-fallthrough=]\r\n #define ASSERT_LV_EX(L, X) { if (L <= ASSERT_LV) ASSERT(X); }\r\n                              ^\r\nextra/mate/../../extra/config.h:415:23: note: in expansion of macro 'ASSERT_LV_EX'\r\n #define ASSERT_LV3(X) ASSERT_LV_EX(3, X)\r\n                       ^~~~~~~~~~~~\r\nextra/mate/mate1ply_without_effect.cpp:1625:4: note: in expansion of macro 'ASSERT_LV3'\r\n    ASSERT_LV3(false); // こっちくんな\r\n    ^~~~~~~~~~\r\nextra/mate/mate1ply_without_effect.cpp:1627:4: note: here\r\n    case LANCE:\r\n    ^~~~\r\nextra/mate/mate1ply_without_effect.cpp:1699:5: warning: this statement may fall through [-Wimplicit-fallthrough=]\r\n     pt = GOLD; // 以下の処理でややこしいのでGOLD扱いにしてしまう。\r\n     ^~\r\nextra/mate/mate1ply_without_effect.cpp:1700:4: note: here\r\n    case GOLD:\r\n    ^~~~\r\nIn file included from extra/mate/../../shogi.h:51:0,\r\n                 from extra/mate/mate1ply.h:4,\r\n                 from extra/mate/mate1ply_without_effect.cpp:1:\r\n```"},"comment":{"url":"https://api.github.com/repos/yaneurao/YaneuraOu/issues/comments/313672750","html_url":"https://github.com/yaneurao/YaneuraOu/pull/50#issuecomment-313672750","issue_url":"https://api.github.com/repos/yaneurao/YaneuraOu/issues/50","id":313672750,"user":{"login":"mizar","id":68691,"avatar_url":"https://avatars0.githubusercontent.com/u/68691?v=3","gravatar_id":"","url":"https://api.github.com/users/mizar","html_url":"https://github.com/mizar","followers_url":"https://api.github.com/users/mizar/followers","following_url":"https://api.github.com/users/mizar/following{/other_user}","gists_url":"https://api.github.com/users/mizar/gists{/gist_id}","starred_url":"https://api.github.com/users/mizar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mizar/subscriptions","organizations_url":"https://api.github.com/users/mizar/orgs","repos_url":"https://api.github.com/users/mizar/repos","events_url":"https://api.github.com/users/mizar/events{/privacy}","received_events_url":"https://api.github.com/users/mizar/received_events","type":"User","site_admin":false},"created_at":"2017-07-07T12:52:32Z","updated_at":"2017-07-07T12:52:32Z","body":"マージありがとうございます。\r\n\r\nhttps://github.com/mizar/YaneuraOu/commit/a5544089b55b1811056dc298320bcaa9eb1088e0\r\n\r\nダメ元でデフォルトコンストラクタのみの変更を上記のように試してみましたが、\r\n手元の環境では `Bitboard() {}` を `Bitboard() : m(_mm_setzero_si128()) {}` とすると bench にて 0～1% 程度のNPS低下、 `Bitboard() : p{} {}` とすると 1～4% 程度のNPS低下といった雰囲気でした。 `EvalSum() {}` の方も弄ってみてはみましたが、今のところ測定誤差に紛れて bench が遅くなっているかどうかはデータが取れていません。参考までに。\r\n"}}
{"actor":{"display_login":"mizar"}}