Skip to content

Commit 7f10fa9

Browse files
authored
[fix](compile)compile error when use clang on aarch64 platform (apache#12319)
1 parent d5e5afe commit 7f10fa9

File tree

2 files changed

+1
-6
lines changed

2 files changed

+1
-6
lines changed

be/src/geo/geo_types.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ void print_s2point(std::ostream& os, const S2Point& point) {
5252
}
5353

5454
static inline bool is_valid_lng_lat(double lng, double lat) {
55-
return abs(lng) <= 180 && abs(lat) <= 90;
55+
return std::abs(lng) <= 180 && std::abs(lat) <= 90;
5656
}
5757

5858
// Return GEO_PARSE_OK, if and only if this can be converted to a valid S2Point

be/src/glibc-compatibility/memcpy/memcpy_aarch64.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,6 @@ static inline __attribute__((always_inline)) __m128i _mm_loadu_si128(const __m12
2222
return vreinterpretq_m128i_s32(vld1q_s32((const int32_t *) p));
2323
}
2424

25-
static inline __attribute__((always_inline)) __m128i _mm_load_si128(const __m128i *p)
26-
{
27-
return vreinterpretq_m128i_s32(vld1q_s32((const int32_t *) p));
28-
}
29-
3025
/** Custom memcpy implementation for ClickHouse.
3126
* It has the following benefits over using glibc's implementation:
3227
* 1. Avoiding dependency on specific version of glibc's symbol, like memcpy@@GLIBC_2.14 for portability.

0 commit comments

Comments
 (0)