[PATCH] D34669: [XRay][compiler-rt][NFC] Add example always/never instrument files.

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 22:54:54 PDT 2017


dberris created this revision.

This change introduces two files that show exaples of the
always/never instrument files that can be provided to clang. We don't
add these as defaults yet in clang, which we can do later on (in a
separate change).

We also add a test that makes sure that these apply in the compiler-rt
project tests, and that changes in clang don't break the expectations in
compiler-rt.


https://reviews.llvm.org/D34669

Files:
  lib/xray/xray_always_instrument.txt
  lib/xray/xray_never_instrument.txt
  test/xray/TestCases/always-never-instrument.cc


Index: test/xray/TestCases/always-never-instrument.cc
===================================================================
--- /dev/null
+++ test/xray/TestCases/always-never-instrument.cc
@@ -0,0 +1,21 @@
+// Test that the always/never instrument lists apply.
+// RUN: echo "fun:main" > %tmp-always.txt
+// RUN: echo "fun:__xray*" > %tmp-never.txt
+// RUN: %clangxx_xray \
+// RUN:     -fxray-never-instrument=%tmp-never.txt \
+// RUN:     -fxray-always-instrument=%tmp-always.txt \
+// RUN:     %s -o %t
+// RUN: %llvm_xray extract -symbolize %t | \
+// RUN:    FileCheck %s --check-prefix NOINSTR
+// RUN: %llvm_xray extract -symbolize %t | \
+// RUN:    FileCheck %s --check-prefix ALWAYSINSTR
+
+// NOINSTR-NOT: {{.*__xray_NeverInstrumented.*}}
+int __xray_NeverInstrumented() {
+  return 0;
+}
+
+// ALWAYSINSTR: {{.*function-name:.*main.*}}
+int main(int argc, char *argv[]) {
+  return __xray_NeverInstrumented();
+}
Index: lib/xray/xray_never_instrument.txt
===================================================================
--- /dev/null
+++ lib/xray/xray_never_instrument.txt
@@ -0,0 +1,6 @@
+# List of function matchers common to C/C++ applications that make sense to
+# never instrument. You can use this as an argument to
+# -fxray-never-instrument=<path> along with your project-specific lists.
+
+# Never instrument any function whose symbol starts with __xray.
+fun:__xray*
Index: lib/xray/xray_always_instrument.txt
===================================================================
--- /dev/null
+++ lib/xray/xray_always_instrument.txt
@@ -0,0 +1,6 @@
+# List of function matchers common to C/C++ applications that make sense to
+# always instrument. You can use this as an argument to
+# -fxray-always-instrument=<path> along with your project-specific lists.
+
+# Always instrument the main function.
+fun:main


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34669.104095.patch
Type: text/x-patch
Size: 1832 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170627/e4ac2623/attachment.bin>


More information about the llvm-commits mailing list