[PATCH] D96686: [Flang][OpenMP 4.5] Add semantic check for OpenMP Do Loop Constructs for Threadprivate

Yashaswini Hegde via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 8 05:40:44 PST 2021


yhegde added a comment.

In D96686#2610323 <https://reviews.llvm.org/D96686#2610323>, @kiranchandramohan wrote:

> LGTM. Yes, using the symbol is the right way here.
> Thanks @yhegde for separating this out into a new patch. Thanks also for you patience and all your work for semantic checks for OpenMP.

Thank you @kiranchandramohan for the review comments. That was a good opportunity to understand flang and it was nice working with you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96686



More information about the llvm-commits mailing list