[flang-commits] [flang] [Flang][OpenMP]Support for lowering task_reduction and in_reduction to MLIR (PR #111155)
Sergio Afonso via flang-commits
flang-commits at lists.llvm.org
Fri Dec 6 07:22:47 PST 2024
================
@@ -104,6 +104,9 @@ class ClauseProcessor {
bool processIsDevicePtr(
mlir::omp::IsDevicePtrClauseOps &result,
llvm::SmallVectorImpl<const semantics::Symbol *> &isDeviceSyms) const;
+ bool processInReduction(
+ mlir::Location currentLocation, mlir::omp::InReductionClauseOps &result,
+ llvm::SmallVectorImpl<const semantics::Symbol *> &InReductionSyms) const;
----------------
skatrak wrote:
Nit: I think it's better to specify that it's intended as an output rather than restating the full clause name. Feel free to disagree, but in that case please fix capitalization.
```suggestion
llvm::SmallVectorImpl<const semantics::Symbol *> &outReductionSyms) const;
```
https://github.com/llvm/llvm-project/pull/111155
More information about the flang-commits
mailing list