[PATCH] [dfsan] Runtime support for ABI list functionality; can now run integration tests with args ABI.

Evgeniy Stepanov eugenis at google.com
Mon Aug 12 03:33:59 PDT 2013


  LGTM


================
Comment at: lib/dfsan/CMakeLists.txt:30
@@ +29,3 @@
+add_custom_target(dfsan_abilist ALL
+                                SOURCES ${CLANG_RESOURCE_DIR}/dfsan_abilist.txt)
+add_custom_command(OUTPUT ${CLANG_RESOURCE_DIR}/dfsan_abilist.txt
----------------
80 chars

================
Comment at: lib/dfsan/dfsan.cc:173
@@ +172,3 @@
+extern "C" SANITIZER_INTERFACE_ATTRIBUTE dfsan_label
+__dfsw_dfsan_get_label(long data, dfsan_label data_label,
+                       dfsan_label *ret_label) {
----------------
This probably deserves a comment.


http://llvm-reviews.chandlerc.com/D1351

BRANCH
  dfsan-up6

ARCANIST PROJECT
  compiler-rt



More information about the llvm-commits mailing list