[PATCH] D20745: [esan] Add handling of large stack size rlimits
Derek Bruening via llvm-commits
llvm-commits at lists.llvm.org
Fri May 27 13:21:42 PDT 2016
bruening added inline comments.
================
Comment at: lib/esan/CMakeLists.txt:21
@@ +20,3 @@
+ # Assume Linux
+ list(APPEND ESAN_SOURCES
+ esan_linux.cpp)
----------------
bruening wrote:
> eugenis wrote:
> > You append esan_linux.cpp to the list, but it's already in it.
> > Also, we usually do it in the source like this:
> > #include "sanitizer_common/sanitizer_platform.h"
> > #if SANITIZER_FREEBSD || SANITIZER_LINUX
> >
> >
> >
> This is cmake, though. This is following how tsan adds tsan_platform_linux.cc (well tsan doesn't leave the file in the main list too ;)). Is there a cmake variable set in a parent file distinguishing Linux from BSD?
Or are you suggesting to not say "assume linux" here, and have the source file say "#if SANITIZER_LINUX" if it really won't work on BSD (should prob be _posix if it works on BSD)?
http://reviews.llvm.org/D20745
More information about the llvm-commits
mailing list