[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