Skip to content

Commit

Permalink
Fixing a -Wsign-compare warning; NFC.
Browse files Browse the repository at this point in the history
I think it might make sense to make COFF::MaxNumberOfSections16 be a uint32_t, however, that may have wider-reaching implications in other projects, which is why I did not change that declaration.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@220384 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
AaronBallman committed Oct 22, 2014
1 parent 418d0cf commit df8b8e7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tools/yaml2obj/yaml2coff.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,8 @@ struct COFFParser {
}

bool useBigObj() const {
return Obj.Sections.size() > COFF::MaxNumberOfSections16;
return static_cast<int32_t>(Obj.Sections.size()) >
COFF::MaxNumberOfSections16;
}

unsigned getHeaderSize() const {
Expand Down

0 comments on commit df8b8e7

Please sign in to comment.