[Allocator] Remove memory poisoning before deallocation
authorReid Kleckner <reid@kleckner.net>
Wed, 22 Apr 2015 20:56:42 +0000 (20:56 +0000)
committerReid Kleckner <reid@kleckner.net>
Wed, 22 Apr 2015 20:56:42 +0000 (20:56 +0000)
commitf52e2f3ef5b492f8f9439f45737e3a5a78d9b5d9
tree5933053e99cd358192f30525886a9e3d4c81418b
parentfd491461fc4bf24de4b29d5fba6f2c09c5d2311f
[Allocator] Remove memory poisoning before deallocation

I added the poisoning back in r76891 (2009) because of some bugs in
Unladen Swallow, and then Evan Cheng added the setRangeWritable() call
in r81308. Profiling a Release+Asserts build on Windows shows that this
memory protection call is actually very expensive. 4 seconds of a 70
second Clang compilation are spent in VirtualQuery. These days we have
more reliable tools like ASan to find these kinds of bugs, so we can go
ahead and retire these checks.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@235542 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/Support/Allocator.h