[flang-commits] [flang] [mlir] [MLIR][OpenMP] NFC: Uniformize OpenMP ops names (PR #85393)
Krzysztof Parzyszek via flang-commits
flang-commits at lists.llvm.org
Fri Mar 15 07:09:00 PDT 2024
kparzysz wrote:
> * `omp.critical.declare` -> `omp.critical_declare` / `omp.declare.critical` / keep it unchanged?
> * `omp.reduction.declare` -> `omp.reduction_declare` / `omp.declare.reduction` / keep it unchanged?
The first one doesn't exist in the spec, the second one is "declare reduction", so for consistency we should use the names with "declare_xyz".
> * `omp.atomic.read` -> `omp.atomic_read`
> * `omp.atomic.write` -> `omp.atomic_write`
> * `omp.atomic.update` -> `omp.atomic_update`
> * `omp.atomic.capture` -> `omp.atomic_capture`
The "read", "write", etc. are clauses, so "atomic read" is not a name. I suggest keeping the . as a separator here.
https://github.com/llvm/llvm-project/pull/85393
More information about the flang-commits
mailing list