opt,
fake>;
-// An argument of a OMPDeclareVariantAttribute that represents the `match`
+// An argument of a OMPDeclareVariantAttr that represents the `match`
// clause of the declare variant by keeping the information (incl. nesting) in
// an OMPTraitInfo object.
//
OMPTraitInfoArgument<"TraitInfos">,
];
let AdditionalMembers = [{
- ~OMPDeclareVariantAttr() { delete traitInfos; }
+ OMPTraitInfo &getTraitInfo() { return traitInfos; }
void printPrettyPragma(raw_ostream & OS, const PrintingPolicy &Policy)
const {
if (const Expr *E = getVariantFuncRef()) {
OS << ")";
}
OS << " match(";
- traitInfos->print(OS, Policy);
+ traitInfos.print(OS, Policy);
OS << ")";
}
}];
/// must be used instead of the original one, specified in \p DG.
/// \param TI The context traits associated with the function variant.
void ActOnOpenMPDeclareVariantDirective(FunctionDecl *FD, Expr *VariantRef,
- OMPTraitInfo *TI, SourceRange SR);
+ OMPTraitInfo &TI, SourceRange SR);
OMPClause *ActOnOpenMPSingleExprClause(OpenMPClauseKind Kind,
Expr *Expr,
}
/// Read an OMPTraitInfo object, advancing Idx.
- OMPTraitInfo *readOMPTraitInfo();
+ OMPTraitInfo readOMPTraitInfo();
/// Read an OpenMP clause, advancing Idx.
OMPClause *readOMPClause();
void AddCXXDefinitionData(const CXXRecordDecl *D);
/// Write an OMPTraitInfo object.
- void writeOMPTraitInfo(OMPTraitInfo *TI);
+ void writeOMPTraitInfo(const OMPTraitInfo &TI);
void writeOMPClause(OMPClause *C);
SmallVector<Expr *, 8> VariantExprs;
SmallVector<VariantMatchInfo, 8> VMIs;
for (const auto *A : FD->specific_attrs<OMPDeclareVariantAttr>()) {
- const OMPTraitInfo *TI = A->getTraitInfos();
- if (!TI)
- continue;
+ A->printPretty(llvm::dbgs(), PrintingPolicy(LangOptions()));
+ const OMPTraitInfo &TI = A->getTraitInfos();
+ TI.print(llvm::dbgs(), PrintingPolicy(LangOptions()));
VMIs.push_back(VariantMatchInfo());
- TI->getAsVariantMatchInfo(CGM.getContext(), VMIs.back());
+ TI.getAsVariantMatchInfo(CGM.getContext(), VMIs.back());
VariantExprs.push_back(A->getVariantFuncRef());
}
}
// Parse inner context selectors.
- OMPTraitInfo *TI = new OMPTraitInfo();
- parseOMPContextSelectors(Loc, *TI);
+ OMPTraitInfo TI;
+ parseOMPContextSelectors(Loc, TI);
// Parse ')'
(void)T.consumeClose();
Optional<std::pair<FunctionDecl *, Expr *>> DeclVarData =
Actions.checkOpenMPDeclareVariantFunction(
- Ptr, AssociatedFunction.get(), *TI,
+ Ptr, AssociatedFunction.get(), TI,
SourceRange(Loc, Tok.getLocation()));
// Skip last tokens.
while (Tok.isNot(tok::annot_pragma_openmp_end))
ConsumeAnyToken();
- if (DeclVarData.hasValue() && !TI->Sets.empty())
+ if (DeclVarData.hasValue() && !TI.Sets.empty())
Actions.ActOnOpenMPDeclareVariantDirective(
DeclVarData.getValue().first, DeclVarData.getValue().second, TI,
SourceRange(Loc, Tok.getLocation()));
- else
- delete TI;
// Skip the last annot_pragma_openmp_end.
(void)ConsumeAnnotationToken();
void Sema::ActOnOpenMPDeclareVariantDirective(FunctionDecl *FD,
Expr *VariantRef,
- OMPTraitInfo *TI,
+ OMPTraitInfo &TI,
SourceRange SR) {
auto *NewAttr =
OMPDeclareVariantAttr::CreateImplicit(Context, VariantRef, TI, SR);
// Copy the template version of the OMPTraitInfo and run substitute on all
// score and condition expressiosn.
- OMPTraitInfo *TI = new OMPTraitInfo();
- *TI = *Attr.getTraitInfos();
+ OMPTraitInfo TI = Attr.getTraitInfos();
// Try to substitute template parameters in score and condition expressions.
auto SubstScoreOrConditionExpr = [&S, Subst](Expr *&E, bool) {
}
return false;
};
- if (TI->anyScoreOrCondition(SubstScoreOrConditionExpr)) {
- delete TI;
+ if (TI.anyScoreOrCondition(SubstScoreOrConditionExpr))
return;
- }
// Check function/variant ref.
Optional<std::pair<FunctionDecl *, Expr *>> DeclVarData =
S.checkOpenMPDeclareVariantFunction(S.ConvertDeclToDeclGroup(New),
- VariantFuncRef.get(), *TI,
+ VariantFuncRef.get(), TI,
Attr.getRange());
- if (!DeclVarData) {
- delete TI;
+ if (!DeclVarData)
return;
- }
S.ActOnOpenMPDeclareVariantDirective(DeclVarData.getValue().first,
DeclVarData.getValue().second, TI,
C->setKindKwLoc(Record.readSourceLocation());
}
-OMPTraitInfo *ASTRecordReader::readOMPTraitInfo() {
- OMPTraitInfo *TI = new OMPTraitInfo();
- TI->Sets.resize(readUInt32());
- for (auto &Set : TI->Sets) {
+OMPTraitInfo ASTRecordReader::readOMPTraitInfo() {
+ OMPTraitInfo TI;
+ TI.Sets.resize(readUInt32());
+ for (auto &Set : TI.Sets) {
Set.Kind = readEnum<llvm::omp::TraitSet>();
Set.Selectors.resize(readUInt32());
for (auto &Selector : Set.Selectors) {
return Reader.readVersionTuple();
}
- OMPTraitInfo *readOMPTraitInfo() { return Reader.readOMPTraitInfo(); }
+ OMPTraitInfo readOMPTraitInfo() { return Reader.readOMPTraitInfo(); }
template <typename T> T *GetLocalDeclAs(uint32_t LocalID) {
return Reader.GetLocalDeclAs<T>(LocalID);
Record.AddSourceLocation(C->getKindKwLoc());
}
-void ASTRecordWriter::writeOMPTraitInfo(OMPTraitInfo *TI) {
- writeUInt32(TI->Sets.size());
- for (const auto &Set : TI->Sets) {
+void ASTRecordWriter::writeOMPTraitInfo(const OMPTraitInfo &TI) {
+ writeUInt32(TI.Sets.size());
+ for (const auto &Set : TI.Sets) {
writeEnum(Set.Kind);
writeUInt32(Set.Selectors.size());
for (const auto &Selector : Set.Selectors) {
.Case("IdentifierInfo *", "Record.readIdentifier()")
.Case("StringRef", "Record.readString()")
.Case("ParamIdx", "ParamIdx::deserialize(Record.readInt())")
- .Case("OMPTraitInfo *", "Record.readOMPTraitInfo()")
+ .Case("OMPTraitInfo", "Record.readOMPTraitInfo()")
.Default("Record.readInt()");
}
.Case("StringRef", "AddString(" + std::string(name) + ");\n")
.Case("ParamIdx",
"push_back(" + std::string(name) + ".serialize());\n")
- .Case("OMPTraitInfo *",
+ .Case("OMPTraitInfo",
"writeOMPTraitInfo(" + std::string(name) + ");\n")
.Default("push_back(" + std::string(name) + ");\n");
}
OS << " if (SA->get" << getUpperName() << "().isValid())\n ";
OS << " OS << \" \" << SA->get" << getUpperName()
<< "().getSourceIndex();\n";
- } else if (type == "OMPTraitInfo *") {
- OS << " OS << \" \" << *SA->get" << getUpperName() << "();\n";
+ } else if (type == "OMPTraitInfo") {
+ OS << " OS << \" \" << SA->get" << getUpperName() << "();\n";
} else {
llvm_unreachable("Unknown SimpleArgument type!");
}
else if (ArgName == "VersionArgument")
Ptr = std::make_unique<VersionArgument>(Arg, Attr);
else if (ArgName == "OMPTraitInfoArgument")
- Ptr = std::make_unique<SimpleArgument>(Arg, Attr, "OMPTraitInfo *");
+ Ptr = std::make_unique<SimpleArgument>(Arg, Attr, "OMPTraitInfo");
if (!Ptr) {
// Search in reverse order so that the most-derived type is handled first.