diff options
| author | Jason Molenda <jmolenda@apple.com> | 2014-10-17 01:52:30 +0000 |
|---|---|---|
| committer | Jason Molenda <jmolenda@apple.com> | 2014-10-17 01:52:30 +0000 |
| commit | 84843ed5364bf456d0e7fdd10f4e6e743c3f88bd (patch) | |
| tree | 677eb558be01b1f1fb32fba63aa39c5ccf5cc9c2 /lldb/source/Plugins/Process/Utility/InstructionUtils.h | |
| parent | b66130209bdf7ffe62811a137ecb916926054049 (diff) | |
| download | bcm5719-llvm-84843ed5364bf456d0e7fdd10f4e6e743c3f88bd.tar.gz bcm5719-llvm-84843ed5364bf456d0e7fdd10f4e6e743c3f88bd.zip | |
A << operation would be undefined for a bit-selecting
function because of a '1u' making it a 32-bit value
when it really needed to be a 64-bit value. Trivial to fix
once I figured out what was going on.
clang static analzyer fixit.
llvm-svn: 220022
Diffstat (limited to 'lldb/source/Plugins/Process/Utility/InstructionUtils.h')
| -rw-r--r-- | lldb/source/Plugins/Process/Utility/InstructionUtils.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lldb/source/Plugins/Process/Utility/InstructionUtils.h b/lldb/source/Plugins/Process/Utility/InstructionUtils.h index 813990095c4..6226fbc04b0 100644 --- a/lldb/source/Plugins/Process/Utility/InstructionUtils.h +++ b/lldb/source/Plugins/Process/Utility/InstructionUtils.h @@ -20,7 +20,7 @@ static inline uint64_t Bits64 (const uint64_t bits, const uint32_t msbit, const uint32_t lsbit) { assert(msbit < 64 && lsbit <= msbit); - return (bits >> lsbit) & ((1u << (msbit - lsbit + 1)) - 1); + return (bits >> lsbit) & ((1ull << (msbit - lsbit + 1)) - 1); } // Return the bit field(s) from the most significant bit (msbit) to the |
