[llvm-commits] AddressSanitizer: use macro to define and declare interceptors for system functions (issue 5609049)

[ramosian.glider at gmail.com]

http://codereview.appspot.com/5609049/diff/1/asan_interceptors.cc
File asan_interceptors.cc (right):

http://codereview.appspot.com/5609049/diff/1/asan_interceptors.cc#newcode486
asan_interceptors.cc:486: INTERCEPTOR(char*, strcpy, char *to, const
char *from) {  // NOLINT
Do you still need NOLINT here?

http://codereview.appspot.com/5609049/diff/1/asan_interceptors.h
File asan_interceptors.h (right):

http://codereview.appspot.com/5609049/diff/1/asan_interceptors.h#newcode26
asan_interceptors.h:26: // you want to use it in other parts of RTL,
you'll need to
either add a colon after "to", or remove it from other places.

http://codereview.appspot.com/5609049/diff/1/asan_interceptors.h#newcode66
asan_interceptors.h:66: FUNC_TYPE(func) REAL(func); \
Do you need DEFINE_REAL? Looks like DECLARE_REAL covers all the possible
use cases.

http://codereview.appspot.com/5609049/diff/1/asan_mac.cc
File asan_mac.cc (right):

http://codereview.appspot.com/5609049/diff/1/asan_mac.cc#newcode464
asan_mac.cc:464: dispatch_queue_t dq, void *ctxt,
Please make sure the indentations are correct. I think they are not now.

http://codereview.appspot.com/5609049/diff/1/asan_mac.h
File asan_mac.h (right):

http://codereview.appspot.com/5609049/diff/1/asan_mac.h#newcode59
asan_mac.h:59: CFAllocatorRef alloc,
Mind the indentation.

http://codereview.appspot.com/5609049/