[flang-commits] [flang] [llvm] [mlir] [flang][OpenMP] Lower target in_reduction for host fallback (PR #199967)

[Sergio Afonso via flang-commits]

================
@@ -2440,6 +2440,39 @@ class OpenMPIRBuilder {
 
   ///}
 
+  /// Emit the host-fallback runtime lookups that redirect the `in_reduction`
----------------
skatrak wrote:

Nit: This is a general comment that involves changes across all files in this patch, and the PR title/description. Host fallback refers to the host version of device code that gets executed when e.g. the device is unavailable, but `in_reduction` is a host-only clause with no associated target device logic. I'd like to ask you to update your references to "`in_reduction` host fallback" and either remove that clarification or more generally refer to the host, depending on what works better for each case.
```suggestion
  /// Emit the host runtime lookups that redirect the `in_reduction`
```

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