[flang-commits] [PATCH] D89583: [Flang][OpenMP] Semantic checks for Atomic construct.

Tue Oct 20 01:44:45 PDT 2020

sameeranjoshi marked an inline comment as done.
sameeranjoshi added inline comments.

================
Comment at: flang/test/Semantics/omp-atomic01.f90:4
+
+  !TODO: At most one hint clause may appear on the construct.
+use omp_lib
----------------
clementval wrote:
> Regarding your general comment. `hint` does not have a `parser::Hint` since it is in the `OmpClause` variant. So you should be able to resolve this todo in this patch as well. 
Thank  you @clementval.
I have resolved the TODO.
`OMPC_Hint` inside `OMP.td` was supposed to use `flangClassValue` instead of `flangClass`.

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

https://reviews.llvm.org/D89583