Skip to content

Commit

Permalink
Fix 1.15 build and run issues on iOS
Browse files Browse the repository at this point in the history
  • Loading branch information
winstondu committed Sep 2, 2020
1 parent e5c130d commit 9784058
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion tensorflow/contrib/makefile/download_dependencies.sh
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ if [ ! -f $BZL_FILE_PATH ]; then
exit 1;
fi

EIGEN_URL="$(grep -o 'https://bitbucket.org/eigen/eigen/get/.*tar\.gz' "${BZL_FILE_PATH}" | grep -v mirror.bazel | head -n1)"
EIGEN_URL="$(grep -o 'https://storage.googleapis.com/mirror.tensorflow.org/bitbucket.org/eigen/eigen/get/.*tar.gz' "${BZL_FILE_PATH}")"
GEMMLOWP_URL="$(grep -o 'https://storage.googleapis.com/mirror.tensorflow.org/github.com/google/gemmlowp/.*zip' "${BZL_FILE_PATH}" | head -n1)"
GOOGLETEST_URL="https://github.com/google/googletest/archive/release-1.8.0.tar.gz"
NSYNC_URL="$(grep -o 'https://storage.googleapis.com/mirror.tensorflow.org/github.com/google/nsync/.*tar\.gz' "${BZL_FILE_PATH}" | head -n1)"
Expand Down
4 changes: 4 additions & 0 deletions tensorflow/core/platform/profile_utils/cpu_utils.h
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,10 @@ class CpuUtils {
__asm__ volatile("rdtsc" : "=a"(low), "=d"(high));
return (high << 32) | low;
// ----------------------------------------------------------------
#elif defined(__aarch64__) && defined(TARGET_OS_IOS)
// On iOS, we are not able to access the cntvct_el0 register.
// As a temporary build fix, we will just return the dummy cycle clock.
return DUMMY_CYCLE_CLOCK

This comment has been minimized.

Copy link
@alionkun

alionkun Apr 12, 2021

build on macos report cpu_utils.h:73:29: error: expected ';' after return statement

@winstondu

#elif defined(__aarch64__)
// System timer of ARMv8 runs at a different frequency than the CPU's.
// The frequency is fixed, typically in the range 1-50MHz. It can because
Expand Down

0 comments on commit 9784058

Please sign in to comment.