[PATCH] D123403: [OpenMP] Refactor OMPScheduleType enum.

Thu Apr 14 18:02:41 PDT 2022

peixin accepted this revision.
peixin added inline comments.
This revision is now accepted and ready to land.

================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMPConstants.h:135
+  UnorderedGuidedSimd = BaseGuidedSimd | ModifierUnordered,   // (46)
+  UnorderedRuntimeSimd = BaseRuntimeSimd | ModifierUnordered, // (47)
+
----------------
Meinersbur wrote:
> peixin wrote:
> > Why not using the following to be consistent with the name in kmp.h?
> > StaticBalancedChunked 
> > GuidedSimd
> > RuntimeSimd
> As mentioned in the summary. to avoid confusion by not using the original name. `StaticBalancedChunked` could mean either the algorithm to use (now `BaseStaticBalancedChunked`, as in `omp_sched_t`/`enum kmp_sched`), or that algorithm with the unordered flag set (now `UnorderedStaticBalancedChunked `). I would the former because that's how the enum is structured.
> 
> The name in `kmp.h` for it is actually `kmp_sch_static_balanced_chunked`. `sch` for "unordered"?
OK. Agree.

> The name in kmp.h for it is actually kmp_sch_static_balanced_chunked. sch for "unordered"?

Yes, I think so.  With simd modifier, it cannot be ordered in semantics.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123403/new/

https://reviews.llvm.org/D123403