From 49978e61250a7229ad1040b002a609858f158584 Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Wed, 16 Oct 2013 18:26:16 +0000 Subject: Fix MCDataAtom never calling remap when adding data. This patch fixes a small mistake in MCDataAtom::addData() where it doesn't ever call remap(): - if (Data.size() > Begin - End - 1) + if (Data.size() > End + 1 - Begin) remap(Begin, End + 1); This is currently not visible because of another bug is the disassembler, so the patch includes a unit test. Patch by Stephen Checkoway. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@192823 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/MC/MCAtom.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/MC') diff --git a/lib/MC/MCAtom.cpp b/lib/MC/MCAtom.cpp index 651028950e..d0a64c3008 100644 --- a/lib/MC/MCAtom.cpp +++ b/lib/MC/MCAtom.cpp @@ -44,7 +44,7 @@ void MCAtom::remapForSplit(uint64_t SplitPt, void MCDataAtom::addData(const MCData &D) { Data.push_back(D); - if (Data.size() > Begin - End - 1) + if (Data.size() > End + 1 - Begin) remap(Begin, End + 1); } -- cgit v1.2.3