[PATCH] D32846: [XRay][compiler-rt] Add function id utilities for XRay

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 4 10:30:37 PDT 2017


dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.

Slightly curious to me that function IDs are > 0, but use a signed type - but no big deal, I guess there's probably a reason.



================
Comment at: test/xray/TestCases/Linux/func-id-utils.cc:26-28
+  // CHECK-LABEL: addresses:
+  // CHECK: #[[MAX]] -> @[[ADDR:.*]]
+  // CHECK-NOT: #0 -> @{{.*}}
----------------
What does this test?

Should the test case print the &foo and &bar, or assert(xray_function_address(1) == &foo), etc?


https://reviews.llvm.org/D32846





More information about the llvm-commits mailing list