ci: macos: Remove use of -DRANDOMIZE_ALLOCATED_MEMORY
authorAndres Freund <[email protected]>
Sat, 12 Aug 2023 20:06:04 +0000 (13:06 -0700)
committerAndres Freund <[email protected]>
Sat, 12 Aug 2023 22:08:07 +0000 (15:08 -0700)
RANDOMIZE_ALLOCATED_MEMORY causes a measurable slowdown. Macos is, by far, the
most expensive platform for CI, therefore it doesn't make sense to run such a
test there.

Ubsan and asan on linux should detect most of the the cases of uninitialized
memory, so it doesn't really seem worth using -DRANDOMIZE_ALLOCATED_MEMORY in
another instance type.

Author: Andres Freund <[email protected]>
Reviewed-by: Heikki Linnakangas <[email protected]>
Discussion: https://postgr.es/m/20230808021541[email protected]
Backpatch: 16-, where 89d16b635 added the use of -DRANDOMIZE_ALLOCATED_MEMORY

.cirrus.yml

index 113f4545244cc86f9f24361f4c2fe3ba245f23ab..13d17da30b0c9a63f209c4018cc9d0b247c368d0 100644 (file)
@@ -434,7 +434,6 @@ task:
 
     CC: ccache cc
     CXX: ccache c++
-    CPPFLAGS: -DRANDOMIZE_ALLOCATED_MEMORY
     CFLAGS: -Og -ggdb
     CXXFLAGS: -Og -ggdb