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

[Johannes Doerfert via Phabricator via llvm-commits]

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