[PATCH] D29458: [asan] Move exception code to sanitizer_common.

Marcos Pividori via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 2 10:53:08 PST 2017

mpividori added inline comments.

Comment at: lib/sanitizer_common/sanitizer_common.h:384
 bool IsHandledDeadlySignal(int signum);
+bool IsHandledDeadlyException(DWORD exceptionCode);
rnk wrote:
> kcc wrote:
> > no #if please. 
> > The only #if we accept (except for some special files) is a single #if around the entire file. 
> sanitizer_common.h has a lot of #ifs, so I thought this was acceptable. If we just remove the if here, a caller will get a link error. Would you prefer that?
@kcc if you prefer I can add an empty implementation for Posix.


More information about the llvm-commits mailing list