Skip to content

Commit

Permalink
GH-41711: [C++] macros.h: Fix ARROW_FORCE_INLINE for MSVC (#41712)
Browse files Browse the repository at this point in the history
### 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 <felipekde@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
  • Loading branch information
felipecrv committed May 17, 2024
1 parent 8d687b0 commit a04339a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cpp/src/arrow/util/macros.h
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit a04339a

Please sign in to comment.