diff options
author | Jordan Rose <jordan_rose@apple.com> | 2013-09-11 16:46:50 +0000 |
---|---|---|
committer | Jordan Rose <jordan_rose@apple.com> | 2013-09-11 16:46:50 +0000 |
commit | 9519ff59ec0accfeee3bc43ce5c01dd7cbd94850 (patch) | |
tree | d00500b7ef1b8cb79d3841d36e1dbe9ff0f63b3b /llvm/lib/Bitcode/Reader/BitcodeReader.h | |
parent | d9ad0681fad9a98f43d9baddb95d505b37153c48 (diff) | |
download | bcm5719-llvm-9519ff59ec0accfeee3bc43ce5c01dd7cbd94850.tar.gz bcm5719-llvm-9519ff59ec0accfeee3bc43ce5c01dd7cbd94850.zip |
[analyzer] Handle zeroing constructors for fields of structs with empty bases.
RegionStore tries to protect against accidentally initializing the same
region twice, but it doesn't take subregions into account very well. If
the outer region being initialized is a struct with an empty base class,
the offset of the first field in the struct will be 0. When we initialize
the base class, we may invalidate the contents of the struct by providing
a default value of Unknown (or some new symbol). We then go to initialize
the member with a zeroing constructor, only to find that the region at
that offset in the struct already has a value. The best we can do here is
to invalidate that value and continue; neither the old default value nor
the new 0 is correct for the entire struct after the member constructor call.
The correct solution for this is to track region extents in the store.
<rdar://problem/14914316>
llvm-svn: 190530
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.h')
0 files changed, 0 insertions, 0 deletions