RESOLVED FIXED 178544
-Wsign-compare triggered by MallocBench 
https://bugs.webkit.org/show_bug.cgi?id=178544
Summary -Wsign-compare triggered by MallocBench
Michael Catanzaro
Reported 2017-10-19 14:20:40 PDT
I see a warning now that MallocBench is enabled for Linux: In file included from ../../PerformanceTests/MallocBench/MallocBench/Interpreter.cpp:28:0: ../../PerformanceTests/MallocBench/MallocBench/Interpreter.cpp: In constructor ‘Interpreter::Interpreter(const char*, bool, bool)’: ../../PerformanceTests/MallocBench/MallocBench/Interpreter.cpp:64:35: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] assert(m_opCount * sizeof(Op) == buf.st_size); ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~ buf.st_size is an off_t, which is signed, so I think we should do static_cast<size_t>(buf.st_size).
Attachments
Patch (1.33 KB, patch)
2017-10-19 14:22 PDT, Michael Catanzaro 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Michael Catanzaro
Comment 1 2017-10-19 14:22:55 PDT
Created attachment 324284 [details] Patch
WebKit Commit Bot
Comment 2 2017-10-19 18:48:00 PDT
Comment on attachment 324284 [details] Patch Clearing flags on attachment: 324284 Committed r223733: <https://trac.webkit.org/changeset/223733>
WebKit Commit Bot
Comment 3 2017-10-19 18:48:02 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 4 2017-11-15 13:01:39 PST
<rdar://problem/35568633>
Note You need to log in before you can comment on or make changes to this bug.