GPU Win Clang Builder broken |
||
Issue descriptionFirst failing build: https://build.chromium.org/p/chromium.gpu.fyi/builders/GPU%20Win%20Clang%20Builder%20%28dbg%29/builds/3530 Suspecting this to be r390494: "Verify that Update 2 of VS 2015 is used to build Chromium" Adding jmadill to see if there is an easy fix by just removing the fields. Adding brucedawson to verify that this is the likely reason for this Windows-only compile failure. Compile failure: FAILED: obj/third_party/angle/src/libANGLE/renderer/d3d/d3d11/libANGLE.TextureStorage11.obj ninja -t msvc -e environment.x86 -- E:\b\build\slave\cache\cipd\goma/gomacc "..\..\third_party/llvm-build/Release+Asserts/bin/clang-cl" -m32 /nologo /showIncludes /FC @obj\third_party\angle\src\libANGLE\renderer\d3d\d3d11\libANGLE.TextureStorage11.obj.rsp /c ..\..\third_party\angle\src\libANGLE\renderer\d3d\d3d11\TextureStorage11.cpp /Foobj\third_party\angle\src\libANGLE\renderer\d3d\d3d11\libANGLE.TextureStorage11.obj /Fdobj\third_party\angle\src\libANGLE.cc.pdb In file included from ..\..\third_party\angle\src\libANGLE\renderer\d3d\d3d11\TextureStorage11.cpp:11: ..\..\third_party\angle\src\libANGLE/renderer/d3d/d3d11/TextureStorage11.h(237,9): error: private field 'mPlaneIndex' is not used [-Werror,-Wunused-private-field] int mPlaneIndex; ^ ..\..\third_party\angle\src\libANGLE/renderer/d3d/d3d11/TextureStorage11.h(241,22): error: private field 'mSwizzleTexture' is not used [-Werror,-Wunused-private-field] ID3D11Texture2D *mSwizzleTexture; ^ ..\..\third_party\angle\src\libANGLE/renderer/d3d/d3d11/TextureStorage11.h(242,29): error: private field 'mSwizzleRenderTarget' is not used [-Werror,-Wunused-private-field] ID3D11RenderTargetView *mSwizzleRenderTarget; ^
,
Apr 29 2016
It's not r390494 - that would trigger a #error and would be very obvious. But, you figured that out already.
,
Apr 29 2016
I removed the fields and resubmitted. I don't have access to a clang builder on Windows to verify so let me or Jamie know if another build error occurs.
,
Apr 29 2016
There is a win_clang builder that is part of the default "git cl try" set, and some other clang options that can be selected manually if desired. So, it *should* be possible to reproduce this without a Windows machine.
,
May 30 2016
|
||
►
Sign in to add a comment |
||
Comment 1 by jmad...@chromium.org
, Apr 29 2016Owner: ewell@google.com