LGTM
https://codereview.chromium.org/208743004/diff/20001/src/zone-inl.h
File src/zone-inl.h (right):
https://codereview.chromium.org/208743004/diff/20001/src/zone-inl.h#newcode34
src/zone-inl.h:34: #include <sanitizer/asan_interface.h>
A style nit. I would do
#ifdef ADDRESS_SANITIZER
# include <...>
#else
// define a few macros as empty
#endif
Then, instead of
#ifdef ADDRESS_SANITIZER
ASAN_MACRO(...)
#endif
I would use
ASAN_MACRO(...)
This is just to make less ifdefs in the code.
Otherwise looks good.
(I can't give a proper comment on the build file changes)
https://codereview.chromium.org/208743004/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
