diff options
| author | Alexey Samsonov <vonosmas@gmail.com> | 2014-08-20 22:46:38 +0000 |
|---|---|---|
| committer | Alexey Samsonov <vonosmas@gmail.com> | 2014-08-20 22:46:38 +0000 |
| commit | e5864c69a8a247f07b88b99cccc1032129cc27ac (patch) | |
| tree | 8d5de85d600eb0812320fdff67e83b8c7701f05a /llvm/lib/MC/MCStreamer.cpp | |
| parent | 50de57df6d0a706ea5451e1ccf8878d8298ae9c7 (diff) | |
| download | bcm5719-llvm-e5864c69a8a247f07b88b99cccc1032129cc27ac.tar.gz bcm5719-llvm-e5864c69a8a247f07b88b99cccc1032129cc27ac.zip | |
Don't allow MCStreamer::EmitIntValue to output 0-byte integers.
It makes no sense and can hide bugs. In particular, it lead
to left shift by 64 bits, which is an undefined behavior,
properly reported by UBSan.
llvm-svn: 216134
Diffstat (limited to 'llvm/lib/MC/MCStreamer.cpp')
| -rw-r--r-- | llvm/lib/MC/MCStreamer.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/MC/MCStreamer.cpp b/llvm/lib/MC/MCStreamer.cpp index e7ee9e481bf..914e3372670 100644 --- a/llvm/lib/MC/MCStreamer.cpp +++ b/llvm/lib/MC/MCStreamer.cpp @@ -72,7 +72,7 @@ void MCStreamer::generateCompactUnwindEncodings(MCAsmBackend *MAB) { /// EmitIntValue - Special case of EmitValue that avoids the client having to /// pass in a MCExpr for constant integers. void MCStreamer::EmitIntValue(uint64_t Value, unsigned Size) { - assert(Size <= 8 && "Invalid size"); + assert(1 <= Size && Size <= 8 && "Invalid size"); assert((isUIntN(8 * Size, Value) || isIntN(8 * Size, Value)) && "Invalid size"); char buf[8]; |
