[PATCH] D46998: [XRay][compiler-rt] Limit reliance on C++ ABI features

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 21 00:16:29 PDT 2018


echristo added a comment.

One general inline comment. Otherwise, yes, any decoupling you can do from the C++ standard library is goodness.



================
Comment at: compiler-rt/lib/xray/xray_basic_logging.cc:165
   uint8_t CPU = 0;
-  uint64_t TSC = ReadTSC(CPU);
+  u64 TSC = ReadTSC(CPU);
 
----------------
Mild preference to avoid u64 and still use uint64_t here.


https://reviews.llvm.org/D46998





More information about the llvm-commits mailing list