Skip to content

Commit b8d9f64

Browse files
committed
resolve comments
1 parent 3b19264 commit b8d9f64

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

onnxruntime/core/mlas/lib/softmax_kernel_neon_fp16.cpp

+6-6
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,11 @@ constexpr MlasExpConstants<_mlas_fp16_> ExpConstantsFp16 = {
6565

6666
template <typename T>
6767
MLAS_FORCEINLINE
68-
MlasExpConstants<T> Get_Exp_Constants();
68+
const MlasExpConstants<T>& Get_Exp_Constants();
6969

7070
template <>
7171
MLAS_FORCEINLINE
72-
MlasExpConstants<float16x8_t> Get_Exp_Constants<float16x8_t>() {
72+
const MlasExpConstants<float16x8_t>& Get_Exp_Constants<float16x8_t>() {
7373
const static MlasExpConstants<float16x8_t> ExpConstantsFp16x8 = {
7474
MlasBroadcastFloat16x8(ExpConstantsFp16.LowerRange),
7575
MlasBroadcastFloat16x8(ExpConstantsFp16.UpperRange),
@@ -94,7 +94,7 @@ MlasExpConstants<float16x8_t> Get_Exp_Constants<float16x8_t>() {
9494

9595
template <>
9696
MLAS_FORCEINLINE
97-
MlasExpConstants<float16x4_t> Get_Exp_Constants<float16x4_t>() {
97+
const MlasExpConstants<float16x4_t>& Get_Exp_Constants<float16x4_t>() {
9898
const static MlasExpConstants<float16x4_t> ExpConstantsFp16x4 = {
9999
MlasBroadcastFloat16x4(ExpConstantsFp16.LowerRange),
100100
MlasBroadcastFloat16x4(ExpConstantsFp16.UpperRange),
@@ -431,11 +431,11 @@ constexpr MlasTanhConstants<_mlas_fp16_> TanhConstantsFp16 = {
431431

432432
template <typename T>
433433
MLAS_FORCEINLINE
434-
MlasTanhConstants<T> Get_Tanh_Constants();
434+
const MlasTanhConstants<T>& Get_Tanh_Constants();
435435

436436
template <>
437437
MLAS_FORCEINLINE
438-
MlasTanhConstants<float16x8_t> Get_Tanh_Constants<float16x8_t>() {
438+
const MlasTanhConstants<float16x8_t>& Get_Tanh_Constants<float16x8_t>() {
439439
const static MlasTanhConstants<float16x8_t> TanhConstantsFp16x8 = {
440440
MlasBroadcastFloat16x8(TanhConstantsFp16.LowerRange),
441441
MlasBroadcastFloat16x8(TanhConstantsFp16.UpperRange),
@@ -453,7 +453,7 @@ MlasTanhConstants<float16x8_t> Get_Tanh_Constants<float16x8_t>() {
453453

454454
template <>
455455
MLAS_FORCEINLINE
456-
MlasTanhConstants<float16x4_t> Get_Tanh_Constants<float16x4_t>() {
456+
const MlasTanhConstants<float16x4_t>& Get_Tanh_Constants<float16x4_t>() {
457457
const static MlasTanhConstants<float16x4_t> TanhConstantsFp16x4 = {
458458
MlasBroadcastFloat16x4(TanhConstantsFp16.LowerRange),
459459
MlasBroadcastFloat16x4(TanhConstantsFp16.UpperRange),

0 commit comments

Comments
 (0)