diff options
| author | Adrian Prantl <aprantl@apple.com> | 2020-07-25 08:27:21 -0700 |
|---|---|---|
| committer | Adrian Prantl <aprantl@apple.com> | 2020-07-27 13:26:35 -0700 |
| commit | 113f56fbb80e8d6f705be19f8ae169a3fee2e4f8 (patch) | |
| tree | 0e4bfabd2e0cea77293d8322a4fea2b77df8b3eb /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | 145acacaea1d7fb4ffc055a3e92ee8fee7c58634 (diff) | |
Unify the return value of GetByteSize to an llvm::Optional<uint64_t> (NFC-ish)
This cleanup patch unifies all methods called GetByteSize() in the
ValueObject hierarchy to return an optional, like the methods in
CompilerType do. This means fewer magic 0 values, which could fix bugs
down the road in languages where types can have a size of zero, such
as Swift and C (but not C++).
Differential Revision: https://reviews.llvm.org/D84285
This re-lands the patch with bogus :m_byte_size(0) initalizations removed.
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
0 files changed, 0 insertions, 0 deletions