[PATCH] D93205: [Flang][OpenMP 4.5] Add semantic check for OpenMP Do Loop Constructs for single directive

Yashaswini Hegde via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 21 09:42:06 PST 2020


yhegde added inline comments.


================
Comment at: flang/lib/Semantics/check-omp-structure.h:198
       const parser::OmpObjectList &, std::vector<const Symbol *> &);
+
+  const parser::Name GetLoopIndex(const parser::DoConstruct *x);
----------------
clementval wrote:
> Remove blank line. Style is not really consistent across file but let's try to keep with the same style in a single file. 
That blank line is left to separate the functions specific to a patch just like a blank line is left before CheckDependList. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93205



More information about the llvm-commits mailing list