[flang-commits] [PATCH] D90538: [Flang][OpenMP] Fix 'Internal: no symbol found' for OpenMP aligned and linear clause.

sameeran joshi via Phabricator via flang-commits flang-commits at lists.llvm.org
Mon Nov 2 22:08:53 PST 2020


sameeranjoshi marked 2 inline comments as done.
sameeranjoshi added a comment.

Thanks for reviewing.



================
Comment at: flang/lib/Semantics/resolve-directives.cpp:288
+    }
+    return true;
+  }
----------------
kiranchandramohan wrote:
> Can this be false here?
Keeping it for consistency sake with the convention followed in the file.
Just asking if you know is there some reason to use some specific boolean value?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90538



More information about the flang-commits mailing list