[PATCH] D35591: [sanitizer_common] Move filesystem-related code out of sanitizer_common.cc

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 19 12:08:28 PDT 2017


alekseyshl accepted this revision.
alekseyshl added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/sanitizer_common/CMakeLists.txt:73
 # headers when building our custom unit tests.
 set(SANITIZER_HEADERS
   sanitizer_addrhashmap.h
----------------
Add sanitizer_file.h to this set


================
Comment at: lib/sanitizer_common/sanitizer_file.h:10
+//
+// This file is shared between run-time libraries of sanitizers.
+//
----------------
Please add here the rationale for the existence of this file (from the patch description).


Repository:
  rL LLVM

https://reviews.llvm.org/D35591





More information about the llvm-commits mailing list