[compiler-rt] [scudo] Do not unmap the memory containing the this object in unmapRingBuffer (PR #83034)

via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 26 14:14:59 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 1d2eced0067bea989e98efc9265725b3aeca0af9 6608d26179202f652e9db110c3e7d1dfe69cdc7d -- compiler-rt/lib/scudo/standalone/combined.h
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/compiler-rt/lib/scudo/standalone/combined.h b/compiler-rt/lib/scudo/standalone/combined.h
index 581be0deeb..fa6077384d 100644
--- a/compiler-rt/lib/scudo/standalone/combined.h
+++ b/compiler-rt/lib/scudo/standalone/combined.h
@@ -1611,8 +1611,8 @@ private:
     RB->RawStackDepotMap.unmap(RB->RawStackDepotMap.getBase(),
                                RB->RawStackDepotMap.getCapacity());
     // Note that the `RB->RawRingBufferMap` is stored on the pages managed by
-    // itself. Take over the ownership before calling unmap() so that any operation
-    // along with unmap() won't touch inaccessible pages.
+    // itself. Take over the ownership before calling unmap() so that any
+    // operation along with unmap() won't touch inaccessible pages.
     MemMapT RawRingBufferMap = RB->RawRingBufferMap;
     RawRingBufferMap.unmap(RawRingBufferMap.getBase(),
                            RawRingBufferMap.getCapacity());

``````````

</details>


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


More information about the llvm-commits mailing list