diff options
| author | James Y Knight <jyknight@google.com> | 2015-05-09 03:13:37 +0000 |
|---|---|---|
| committer | James Y Knight <jyknight@google.com> | 2015-05-09 03:13:37 +0000 |
| commit | fca02be3c1059fd513c2e912411a8bcf284a329a (patch) | |
| tree | c3be6c59dce2c6b43136f612307277af9d948dcf /clang/lib/CodeGen/CodeGenModule.h | |
| parent | 71d19f3cd62a0b6c8c5bd4b8023e4533d44a05a2 (diff) | |
| download | llvm-fca02be3c1059fd513c2e912411a8bcf284a329a.zip llvm-fca02be3c1059fd513c2e912411a8bcf284a329a.tar.gz llvm-fca02be3c1059fd513c2e912411a8bcf284a329a.tar.bz2 | |
Fix MergeConsecutiveStore for non-byte-sized memory accesses.
The bug showed up as a compile-time assertion failure:
Assertion `NumBits >= MIN_INT_BITS && "bitwidth too small"' failed
when building msan tests on x86-64.
Prior to r236850, this bug was masked due to a bogus alignment check,
which also accidentally rejected non-byte-sized accesses. Afterwards,
an invalid ElementSizeBytes == 0 got further into the function, and
triggered the assertion failure.
It would probably be a good idea to allow it to handle merging stores
of unusual widths as well, but for now, to un-break it, I'm just
making the minimal fix.
Differential Revision: http://reviews.llvm.org/D9626
llvm-svn: 236927
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.h')
0 files changed, 0 insertions, 0 deletions