[all-commits] [llvm/llvm-project] a8a10a: [openacc][openmp] Allow duplicate between required...

[Valentin Clement (バレンタイン クレメン) via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: a8a10acba2a769ae0f77e61380e649e7428f68fb
      https://github.com/llvm/llvm-project/commit/a8a10acba2a769ae0f77e61380e649e7428f68fb
  Author: Valentin Clement <clementval at gmail.com>
  Date:   2020-11-05 (Thu, 05 Nov 2020)

  Changed paths:
    M llvm/include/llvm/Frontend/Directive/DirectiveBase.td
    M llvm/test/TableGen/directive3.td
    M llvm/utils/TableGen/DirectiveEmitter.cpp

  Log Message:
  -----------
  [openacc][openmp] Allow duplicate between required and allowed once/exclusive

Validity check introduce in D90241 are a bit too restrict and this patch propose to losen
them a bit. The duplicate clauses is now check only between the three allowed lists and between the
requiredClauses and allowedClauses lists. This allows to enable some check where a clause can be
required but also appear only once on the directive. We found these kind of restriction useful
on the set directive in OpenACC for example.

Reviewed By: kiranchandramohan

Differential Revision: https://reviews.llvm.org/D90770