[flang-commits] [flang] [OpenMP][flang] Move `todo` for checking reduction support status on the GPU (PR #175172)

[Sergio Afonso via flang-commits]

================
@@ -29,6 +30,34 @@ namespace flangomp {
 using namespace mlir;
 
 namespace {
+void checkDeviceImplementationStatus(
----------------
skatrak wrote:

Nit(x2): The purpose of this function should be documented, to prevent others from adding checks here that should go into MLIR to LLVM IR translation. Something like this (modify as you please):
```
/**
 * This function triggers TODO errors and halts compilation if it detects
 * patterns representing unimplemented features.
 *
 * It exclusively checks situations that cannot be detected after all of the
 * MLIR pipeline has ran (i.e. at the MLIR to LLVM IR translation stage, where
 * the preferred location for these types of checks is), and it only checks for
 * features that have not been implemented for target offload, but are supported
 * on host execution.
 */
```

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