diff options
author | Zachary Turner <zturner@google.com> | 2017-12-14 22:07:03 +0000 |
---|---|---|
committer | Zachary Turner <zturner@google.com> | 2017-12-14 22:07:03 +0000 |
commit | 260fe3eca6903f23fd017bafb7e555f4be8acf11 (patch) | |
tree | ef63acf019f5de6c362d767ff1fd8165f083f59d /llvm/lib/Support/ARMAttributeParser.cpp | |
parent | 0ab0c1a201ece292fdced1f913fa257bdcb5280f (diff) | |
download | bcm5719-llvm-260fe3eca6903f23fd017bafb7e555f4be8acf11.tar.gz bcm5719-llvm-260fe3eca6903f23fd017bafb7e555f4be8acf11.zip |
Fix many -Wsign-compare and -Wtautological-constant-compare warnings.
Most of the -Wsign-compare warnings are due to the fact that
enums are signed by default in the MS ABI, while the
tautological comparison warnings trigger on x86 builds where
sizeof(size_t) is 4 bytes, so N > numeric_limits<unsigned>::max()
is always false.
Differential Revision: https://reviews.llvm.org/D41256
llvm-svn: 320750
Diffstat (limited to 'llvm/lib/Support/ARMAttributeParser.cpp')
-rw-r--r-- | llvm/lib/Support/ARMAttributeParser.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Support/ARMAttributeParser.cpp b/llvm/lib/Support/ARMAttributeParser.cpp index a9a0c1d1a4d..3d800eb7a96 100644 --- a/llvm/lib/Support/ARMAttributeParser.cpp +++ b/llvm/lib/Support/ARMAttributeParser.cpp @@ -592,7 +592,7 @@ void ARMAttributeParser::ParseAttributeList(const uint8_t *Data, bool Handled = false; for (unsigned AHI = 0, AHE = array_lengthof(DisplayRoutines); AHI != AHE && !Handled; ++AHI) { - if (DisplayRoutines[AHI].Attribute == Tag) { + if (uint64_t(DisplayRoutines[AHI].Attribute) == Tag) { (this->*DisplayRoutines[AHI].Routine)(ARMBuildAttrs::AttrType(Tag), Data, Offset); Handled = true; |