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

Eli Bendersky eliben at google.com
Wed Feb 20 15:36:20 PST 2013


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



More information about the llvm-commits mailing list