[llvm-branch-commits] [openmp] [OpenMP][offload] Inline target reductions (PR #196061)

[Joseph Huber via llvm-branch-commits]

================
@@ -22,15 +22,19 @@ using namespace ompx;
 
 namespace {
 
-void gpu_regular_warp_reduce(void *reduce_data, ShuffleReductFnTy shflFct) {
+[[clang::always_inline]]
+static void gpu_regular_warp_reduce(void *reduce_data,
----------------
jhuber6 wrote:

It's in the style guide, supposedly it makes the linkage more obvious compared to a big namespace. And, I generally do think that needing to add always inline represents a problem with our codegen or inliner heuristics.

https://github.com/llvm/llvm-project/pull/196061