[compiler-rt] Revert "[scudo] Apply the min release threshold to the group" (PR #112252)

via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 14 12:53:31 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: None (ChiaHungDuan)

<details>
<summary>Changes</summary>

Reverts llvm/llvm-project#<!-- -->112014

The change didn't update the iterator

---
Full diff: https://github.com/llvm/llvm-project/pull/112252.diff


1 Files Affected:

- (modified) compiler-rt/lib/scudo/standalone/primary64.h (-2) 


``````````diff
diff --git a/compiler-rt/lib/scudo/standalone/primary64.h b/compiler-rt/lib/scudo/standalone/primary64.h
index 1865ed41368e3c..97188a5ac235cc 100644
--- a/compiler-rt/lib/scudo/standalone/primary64.h
+++ b/compiler-rt/lib/scudo/standalone/primary64.h
@@ -1492,8 +1492,6 @@ template <typename Config> class SizeClassAllocator64 {
       }
 
       const uptr PushedBytesDelta = BytesInBG - BG->BytesInBGAtLastCheckpoint;
-      if (PushedBytesDelta < getMinReleaseAttemptSize(BlockSize))
-        continue;
 
       // Given the randomness property, we try to release the pages only if the
       // bytes used by free blocks exceed certain proportion of group size. Note

``````````

</details>


https://github.com/llvm/llvm-project/pull/112252


More information about the llvm-commits mailing list