Skip to content

Conversation

@wrotki
Copy link
Contributor

@wrotki wrotki commented Jan 20, 2026

Must've remained from debugging the test case.

rdar://119958411

Must've remain from debugging the test case.

rdar://119958411
@llvmbot
Copy link
Member

llvmbot commented Jan 20, 2026

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

Author: Mariusz Borsa (wrotki)

Changes

Must've remain from debugging the test case.

rdar://119958411


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

1 Files Affected:

  • (modified) compiler-rt/lib/sanitizer_common/tests/sanitizer_procmaps_mac_test.cpp (-1)
diff --git a/compiler-rt/lib/sanitizer_common/tests/sanitizer_procmaps_mac_test.cpp b/compiler-rt/lib/sanitizer_common/tests/sanitizer_procmaps_mac_test.cpp
index 7547528e2adb8..fbb20ad59341d 100644
--- a/compiler-rt/lib/sanitizer_common/tests/sanitizer_procmaps_mac_test.cpp
+++ b/compiler-rt/lib/sanitizer_common/tests/sanitizer_procmaps_mac_test.cpp
@@ -106,7 +106,6 @@ class MemoryMappingLayoutMock final : public MemoryMappingLayout {
    start_load_cmd_addr = (const char *)(mock_header.data() + header_size);
    sizeofcmds = header->sizeofcmds;
 
-   const char *last_byte_load_cmd_addr = (start_load_cmd_addr + sizeofcmds - 1);
    data_.current_image = -1;  // So the loop in ::Next runs just once
  }
 

@wrotki wrotki requested review from DanBlackwell and ndrewh January 20, 2026 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants