<div dir="ltr"><span style="font-family:monospace,monospace">Hello Zachary,<br><br>It looks like this commit added an error to one of our builders:<br><a href="http://lab.llvm.org:8011/builders/ubuntu-gcc7.1-werror/builds/3735">http://lab.llvm.org:8011/builders/ubuntu-gcc7.1-werror/builds/3735</a><br><br>. . .<br>FAILED: /usr/local/gcc-7.1/bin/g++-7.1 -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Ilib/Object -I/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/lib/Object -Iinclude -I/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/include -Wno-noexcept-type -fPIC -fvisibility-inlines-hidden -Werror -Werror=date-time -std=c++11 -Wall -W -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wno-missing-field-initializers -pedantic -Wno-long-long -Wno-maybe-uninitialized -Wdelete-non-virtual-dtor -Wno-comment -ffunction-sections -fdata-sections -O3 -fPIC -UNDEBUG -fno-exceptions -fno-rtti -MD -MT lib/Object/CMakeFiles/LLVMObject.dir/COFFImportFile.cpp.o -MF lib/Object/CMakeFiles/LLVMObject.dir/COFFImportFile.cpp.o.d -o lib/Object/CMakeFiles/LLVMObject.dir/COFFImportFile.cpp.o -c /home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/lib/Object/COFFImportFile.cpp<br>/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/lib/Object/COFFImportFile.cpp: In member function ‘llvm::NewArchiveMember llvm::object::{anonymous}::ObjectFactory::createImportDescriptor(std::vector<unsigned char, std::allocator<unsigned char> >&)’:<br>/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/lib/Object/COFFImportFile.cpp:191:28: error: enumeral and non-enumeral type in conditional expression [-Werror=extra]<br> u16(is32bit(Machine) ? IMAGE_FILE_32BIT_MACHINE : 0),<br> ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/lib/Object/COFFImportFile.cpp: In member function ‘llvm::NewArchiveMember llvm::object::{anonymous}::ObjectFactory::createNullImportDescriptor(std::vector<unsigned char, std::allocator<unsigned char> >&)’:<br>/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/lib/Object/COFFImportFile.cpp:327:28: error: enumeral and non-enumeral type in conditional expression [-Werror=extra]<br> u16(is32bit(Machine) ? IMAGE_FILE_32BIT_MACHINE : 0),<br> ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/lib/Object/COFFImportFile.cpp: In member function ‘llvm::NewArchiveMember llvm::object::{anonymous}::ObjectFactory::createNullThunk(std::vector<unsigned char, std::allocator<unsigned char> >&)’:<br>/home/buildslave/am1i-slv2/ubuntu-gcc7.1-werror/llvm/lib/Object/COFFImportFile.cpp:390:28: error: enumeral and non-enumeral type in conditional expression [-Werror=extra]<br> u16(is32bit(Machine) ? IMAGE_FILE_32BIT_MACHINE : 0),<br> ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>cc1plus: all warnings being treated as errors<br><br>The builder was already red and did not send notifications.<br>Please have a look?<br><br>Thanks<br><br>Galina<br></span></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 14, 2017 at 2:07 PM, Zachary Turner via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: zturner<br>
Date: Thu Dec 14 14:07:03 2017<br>
New Revision: 320750<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=320750&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=320750&view=rev</a><br>
Log:<br>
Fix many -Wsign-compare and -Wtautological-constant-<wbr>compare warnings.<br>
<br>
Most of the -Wsign-compare warnings are due to the fact that<br>
enums are signed by default in the MS ABI, while the<br>
tautological comparison warnings trigger on x86 builds where<br>
sizeof(size_t) is 4 bytes, so N > numeric_limits<unsigned>::max(<wbr>)<br>
is always false.<br>
<br>
Differential Revision: <a href="https://reviews.llvm.org/D41256" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D41256</a><br>
<br>
Modified:<br>
cfe/trunk/lib/CodeGen/CGExpr.<wbr>cpp<br>
<br>
Modified: cfe/trunk/lib/CodeGen/CGExpr.<wbr>cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGExpr.cpp?rev=320750&r1=320749&r2=320750&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/cfe/trunk/lib/CodeGen/<wbr>CGExpr.cpp?rev=320750&r1=<wbr>320749&r2=320750&view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- cfe/trunk/lib/CodeGen/CGExpr.<wbr>cpp (original)<br>
+++ cfe/trunk/lib/CodeGen/CGExpr.<wbr>cpp Thu Dec 14 14:07:03 2017<br>
@@ -2825,7 +2825,7 @@ void CodeGenFunction::EmitCheck(<br>
assert(IsSanitizerScope);<br>
assert(Checked.size() > 0);<br>
assert(CheckHandler >= 0 &&<br>
- CheckHandler < sizeof(SanitizerHandlers) / sizeof(*SanitizerHandlers));<br>
+ size_t(CheckHandler) < llvm::array_lengthof(<wbr>SanitizerHandlers));<br>
const StringRef CheckName = SanitizerHandlers[<wbr>CheckHandler].Name;<br>
<br>
llvm::Value *FatalCond = nullptr;<br>
<br>
<br>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>