[PATCH] D39798: [Sanitizers] Use SANITIZER_* macros in lib/interception

[Rainer Orth via Phabricator via llvm-commits]

ro marked 2 inline comments as done.
ro added inline comments.


================
Comment at: lib/interception/interception_mac.cc:17
 
+#if SANITIZER_MAC
 
----------------
krytarowski wrote:
> ro wrote:
> > ro wrote:
> > > krytarowski wrote:
> > > > Does it make sense?
> > > How do you mean?  One could do away with the effectively empty
> > > file completely.
> > Sorry but I still don't know what you mean here.
> I would just drop this:
> 
> ```
> #if SANITIZER_MAC
> #endif
> ```
Fine with me.  However, there's the risk of breaking non-OS X
builds if there's code added to the file and the guard forgotten...

But let's cross that bridge when/if we get there.


https://reviews.llvm.org/D39798