Incorporate suggestions by Daniel Dunbar after his review. Thanks Daniel!
authorKevin Enderby <enderby@apple.com>
Mon, 4 Oct 2010 20:17:24 +0000 (20:17 +0000)
committerKevin Enderby <enderby@apple.com>
Mon, 4 Oct 2010 20:17:24 +0000 (20:17 +0000)
commit3f55c24df9527de345f6cc960944840a7a101c6a
tree30f8fb4b5322183a620261d679d41c49c69e2993
parentf0233c6cd9e3f532f0611162c42b382d7a404dfe
Incorporate suggestions by Daniel Dunbar after his review.  Thanks Daniel!

1) Changed ValidateDwarfFileNumber() to isValidDwarfFileNumber() to be better
   named.  Since it is just a predicate and isn't actually changing any state.

2) Added a missing return in the comments for setCurrentDwarfLoc() in
   include/llvm/MC/MCContext.h for fix formatting.

3) Changed clearDwarfLocSeen() to ClearDwarfLocSeen() since it does change
   state.

4) Simplified the last test in isValidDwarfFileNumber() to just a one line
   boolean test of MCDwarfFiles[FileNumber] != 0 for the final return statement.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@115551 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/MC/MCContext.h
lib/MC/MCContext.cpp
lib/MC/MCDwarf.cpp
lib/MC/MCParser/AsmParser.cpp