[PATCH] D89879: [Flang][OpenMP-5.0] Semantic checks for flush construct.

Wed Nov 25 22:26:22 PST 2020

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

================
Comment at: flang/test/Semantics/omp-flush01.f90:28
+    !ERROR: expected end of line
+    !$omp flush acq_rel acq_rel
+    !ERROR: expected end of line
----------------
clementval wrote:
> Why is the error reported twice here? Would it not be nicer to have a list of `memory-order-clause` and then the error being reported by the semantic check? I mean it would be more user friendly. 
It's a known issue due the improper error recovery in openmp.
omp-atomic.f90 file too had to get a workaround using the same way.

Yes, I have forwarded the errors to semantic phase.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89879