================ @@ -0,0 +1,89 @@ +//===--- CIRGenOpenMPClause.h - OpenMP clause emitter -----------*- C++ -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception +// +//===----------------------------------------------------------------------===// + +#ifndef LLVM_CLANG_LIB_CIR_CODEGEN_CIRGENOPENMPCLAUSE_H +#define LLVM_CLANG_LIB_CIR_CODEGEN_CIRGENOPENMPCLAUSE_H + +#include "CIRGenBuilder.h" +#include "CIRGenModule.h" +#include "mlir/Dialect/OpenMP/OpenMPClauseOperands.h" +#include "clang/AST/OpenMPClause.h" +#include "clang/AST/StmtOpenMP.h" +#include "llvm/Frontend/OpenMP/OMPConstants.h" + +#include <type_traits> + +namespace clang::CIRGen { + +class CIRGenFunction; + +/// A type-only list of OpenMP clause AST node types. +/// Note: The clause AST classes do not have a default constructor, so a +/// std::tuple is not practical. +template <typename... Clauses> struct OpenMPClauseList {}; ---------------- skatrak wrote:
One problem I have with the name of this helper type is that it implies that it can store clause information, though the description does state that it doesn't. Since its only use is to hold a list of types of not-yet-implemented clauses, I wonder if it would be better to name it `OpenMPNYIClauses` or similar. That way, it would be self-documenting wherever it's used. Feel free to ignore this suggestion if you don't think it would be an improvement. https://github.com/llvm/llvm-project/pull/195452 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
