diff options
| author | Kevin Enderby <enderby@apple.com> | 2010-10-04 20:17:24 +0000 |
|---|---|---|
| committer | Kevin Enderby <enderby@apple.com> | 2010-10-04 20:17:24 +0000 |
| commit | 3f55c24df9527de345f6cc960944840a7a101c6a (patch) | |
| tree | 30f8fb4b5322183a620261d679d41c49c69e2993 /lib/VMCore/AutoUpgrade.cpp | |
| parent | f0233c6cd9e3f532f0611162c42b382d7a404dfe (diff) | |
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
Diffstat (limited to 'lib/VMCore/AutoUpgrade.cpp')
0 files changed, 0 insertions, 0 deletions