This is an archive of the discontinued LLVM Phabricator instance.

Fix a comparison of integers of different signs warning
ClosedPublic

Authored by ttsugrii on Nov 23 2016, 4:16 PM.

Details

Summary
source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp:403:21: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') [-Wsign-compare]
for (int i = 0; i < llvm::array_lengthof (magicks); i++)
                ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Diff Detail

Repository
rL LLVM

Event Timeline

ttsugrii updated this revision to Diff 79173.Nov 23 2016, 4:16 PM
ttsugrii retitled this revision from to Fix a comparison of integers of different signs warning.
ttsugrii updated this object.
ttsugrii added a project: Restricted Project.
ttsugrii added a reviewer: jasonmolenda.
inglorion accepted this revision.Nov 23 2016, 4:56 PM
inglorion added a reviewer: inglorion.
inglorion added a subscriber: inglorion.

lgtm

This revision is now accepted and ready to land.Nov 23 2016, 4:56 PM
This revision was automatically updated to reflect the committed changes.