[lldb-dev] Possible bug in DataExtractor::GetMaxU64Bitfield

Ted Woodward via lldb-dev lldb-dev at lists.llvm.org
Mon Nov 25 09:49:06 PST 2019

Here's an interesting bug, in source/Utility/DataExtractor.cpp, DataExtractor::GetMaxU64Bitfield:

    uint64_t bitfield_mask = ((1ul << bitfield_bit_size) - 1);

In Visual Studio 2017, ul is a 32 bit type. When I run it with a size of 36 (from large_packed in test/API/lang/c/bitfields/main.c), I get a mask of 0x0f.
I think that should be 1ull, or (uint64_t) 1, not 1ul.

MSDN problem talking about the size of ul:

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20191125/79f082d5/attachment.html>

More information about the lldb-dev mailing list