chenBright commented on code in PR #3247:
URL: https://github.com/apache/brpc/pull/3247#discussion_r3652196567
##########
src/butil/time.h:
##########
@@ -279,7 +279,7 @@ extern int64_t invariant_cpu_freq;
// note: Inlining shortens time cost per-call for 15ns in a loop of many
// calls to this function.
inline int64_t cpuwide_time_ns() {
-#if !defined(BAIDU_INTERNAL)
+#if !defined(BAIDU_INTERNAL) && !defined(__aarch64__)
Review Comment:
Add a macro switch, such as BUTIL_USE_CPU_FREQUENCY, to let users choose
whether to enable it instead of enabling it by default on arm64.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
