This is an automated email from the ASF dual-hosted git repository.
kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new a04339a49e GH-41711: [C++] macros.h: Fix ARROW_FORCE_INLINE for MSVC
(#41712)
a04339a49e is described below
commit a04339a49e08bec8c559a32c01eacc9469ef4196
Author: Felipe Oliveira Carvalho <[email protected]>
AuthorDate: Fri May 17 18:40:53 2024 -0300
GH-41711: [C++] macros.h: Fix ARROW_FORCE_INLINE for MSVC (#41712)
### Rationale for this change
Define the macro correctly. Nothing is broken at the moment because the
macro is not used within Arrow at the moment.
### What changes are included in this PR?
Correct definition of the macro.
### Are these changes tested?
Yes, by having this commit in other PRs that pass CI tests on Windows.
* GitHub Issue: #41711
Authored-by: Felipe Oliveira Carvalho <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
---
cpp/src/arrow/util/macros.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpp/src/arrow/util/macros.h b/cpp/src/arrow/util/macros.h
index d0c05a7908..484df3400d 100644
--- a/cpp/src/arrow/util/macros.h
+++ b/cpp/src/arrow/util/macros.h
@@ -102,7 +102,7 @@
#elif defined(_MSC_VER) // MSVC
#define ARROW_NORETURN __declspec(noreturn)
#define ARROW_NOINLINE __declspec(noinline)
-#define ARROW_FORCE_INLINE __declspec(forceinline)
+#define ARROW_FORCE_INLINE __forceinline
#define ARROW_PREDICT_FALSE(x) (x)
#define ARROW_PREDICT_TRUE(x) (x)
#define ARROW_PREFETCH(addr)
