[PATCH] D64889: [OPENMP] getDSA(): handle loop control variables

2019-07-18 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added a comment. Thanks for the report, but the fix is not quite correct. I'll fix this myself. I just want to push it to 9.0 release branch ASAP. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64889/new/ https://reviews.llvm.org/D64889 _

[PATCH] D64889: [OPENMP] getDSA(): handle loop control variables

2019-07-17 Thread Princeton Ferro via Phabricator via cfe-commits
Prince781 updated this revision to Diff 210459. Prince781 added a comment. Added a lit test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64889/new/ https://reviews.llvm.org/D64889 Files: clang/lib/Sema/SemaOpenMP.cpp clang/test/OpenMP/loop_c

[PATCH] D64889: [OPENMP] getDSA(): handle loop control variables

2019-07-17 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. Needs a test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64889/new/ https://reviews.llvm.org/D64889 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.l

[PATCH] D64889: [OPENMP] getDSA(): handle loop control variables

2019-07-17 Thread Princeton Ferro via Phabricator via cfe-commits
Prince781 created this revision. Prince781 added reviewers: ABataev, rsmith. Prince781 added projects: clang, OpenMP. Herald added subscribers: cfe-commits, jdoerfert, guansong. The following example compiles incorrectly since at least clang 8.0.0: #include #include #define N 100