[PATCH] D83363: [openmp] Use switch in isAllowedClauseForDirective instead of multiple if

[Valentin Clement via Phabricator via llvm-commits]

clementval marked 2 inline comments as done.
clementval added inline comments.


================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMP.td:471
     VersionedClause<OMPC_NumThreads>,
-    VersionedClause<OMPC_DefaultMap>,
     VersionedClause<OMPC_ProcBind>,
----------------
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. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83363/new/

https://reviews.llvm.org/D83363