[PATCH] D29438: [XRay] Probe for CPU features that XRay needs
Dean Michael Berris via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 10 05:44:58 PDT 2017
dberris marked an inline comment as done.
dberris added inline comments.
================
Comment at: compiler-rt/trunk/lib/xray/xray_x86_64.cc:211-213
+ // EDX. That's 0x8000000 (or 1u << 26).
+ __get_cpuid(0x80000001, &EAX, &EBX, &ECX, &EDX);
+ if (!(EDX & (1u << 26))) {
----------------
jbulow wrote:
> 1u<<26 == 0x4000000
>
> Changing 26 to 27 solves this problem when running with xray enabled:
>
>
> ```
> ==20069==Missing rdtscp support.
>
> ```
>
>
Oh wow, that's embarrassing! Good catch, thanks @jbulow!
Repository:
rL LLVM
https://reviews.llvm.org/D29438
More information about the llvm-commits
mailing list