• Bill Budge's avatar
    Revert "[Memory] Use OS::Allocate for all OS memory allocations." · 1ea3fd2e
    Bill Budge authored
    This reverts commit 7e78506f.
    
    Reason for revert: Broke Android build on Arm64.
    
    Original change's description:
    > [Memory] Use OS::Allocate for all OS memory allocations.
    > 
    > - Eliminates OS::ReserveRegion and OS::ReserveAlignedRegion.
    > - Changes OS::Allocate to take alignment parameter, reorders parameters
    >   to match page_allocator.
    > - Since the size of memory allocation can be deduced, don't return the
    >   amount of memory allocated.
    > - Changes reservation of aligned address space. Before we would reserve
    >   (size + alignment) rounded up to page size. This is too much, because
    >   maximum misalignment is (alignment - page_size).
    > - On Windows and Cygwin, we release an oversize allocation and
    >   immediately retry at the aligned address in the allocation. If we
    >   lose the address due to a race, we just retry.
    > - Clean up all the calls to OS::Allocate in codegen and tests by adding
    >   helper AllocateSystemPage function (allocation.h) and
    >   AllocateAssemblerBuffer (cctest.h).
    > - Changes 'assm' to 'masm' in some targets for consistency when using
    >   a macro-assembler.
    > 
    > Bug: chromium:756050
    > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    > Change-Id: I306dbe042cc867670fdc935abca29db074b0da71
    > Reviewed-on: https://chromium-review.googlesource.com/749848
    > Commit-Queue: Bill Budge <bbudge@chromium.org>
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Reviewed-by: Hannes Payer <hpayer@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#49235}
    
    TBR=bbudge@chromium.org,hpayer@chromium.org,mlippautz@chromium.org
    
    Change-Id: Ic09de4d63c19746a62e804b1f889817ffaebc330
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:756050
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Reviewed-on: https://chromium-review.googlesource.com/758625Reviewed-by: 's avatarBill Budge <bbudge@chromium.org>
    Commit-Queue: Bill Budge <bbudge@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#49242}
    1ea3fd2e
allocation.cc 5.71 KB