This revision was automatically updated to reflect the committed changes.
Closed by commit rC319458: [OpenMP] Diagnose undeclared variables on declare
target clause (authored by kli).
Repository:
rC Clang
https://reviews.llvm.org/D40588
Files:
lib/Sema/SemaOpenMP.cpp
test/OpenMP/declare_t
This revision was automatically updated to reflect the committed changes.
Closed by commit rL319458: [OpenMP] Diagnose undeclared variables on declare
target clause (authored by kli).
Changed prior to commit:
https://reviews.llvm.org/D40588?vs=124811&id=124969#toc
Repository:
rL LLVM
https:
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
https://reviews.llvm.org/D40588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
kkwli0 updated this revision to Diff 124811.
kkwli0 added a comment.
The assert occurs in VarOrFuncDeclFilterCCC::ValidateCandidate when
clang::Sema::CorrectTypo is called.
https://reviews.llvm.org/D40588
Files:
lib/Sema/SemaOpenMP.cpp
test/OpenMP/declare_target_messages.cpp
Index: test/
kkwli0 added a comment.
The original case has the variables named "foo1" and foo2". Using "foo1" or
"foo2" causes the assert! I update the description.
https://reviews.llvm.org/D40588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http:/
ABataev requested changes to this revision.
ABataev added a comment.
This revision now requires changes to proceed.
I'm unable to reproduce a crash, the test works correctly even without you patch
https://reviews.llvm.org/D40588
___
cfe-commits mail
