[PATCH] D12763: [PATCH] [compiler-rt] [sanitizers] Add VMA size check at runtime

Adhemerval Zanella via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 10 09:57:18 PDT 2015


zatrazz added inline comments.

================
Comment at: lib/sanitizer_common/sanitizer_posix.cc:336
@@ +335,3 @@
+    Printf("WARNING: Runtime VMA is not the one built for.\n");
+    Printf("\tBuilt VMA:   %p\n", maxBuiltVirtualSize);
+    Printf("\tRuntime VMA: %p\n", maxRuntimeVirtualSize);
----------------
rengolin wrote:
> I was expecting this to print things like 39/42, not the max size.
I tried to be less intrusive as possible, since only GetMaxVirtualAddress is really defined in a multiplatform way. However if the idea is to show 39/42 I think we can create a symbol to return it as well.


http://reviews.llvm.org/D12763





More information about the llvm-commits mailing list