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
_
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
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
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