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

[Valentin Clement via Phabricator via llvm-commits]

clementval created this revision.
clementval added reviewers: jdoerfert, jdenny.
Herald added subscribers: llvm-commits, sstefan1, guansong, yaxunl.
Herald added a project: LLVM.

Change the test in isAllowedClauseForDirective from if with multiple conditions
to a main switch on directive and then switches on clause for each directive. Version
check is still done with a condition in the return statment.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D83363

Files:
  llvm/include/llvm/Frontend/OpenMP/OMP.td
  llvm/test/TableGen/directive1.td
  llvm/test/TableGen/directive2.td
  llvm/utils/TableGen/DirectiveEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83363.276289.patch
Type: text/x-patch
Size: 7583 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200708/a08eafa0/attachment.bin>