[PATCH] D92533: [flang][openacc] Add missing loop construct restriction and validity tests

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 4 12:28:22 PST 2020


clementval marked 2 inline comments as done.
clementval added inline comments.


================
Comment at: flang/lib/Semantics/canonicalize-acc.cpp:87
 
+  // Check constraint on line 1835 in Section 2.9
+  // A tile and collapse clause may not appear on loop that is associated with
----------------
clementval wrote:
> sameeranjoshi wrote:
> > Which standard ? 
> > There are patches for 3.1 up gradation, will line numbers change in that case? 
> > Is 3.0 assumed when no reference?
> From no one all reference are from 3.1. Right now it's like a transition phase until patch D92120 is approved and landed. 
>From now on ... and not "no one"!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92533



More information about the llvm-commits mailing list