[PATCH] D83363: [openmp] Use switch in isAllowedClauseForDirective instead of multiple if
Johannes Doerfert via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 7 19:15:20 PDT 2020
jdoerfert added inline comments.
================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMP.td:471
VersionedClause<OMPC_NumThreads>,
- VersionedClause<OMPC_DefaultMap>,
VersionedClause<OMPC_ProcBind>,
----------------
clementval wrote:
> clementval wrote:
> > jdoerfert wrote:
> > > [Drive By] Unrelated?
> > It was a duplicate. Already on line 469.
> Makes compilation error because of duplicate case when code is generated.
Commit NFC changes like that w/o review if it "just makes sense".
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83363/new/
https://reviews.llvm.org/D83363
More information about the llvm-commits
mailing list