[llvm] r175687 - as the allocator is reset zero out the number of bytes allocated, this was just

[Pedro Artigas]

The allocator is pretty well tested by the compiler. How would I add a test to verify this particular fix?

Pedro

On Feb 20, 2013, at 3:36 PM, Eli Bendersky <eliben at google.com> wrote:

> The allocator has unit tests in unittests/Support/AllocatorTest.cpp -
> can you add a test that exercises this?
> 
> Eli
> 
> 
> On Wed, Feb 20, 2013 at 3:30 PM, Pedro Artigas <partigas at apple.com> wrote:
>> Author: partigas
>> Date: Wed Feb 20 17:30:56 2013
>> New Revision: 175687
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=175687&view=rev
>> Log:
>> as the allocator is reset zero out the number of bytes allocated, this was just
>> missed before but probably what was intended.
>> 
>> Modified:
>>    llvm/trunk/lib/Support/Allocator.cpp
>> 
>> Modified: llvm/trunk/lib/Support/Allocator.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Allocator.cpp?rev=175687&r1=175686&r2=175687&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Support/Allocator.cpp (original)
>> +++ llvm/trunk/lib/Support/Allocator.cpp Wed Feb 20 17:30:56 2013
>> @@ -83,6 +83,7 @@ void BumpPtrAllocator::Reset() {
>>   CurSlab->NextPtr = 0;
>>   CurPtr = (char*)(CurSlab + 1);
>>   End = ((char*)CurSlab) + CurSlab->Size;
>> +  BytesAllocated = 0;
>> }
>> 
>> /// Allocate - Allocate space at the specified alignment.
>> 
>> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits