[PATCH] D82906: [flang][openmp] Use common Directive and Clause enum from llvm/Frontend
Valentin Clement via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 1 16:13:58 PDT 2020
clementval added a comment.
In D82906#2126684 <https://reviews.llvm.org/D82906#2126684>, @dblaikie wrote:
> In D82906#2126543 <https://reviews.llvm.org/D82906#2126543>, @clementval wrote:
>
> > In D82906#2126444 <https://reviews.llvm.org/D82906#2126444>, @ymandel wrote:
> >
> > > Seems like this is causing a slew of compilation warnings, like:
> > >
> > > [745/3847] Building CXX object tools/clang/lib/Sema/CMakeFiles/obj.clangSema.dir/SemaConcept.cpp.o
> > > In file included from /usr/local/google/home/yitzhakm/remote-build/llvm-git/llvm-project/clang/lib/Sema/SemaConcept.cpp:24:
> > > /usr/local/google/home/yitzhakm/remote-build/llvm-git/llvm-project/clang/include/clang/AST/RecursiveASTVisitor.h:2994:14: warning: enumeration values 'OMPC_inbranch', 'OMPC_link', and 'OMPC_notinbranch' not handled in switch [-Wswitch]
> > > switch (C->getClauseKind()) {
> > > ^
> > > 1 warning generated.
> > > [766/3847] Building CXX object tools/clang/lib/Parse/CMakeFiles/obj.clangParse.dir/ParseOpenMP.cpp.o
> > > /usr/local/google/home/yitzhakm/remote-build/llvm-git/llvm-project/clang/lib/Parse/ParseOpenMP.cpp:1689:11: warning: 19 enumeration values not handled in switch: 'OMPD_distribute_parallel_do', 'OMPD_distribute_parallel_do_simd', 'OMPD_do'
> > > ... [-Wswitch]
> > > switch (DKind) {
> > > ^
> > > /usr/local/google/home/yitzhakm/remote-build/llvm-git/llvm-project/clang/lib/Parse/ParseOpenMP.cpp:2078:11: warning: 19 enumeration values not handled in switch: 'OMPD_distribute_parallel_do', 'OMPD_distribute_parallel_do_simd', 'OMPD_do'
> > > ... [-Wswitch]
> > > switch (DKind) {
> > > ^
> > >
> >
> >
> > Yes, sorry for that. I'm working on a fix to handle newly introduce directives/clauses. Should be ok tonight.
>
>
> If a fix is going to take more than a few minutes (a few hours/the rest of the day is pretty high) could you revert this patch & recommit with the fixes?
I reverted it. Will commit back later today.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82906/new/
https://reviews.llvm.org/D82906
More information about the llvm-commits
mailing list