[PATCH] D35789: [XRay][compiler-rt] Do not print the warning when the binary is not XRay instrumented.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 30 22:00:58 PDT 2017


dblaikie accepted this revision.
dblaikie added a comment.

Phrasing looks like it could still use some polish/consistency, but all good in any case.



================
Comment at: test/xray/TestCases/Linux/quiet-start.cc:5-8
+// RUN: XRAY_OPTIONS="patch_premain=true verbosity=1" %run %t 2>&1 | \
+// RUN:    FileCheck %s --check-prefix NOISY
+// RUN: XRAY_OPTIONS="patch_premain=true verbosity=0" %run %t 2>&1 | \
+// RUN:    FileCheck %s --check-prefix QUIET
----------------
Does this test the default (the comment says it checks what happens when there are no XRay instrumentation sleds - but the code seems to explicitly specify verbosity, rather than relying on any default based on the presence/absence of sleds? (unless that would override the XRAY_OPTIONS?))

Seems like it might be better to phrase this whole change, etc in terms of tying it into the verbosity level & mention as an aside that this means it defaults to off. (& I don't think it's necessary to test that default if the compiler-rt verbosity work already handles the default, tests & documents it, etc))


https://reviews.llvm.org/D35789





More information about the llvm-commits mailing list