[PATCH] D34279: Fix release_40 build with MSVC (VS 2015)

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 19 08:17:17 PDT 2017


hans added a comment.

I have 19.00.24210, so slightly earlier I suppose, but I believe we use Update 3 on our Chromium buildbots, and they seem happy.

Can you paste the full error message? The part I see in your screenshot doesn't really make sense. Why should converting 1 to unsigned int be a narrowing conversion? I'm just trying to understand why we need this.


https://reviews.llvm.org/D34279





More information about the cfe-commits mailing list