[PATCH] D90538: [Flang][OpenMP] Fix 'Internal: no symbol found' for OpenMP aligned and linear clause.
sameeran joshi via Phabricator via llvm-commits
llvm-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 llvm-commits
mailing list