[PATCH] D64086: [scudo][standalone] Potential fix for missing sized delete

[Kostya Kortchinsky via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL365045: [scudo][standalone] Potential fix for missing sized delete (authored by cryptoad, committed by ).

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64086/new/

https://reviews.llvm.org/D64086

Files:
  compiler-rt/trunk/lib/scudo/standalone/tests/CMakeLists.txt
  compiler-rt/trunk/lib/scudo/standalone/tests/wrappers_cpp_test.cc


Index: compiler-rt/trunk/lib/scudo/standalone/tests/wrappers_cpp_test.cc
===================================================================
--- compiler-rt/trunk/lib/scudo/standalone/tests/wrappers_cpp_test.cc
+++ compiler-rt/trunk/lib/scudo/standalone/tests/wrappers_cpp_test.cc
@@ -12,6 +12,9 @@
 #include <mutex>
 #include <thread>
 
+void operator delete(void *, size_t) noexcept;
+void operator delete[](void *, size_t) noexcept;
+
 // Note that every Cxx allocation function in the test binary will be fulfilled
 // by Scudo. See the comment in the C counterpart of this file.
 
Index: compiler-rt/trunk/lib/scudo/standalone/tests/CMakeLists.txt
===================================================================
--- compiler-rt/trunk/lib/scudo/standalone/tests/CMakeLists.txt
+++ compiler-rt/trunk/lib/scudo/standalone/tests/CMakeLists.txt
@@ -12,7 +12,7 @@
   -I${COMPILER_RT_SOURCE_DIR}/lib/scudo/standalone
   -DGTEST_HAS_RTTI=0
   # Extra flags for the C++ tests
-  -fsized-deallocation
+  # TODO(kostyak): find a way to make -fsized-deallocation work
   -Wno-mismatched-new-delete)
 
 set(SCUDO_TEST_ARCH ${SCUDO_STANDALONE_SUPPORTED_ARCH})


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64086.207794.patch
Type: text/x-patch
Size: 1155 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190703/be9815e7/attachment.bin>