diff --git a/clang/include/clang/AST/OpenMPClause.h b/clang/include/clang/AST/OpenMPClause.h --- a/clang/include/clang/AST/OpenMPClause.h +++ b/clang/include/clang/AST/OpenMPClause.h @@ -7229,11 +7229,9 @@ /// former is a flat representation the actual main difference is that the /// latter uses clang::Expr to store the score/condition while the former is /// independent of clang. Thus, expressions and conditions are evaluated in - /// this method. If \p DeviceSetOnly is true, only the device selector set, if - /// present, is put in \p VMI, otherwise all selector sets are put in \p VMI. + /// this method. void getAsVariantMatchInfo(ASTContext &ASTCtx, - llvm::omp::VariantMatchInfo &VMI, - bool DeviceSetOnly) const; + llvm::omp::VariantMatchInfo &VMI) const; /// Return a string representation identifying this context selector. std::string getMangledName() const; diff --git a/clang/include/clang/Basic/AttrDocs.td b/clang/include/clang/Basic/AttrDocs.td --- a/clang/include/clang/Basic/AttrDocs.td +++ b/clang/include/clang/Basic/AttrDocs.td @@ -3394,6 +3394,20 @@ and where `variant-func-id` is the name of a function variant that is either a base language identifier or, for C++, a template-id. +Clang provides the following context selector extensions, used via `implementation={extension(EXTENSION)}`: + + .. code-block:: none + + match_all + match_any + match_none + +The match extensions change when the *entire* context selector is considered a +match for an OpenMP context. The default is `all`, with `none` no trait in the +selector is allowed to be in the OpenMP context, with `any` a single trait in +both the selector and OpenMP context is sufficient. Only a single match +extension trait is allowed per context selector. + }]; } diff --git a/clang/include/clang/Basic/DiagnosticParseKinds.td b/clang/include/clang/Basic/DiagnosticParseKinds.td --- a/clang/include/clang/Basic/DiagnosticParseKinds.td +++ b/clang/include/clang/Basic/DiagnosticParseKinds.td @@ -1313,6 +1313,8 @@ def warn_omp_more_one_device_type_clause : Warning<"more than one 'device_type' clause is specified">, InGroup; +def err_omp_variant_ctx_second_match_extension : Error< + "only a single match extension allowed per OpenMP context selector">; // Pragma loop support. def err_pragma_loop_missing_argument : Error< diff --git a/clang/lib/AST/OpenMPClause.cpp b/clang/lib/AST/OpenMPClause.cpp --- a/clang/lib/AST/OpenMPClause.cpp +++ b/clang/lib/AST/OpenMPClause.cpp @@ -1883,11 +1883,8 @@ } void OMPTraitInfo::getAsVariantMatchInfo(ASTContext &ASTCtx, - VariantMatchInfo &VMI, - bool DeviceSetOnly) const { + VariantMatchInfo &VMI) const { for (const OMPTraitSet &Set : Sets) { - if (DeviceSetOnly && Set.Kind != TraitSet::device) - continue; for (const OMPTraitSelector &Selector : Set.Selectors) { // User conditions are special as we evaluate the condition here. @@ -1899,20 +1896,25 @@ TraitProperty::user_condition_unknown && "Ill-formed user condition, expected unknown trait property!"); - llvm::APInt CondVal = - Selector.ScoreOrCondition->EvaluateKnownConstInt(ASTCtx); - VMI.addTrait(CondVal.isNullValue() - ? TraitProperty::user_condition_false - : TraitProperty::user_condition_true); + llvm::APSInt CondVal; + if (Selector.ScoreOrCondition->isIntegerConstantExpr(CondVal, ASTCtx)) + VMI.addTrait(CondVal.isNullValue() + ? TraitProperty::user_condition_false + : TraitProperty::user_condition_true); + else + VMI.addTrait(TraitProperty::user_condition_false); continue; } - llvm::APInt Score; + llvm::APSInt Score; llvm::APInt *ScorePtr = nullptr; if (Selector.ScoreOrCondition) { - Score = Selector.ScoreOrCondition->EvaluateKnownConstInt(ASTCtx); - ScorePtr = &Score; + if (Selector.ScoreOrCondition->isIntegerConstantExpr(Score, ASTCtx)) + ScorePtr = &Score; + else + VMI.addTrait(TraitProperty::user_condition_false); } + for (const OMPTraitProperty &Property : Selector.Properties) VMI.addTrait(Set.Kind, Property.Kind, ScorePtr); diff --git a/clang/lib/Parse/ParseOpenMP.cpp b/clang/lib/Parse/ParseOpenMP.cpp --- a/clang/lib/Parse/ParseOpenMP.cpp +++ b/clang/lib/Parse/ParseOpenMP.cpp @@ -935,6 +935,43 @@ << CONTEXT_TRAIT_LVL << listOpenMPContextTraitProperties(Set, Selector); } +static bool checkExtensionProperty(Parser &P, SourceLocation Loc, + OMPTraitProperty &TIProperty, + OMPTraitSelector &TISelector, + llvm::StringMap &Seen) { + assert(TISelector.Kind == + llvm::omp::TraitSelector::implementation_extension && + "Only for extension properties, e.g., " + "`implementation={extension(PROPERTY)}`"); + if (TIProperty.Kind == TraitProperty::invalid) + return false; + + auto IsMatchExtension = [](OMPTraitProperty &TP) { + return (TP.Kind == + llvm::omp::TraitProperty::implementation_extension_match_all || + TP.Kind == + llvm::omp::TraitProperty::implementation_extension_match_any || + TP.Kind == + llvm::omp::TraitProperty::implementation_extension_match_none); + }; + + if (IsMatchExtension(TIProperty)) { + for (OMPTraitProperty &SeenProp : TISelector.Properties) + if (IsMatchExtension(SeenProp)) { + P.Diag(Loc, diag::err_omp_variant_ctx_second_match_extension); + StringRef SeenName = + llvm::omp::getOpenMPContextTraitPropertyName(SeenProp.Kind); + SourceLocation SeenLoc = Seen[SeenName]; + P.Diag(SeenLoc, diag::note_omp_declare_variant_ctx_used_here) + << CONTEXT_TRAIT_LVL << SeenName; + return false; + } + return true; + } + + llvm_unreachable("Unknown extension property!"); +} + void Parser::parseOMPContextProperty(OMPTraitSelector &TISelector, llvm::omp::TraitSet Set, llvm::StringMap &Seen) { @@ -945,6 +982,11 @@ OMPTraitProperty TIProperty; parseOMPTraitPropertyKind(TIProperty, Set, TISelector.Kind, Seen); + if (TISelector.Kind == llvm::omp::TraitSelector::implementation_extension) + if (!checkExtensionProperty(*this, Tok.getLocation(), TIProperty, + TISelector, Seen)) + TIProperty.Kind = TraitProperty::invalid; + // If we have an invalid property here we already issued a warning. if (TIProperty.Kind == TraitProperty::invalid) { if (PropertyLoc != Tok.getLocation()) @@ -955,6 +997,7 @@ if (isValidTraitPropertyForTraitSetAndSelector(TIProperty.Kind, TISelector.Kind, Set)) { + // If we make it here the property, selector, set, score, condition, ... are // all valid (or have been corrected). Thus we can record the property. TISelector.Properties.push_back(TIProperty); @@ -1783,11 +1826,11 @@ VariantMatchInfo VMI; ASTContext &ASTCtx = Actions.getASTContext(); - TI.getAsVariantMatchInfo(ASTCtx, VMI, /* DeviceSetOnly */ true); + TI.getAsVariantMatchInfo(ASTCtx, VMI); OMPContext OMPCtx(ASTCtx.getLangOpts().OpenMPIsDevice, ASTCtx.getTargetInfo().getTriple()); - if (isVariantApplicableInContext(VMI, OMPCtx)) { + if (isVariantApplicableInContext(VMI, OMPCtx, /* DeviceSetOnly */ true)) { Actions.ActOnOpenMPBeginDeclareVariant(Loc, TI); break; } diff --git a/clang/lib/Sema/SemaOpenMP.cpp b/clang/lib/Sema/SemaOpenMP.cpp --- a/clang/lib/Sema/SemaOpenMP.cpp +++ b/clang/lib/Sema/SemaOpenMP.cpp @@ -5650,8 +5650,8 @@ VariantMatchInfo VMI; OMPTraitInfo &TI = A->getTraitInfo(); - TI.getAsVariantMatchInfo(Context, VMI, /* DeviceSetOnly */ false); - if (!isVariantApplicableInContext(VMI, OMPCtx)) + TI.getAsVariantMatchInfo(Context, VMI); + if (!isVariantApplicableInContext(VMI, OMPCtx, /* DeviceSetOnly */ false)) continue; VMIs.push_back(VMI); diff --git a/clang/test/AST/ast-dump-openmp-declare-variant-extensions-messages.c b/clang/test/AST/ast-dump-openmp-declare-variant-extensions-messages.c new file mode 100644 --- /dev/null +++ b/clang/test/AST/ast-dump-openmp-declare-variant-extensions-messages.c @@ -0,0 +1,18 @@ +// RUN: %clang_cc1 -triple x86_64-unknown-unknown -fopenmp -verify %s +// RUN: %clang_cc1 -triple x86_64-unknown-unknown -fopenmp -verify %s -x c++ + +int dummy() { return 1; } + +#pragma omp declare variant(dummy) match(implementation={extension(match_any,match_all)}, device={kind(cpu, gpu)}) // expected-error {{only a single match extension allowed per OpenMP context selector}} expected-note {{the previous context property 'match_any' used here}} // expected-note {{the ignored property spans until here}} +int base1() { return 2; } + +#pragma omp declare variant(dummy) match(implementation={extension(match_none,match_none)}, device={kind(gpu, fpga)}) // expected-warning {{the context property 'match_none' was used already in the same 'omp declare variant' directive; property ignored}} expected-note {{the previous context property 'match_none' used here}} expected-note {{the ignored property spans until here}} +int base2() { return 3; } + +#pragma omp declare variant(dummy) match(implementation={vendor(pgi), extension(match_none,match_any)}, device={kind(cpu, gpu)}) // expected-error {{only a single match extension allowed per OpenMP context selector}} expected-note {{the previous context property 'match_none' used here}} // expected-note {{the ignored property spans until here}} +int base3() { return 4; } + + +int test() { + return base1() + base2() + base3(); +} diff --git a/clang/test/AST/ast-dump-openmp-declare-variant-extensions.c b/clang/test/AST/ast-dump-openmp-declare-variant-extensions.c new file mode 100644 --- /dev/null +++ b/clang/test/AST/ast-dump-openmp-declare-variant-extensions.c @@ -0,0 +1,343 @@ +// RUN: %clang_cc1 -triple x86_64-unknown-unknown -fopenmp -verify -ast-dump %s | FileCheck %s +// RUN: %clang_cc1 -triple x86_64-unknown-unknown -fopenmp -verify -ast-dump %s -x c++| FileCheck %s +// expected-no-diagnostics + +int picked1() { return 0; } +int picked2() { return 0; } +int picked3(); +int picked4(); +int picked5() { return 0; } +int picked6() { return 0; } +int picked7() { return 0; } +int not_picked1() { return 1; } +int not_picked2() { return 2; } +int not_picked3(); +int not_picked4(); +int not_picked5(); +int not_picked6(); + +#pragma omp declare variant(picked1) match(implementation={extension(match_any)}, device={kind(cpu, gpu)}) +int base1() { return 3; } + +#pragma omp declare variant(picked2) match(implementation={extension(match_none)}, device={kind(gpu, fpga)}) +int base2() { return 4; } + +#pragma omp declare variant(picked3) match(implementation={vendor(pgi), extension(match_any)}, device={kind(cpu, gpu)}) +int base3() { return 5; } + +#pragma omp declare variant(picked4) match(user={condition(0)}, implementation={extension(match_none)}, device={kind(gpu, fpga)}) +int base4() { return 6; } + +#pragma omp declare variant(picked5) match(user={condition(1)}, implementation={extension(match_all)}, device={kind(cpu)}) +int base5() { return 7; } + +#pragma omp declare variant(not_picked1) match(implementation={extension(match_any)}, device={kind(gpu, fpga)}) +int base6() { return 0; } + +#pragma omp declare variant(not_picked2) match(implementation={extension(match_none)}, device={kind(gpu, cpu)}) +int base7() { return 0; } + +#pragma omp declare variant(not_picked3) match(implementation={vendor(llvm), extension(match_any)}, device={kind(fpga, gpu)}) +int base8() { return 0; } + +#pragma omp declare variant(not_picked4) match(user={condition(1)}, implementation={extension(match_none)}, device={kind(gpu, fpga)}) +int base9() { return 0; } + +#pragma omp declare variant(not_picked5) match(user={condition(1)}, implementation={extension(match_all)}, device={kind(cpu, gpu)}) +int base10() { return 0; } + +#pragma omp declare variant(not_picked6) match(implementation={extension(match_any)}) +int base11() { return 0; } + +#pragma omp declare variant(picked6) match(implementation={extension(match_all)}) +int base12() { return 8; } + +#pragma omp declare variant(picked7) match(implementation={extension(match_none)}) +int base13() { return 9; } + +#pragma omp begin declare variant match(implementation={extension(match_any)}, device={kind(cpu, gpu)}) +int overloaded1() { return 0; } +#pragma omp end declare variant + +int overloaded2() { return 1; } +#pragma omp begin declare variant match(implementation={extension(match_none)}, device={kind(fpga, gpu)}) +int overloaded2() { return 0; } +#pragma omp end declare variant + +#pragma omp begin declare variant match(implementation={extension(match_none)}, device={kind(cpu)}) +NOT PARSED +#pragma omp end declare variant + + +int picked3() { return 0; } +int picked4() { return 0; } +int not_picked3() { return 10; } +int not_picked4() { return 11; } +int not_picked5() { return 12; } +int not_picked6() { return 13; } + +int test() { + // Should return 0. + return base1() + base2() + base3() + base4() + base5() + base6() + base7() + + base8() + base9() + base10() + base11() + base12() + base13() + + overloaded1() + overloaded2(); +} + +// 1) All "picked" versions are called but none of the "non_picked" ones is. +// 2) The overloaded functions that return 0 are called. + +// CHECK: |-FunctionDecl [[ADDR_0:0x[a-z0-9]*]] <{{.*}}, col:27> col:5 referenced picked1 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_1:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_2:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_3:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_4:0x[a-z0-9]*]] col:5 referenced picked2 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_5:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_6:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_7:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_8:0x[a-z0-9]*]] col:5 referenced picked3 'int ({{.*}})' +// CHECK-NEXT: |-FunctionDecl [[ADDR_9:0x[a-z0-9]*]] col:5 referenced picked4 'int ({{.*}})' +// CHECK-NEXT: |-FunctionDecl [[ADDR_10:0x[a-z0-9]*]] col:5 referenced picked5 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_11:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_12:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_13:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_14:0x[a-z0-9]*]] col:5 referenced picked6 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_15:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_16:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_17:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_18:0x[a-z0-9]*]] col:5 referenced picked7 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_19:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_20:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_21:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_22:0x[a-z0-9]*]] col:5 referenced not_picked1 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_23:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_24:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_25:0x[a-z0-9]*]] 'int' 1 +// CHECK-NEXT: |-FunctionDecl [[ADDR_26:0x[a-z0-9]*]] col:5 referenced not_picked2 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_27:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_28:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_29:0x[a-z0-9]*]] 'int' 2 +// CHECK-NEXT: |-FunctionDecl [[ADDR_30:0x[a-z0-9]*]] col:5 referenced not_picked3 'int ({{.*}})' +// CHECK-NEXT: |-FunctionDecl [[ADDR_31:0x[a-z0-9]*]] col:5 referenced not_picked4 'int ({{.*}})' +// CHECK-NEXT: |-FunctionDecl [[ADDR_32:0x[a-z0-9]*]] col:5 referenced not_picked5 'int ({{.*}})' +// CHECK-NEXT: |-FunctionDecl [[ADDR_33:0x[a-z0-9]*]] col:5 referenced not_picked6 'int ({{.*}})' +// CHECK-NEXT: |-FunctionDecl [[ADDR_34:0x[a-z0-9]*]] col:5 used base1 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_35:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_36:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_37:0x[a-z0-9]*]] 'int' 3 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_38:0x[a-z0-9]*]] Implicit implementation={extension(match_any)}, device={kind(cpu, gpu)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_39:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_0]] 'picked1' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_40:0x[a-z0-9]*]] col:5 used base2 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_41:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_42:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_43:0x[a-z0-9]*]] 'int' 4 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_44:0x[a-z0-9]*]] Implicit implementation={extension(match_none)}, device={kind(gpu, fpga)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_45:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_4]] 'picked2' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_46:0x[a-z0-9]*]] col:5 used base3 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_47:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_48:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_49:0x[a-z0-9]*]] 'int' 5 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_50:0x[a-z0-9]*]] Implicit implementation={vendor(pgi), extension(match_any)}, device={kind(cpu, gpu)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_51:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_8]] 'picked3' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_52:0x[a-z0-9]*]] col:5 used base4 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_53:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_54:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_55:0x[a-z0-9]*]] 'int' 6 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_56:0x[a-z0-9]*]] Implicit user={condition(0)}, implementation={extension(match_none)}, device={kind(gpu, fpga)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_57:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_9]] 'picked4' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_58:0x[a-z0-9]*]] col:5 used base5 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_59:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_60:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_61:0x[a-z0-9]*]] 'int' 7 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_62:0x[a-z0-9]*]] Implicit user={condition(1)}, implementation={extension(match_all)}, device={kind(cpu)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_63:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_10]] 'picked5' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_64:0x[a-z0-9]*]] col:5 used base6 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_65:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_66:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_67:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_68:0x[a-z0-9]*]] Implicit implementation={extension(match_any)}, device={kind(gpu, fpga)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_69:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_22]] 'not_picked1' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_70:0x[a-z0-9]*]] col:5 used base7 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_71:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_72:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_73:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_74:0x[a-z0-9]*]] Implicit implementation={extension(match_none)}, device={kind(gpu, cpu)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_75:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_26]] 'not_picked2' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_76:0x[a-z0-9]*]] col:5 used base8 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_77:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_78:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_79:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_80:0x[a-z0-9]*]] Implicit implementation={vendor(llvm), extension(match_any)}, device={kind(fpga, gpu)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_81:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_30]] 'not_picked3' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_82:0x[a-z0-9]*]] col:5 used base9 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_83:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_84:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_85:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_86:0x[a-z0-9]*]] Implicit user={condition(1)}, implementation={extension(match_none)}, device={kind(gpu, fpga)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_87:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_31]] 'not_picked4' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_88:0x[a-z0-9]*]] col:5 used base10 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_89:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_90:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_91:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_92:0x[a-z0-9]*]] Implicit user={condition(1)}, implementation={extension(match_all)}, device={kind(cpu, gpu)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_93:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_32]] 'not_picked5' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_94:0x[a-z0-9]*]] col:5 used base11 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_95:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_96:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_97:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_98:0x[a-z0-9]*]] Implicit implementation={extension(match_any)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_99:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_33]] 'not_picked6' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_100:0x[a-z0-9]*]] col:5 used base12 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_101:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_102:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_103:0x[a-z0-9]*]] 'int' 8 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_104:0x[a-z0-9]*]] Implicit implementation={extension(match_all)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_105:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_14]] 'picked6' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_106:0x[a-z0-9]*]] col:5 used base13 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_107:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_108:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_109:0x[a-z0-9]*]] 'int' 9 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_110:0x[a-z0-9]*]] Implicit implementation={extension(match_none)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_111:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_18]] 'picked7' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: |-FunctionDecl [[ADDR_112:0x[a-z0-9]*]] col:5 implicit used overloaded1 'int ({{.*}})' +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_113:0x[a-z0-9]*]] <> Implicit implementation={extension(match_any)}, device={kind(cpu, gpu)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_114:0x[a-z0-9]*]] 'int ({{.*}})' Function [[ADDR_115:0x[a-z0-9]*]] 'overloaded1[implementation={extension(match_any)}]' 'int ({{.*}})' +// CHECK-NEXT: |-FunctionDecl [[ADDR_115]] col:1 overloaded1[implementation={extension(match_any)}] 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_116:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_117:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_118:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_119:0x[a-z0-9]*]] col:5 used overloaded2 'int ({{.*}})' +// CHECK-NEXT: | |-CompoundStmt [[ADDR_120:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-ReturnStmt [[ADDR_121:0x[a-z0-9]*]] +// CHECK-NEXT: | | `-IntegerLiteral [[ADDR_122:0x[a-z0-9]*]] 'int' 1 +// CHECK-NEXT: | `-OMPDeclareVariantAttr [[ADDR_123:0x[a-z0-9]*]] <> Implicit implementation={extension(match_none)}, device={kind(fpga, gpu)} +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_124:0x[a-z0-9]*]] 'int ({{.*}})' Function [[ADDR_125:0x[a-z0-9]*]] 'overloaded2[implementation={extension(match_none)}]' 'int ({{.*}})' +// CHECK-NEXT: |-FunctionDecl [[ADDR_125]] col:1 overloaded2[implementation={extension(match_none)}] 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_126:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_127:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_128:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_129:0x[a-z0-9]*]] prev [[ADDR_8]] col:5 picked3 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_130:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_131:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_132:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_133:0x[a-z0-9]*]] prev [[ADDR_9]] col:5 picked4 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_134:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_135:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_136:0x[a-z0-9]*]] 'int' 0 +// CHECK-NEXT: |-FunctionDecl [[ADDR_137:0x[a-z0-9]*]] prev [[ADDR_30]] col:5 not_picked3 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_138:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_139:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_140:0x[a-z0-9]*]] 'int' 10 +// CHECK-NEXT: |-FunctionDecl [[ADDR_141:0x[a-z0-9]*]] prev [[ADDR_31]] col:5 not_picked4 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_142:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_143:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_144:0x[a-z0-9]*]] 'int' 11 +// CHECK-NEXT: |-FunctionDecl [[ADDR_145:0x[a-z0-9]*]] prev [[ADDR_32]] col:5 not_picked5 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_146:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_147:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_148:0x[a-z0-9]*]] 'int' 12 +// CHECK-NEXT: |-FunctionDecl [[ADDR_149:0x[a-z0-9]*]] prev [[ADDR_33]] col:5 not_picked6 'int ({{.*}})' +// CHECK-NEXT: | `-CompoundStmt [[ADDR_150:0x[a-z0-9]*]] +// CHECK-NEXT: | `-ReturnStmt [[ADDR_151:0x[a-z0-9]*]] +// CHECK-NEXT: | `-IntegerLiteral [[ADDR_152:0x[a-z0-9]*]] 'int' 13 +// CHECK-NEXT: `-FunctionDecl [[ADDR_153:0x[a-z0-9]*]] line:79:5 test 'int ({{.*}})' +// CHECK-NEXT: `-CompoundStmt [[ADDR_154:0x[a-z0-9]*]] +// CHECK-NEXT: `-ReturnStmt [[ADDR_155:0x[a-z0-9]*]] +// CHECK-NEXT: `-BinaryOperator [[ADDR_156:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: |-BinaryOperator [[ADDR_157:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | |-BinaryOperator [[ADDR_158:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | |-BinaryOperator [[ADDR_159:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | |-BinaryOperator [[ADDR_160:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | |-BinaryOperator [[ADDR_161:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | |-BinaryOperator [[ADDR_162:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | | |-BinaryOperator [[ADDR_163:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | | | |-BinaryOperator [[ADDR_164:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | | | | |-BinaryOperator [[ADDR_165:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | | | | | |-BinaryOperator [[ADDR_166:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | | | | | | |-BinaryOperator [[ADDR_167:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | | | | | | | |-BinaryOperator [[ADDR_168:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | | | | | | | | |-BinaryOperator [[ADDR_169:0x[a-z0-9]*]] 'int' '+' +// CHECK-NEXT: | | | | | | | | | | | | | |-PseudoObjectExpr [[ADDR_170:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | | | |-CallExpr [[ADDR_171:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_172:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | | | | `-DeclRefExpr [[ADDR_173:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_34]] 'base1' 'int ({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | | | `-CallExpr [[ADDR_174:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_175:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | | | `-DeclRefExpr [[ADDR_39]] 'int ({{.*}})' {{.*}}Function [[ADDR_0]] 'picked1' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: | | | | | | | | | | | | | `-PseudoObjectExpr [[ADDR_176:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | | |-CallExpr [[ADDR_177:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_178:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | | | `-DeclRefExpr [[ADDR_179:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_40]] 'base2' 'int ({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | | `-CallExpr [[ADDR_180:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_181:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | | `-DeclRefExpr [[ADDR_45]] 'int ({{.*}})' {{.*}}Function [[ADDR_4]] 'picked2' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: | | | | | | | | | | | | `-PseudoObjectExpr [[ADDR_182:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | |-CallExpr [[ADDR_183:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_184:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | | `-DeclRefExpr [[ADDR_185:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_46]] 'base3' 'int ({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | `-CallExpr [[ADDR_186:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_187:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | `-DeclRefExpr [[ADDR_51]] 'int ({{.*}})' {{.*}}Function [[ADDR_8]] 'picked3' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: | | | | | | | | | | | `-PseudoObjectExpr [[ADDR_188:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | |-CallExpr [[ADDR_189:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_190:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | | `-DeclRefExpr [[ADDR_191:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_52]] 'base4' 'int ({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | `-CallExpr [[ADDR_192:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_193:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | `-DeclRefExpr [[ADDR_57]] 'int ({{.*}})' {{.*}}Function [[ADDR_9]] 'picked4' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: | | | | | | | | | | `-PseudoObjectExpr [[ADDR_194:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | |-CallExpr [[ADDR_195:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | | `-ImplicitCastExpr [[ADDR_196:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | | `-DeclRefExpr [[ADDR_197:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_58]] 'base5' 'int ({{.*}})' +// CHECK-NEXT: | | | | | | | | | | `-CallExpr [[ADDR_198:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | | `-ImplicitCastExpr [[ADDR_199:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | | `-DeclRefExpr [[ADDR_63]] 'int ({{.*}})' {{.*}}Function [[ADDR_10]] 'picked5' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: | | | | | | | | | `-CallExpr [[ADDR_200:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | | `-ImplicitCastExpr [[ADDR_201:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | | `-DeclRefExpr [[ADDR_202:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_64]] 'base6' 'int ({{.*}})' +// CHECK-NEXT: | | | | | | | | `-CallExpr [[ADDR_203:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | `-ImplicitCastExpr [[ADDR_204:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | `-DeclRefExpr [[ADDR_205:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_70]] 'base7' 'int ({{.*}})' +// CHECK-NEXT: | | | | | | | `-PseudoObjectExpr [[ADDR_206:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | |-CallExpr [[ADDR_207:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | | `-ImplicitCastExpr [[ADDR_208:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | | `-DeclRefExpr [[ADDR_209:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_76]] 'base8' 'int ({{.*}})' +// CHECK-NEXT: | | | | | | | `-CallExpr [[ADDR_210:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | | `-ImplicitCastExpr [[ADDR_211:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | | `-DeclRefExpr [[ADDR_81]] 'int ({{.*}})' {{.*}}Function [[ADDR_30]] 'not_picked3' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: | | | | | | `-CallExpr [[ADDR_212:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | | `-ImplicitCastExpr [[ADDR_213:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | | `-DeclRefExpr [[ADDR_214:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_82]] 'base9' 'int ({{.*}})' +// CHECK-NEXT: | | | | | `-CallExpr [[ADDR_215:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | | `-ImplicitCastExpr [[ADDR_216:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | | `-DeclRefExpr [[ADDR_217:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_88]] 'base10' 'int ({{.*}})' +// CHECK-NEXT: | | | | `-CallExpr [[ADDR_218:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | `-ImplicitCastExpr [[ADDR_219:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | `-DeclRefExpr [[ADDR_220:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_94]] 'base11' 'int ({{.*}})' +// CHECK-NEXT: | | | `-PseudoObjectExpr [[ADDR_221:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | |-CallExpr [[ADDR_222:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | | `-ImplicitCastExpr [[ADDR_223:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | | `-DeclRefExpr [[ADDR_224:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_100]] 'base12' 'int ({{.*}})' +// CHECK-NEXT: | | | `-CallExpr [[ADDR_225:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | `-ImplicitCastExpr [[ADDR_226:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | `-DeclRefExpr [[ADDR_105]] 'int ({{.*}})' {{.*}}Function [[ADDR_14]] 'picked6' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: | | `-PseudoObjectExpr [[ADDR_227:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | |-CallExpr [[ADDR_228:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | | `-ImplicitCastExpr [[ADDR_229:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | | `-DeclRefExpr [[ADDR_230:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_106]] 'base13' 'int ({{.*}})' +// CHECK-NEXT: | | `-CallExpr [[ADDR_231:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | `-ImplicitCastExpr [[ADDR_232:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | `-DeclRefExpr [[ADDR_111]] 'int ({{.*}})' {{.*}}Function [[ADDR_18]] 'picked7' 'int ({{.*}})' non_odr_use_unevaluated +// CHECK-NEXT: | `-PseudoObjectExpr [[ADDR_233:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | |-CallExpr [[ADDR_234:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | | `-ImplicitCastExpr [[ADDR_235:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | | `-DeclRefExpr [[ADDR_236:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_112]] 'overloaded1' 'int ({{.*}})' +// CHECK-NEXT: | `-CallExpr [[ADDR_237:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | `-ImplicitCastExpr [[ADDR_238:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_114]] 'int ({{.*}})' Function [[ADDR_115]] 'overloaded1[implementation={extension(match_any)}]' 'int ({{.*}})' +// CHECK-NEXT: `-PseudoObjectExpr [[ADDR_239:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: |-CallExpr [[ADDR_240:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: | `-ImplicitCastExpr [[ADDR_241:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: | `-DeclRefExpr [[ADDR_242:0x[a-z0-9]*]] 'int ({{.*}})' {{.*}}Function [[ADDR_119]] 'overloaded2' 'int ({{.*}})' +// CHECK-NEXT: `-CallExpr [[ADDR_243:0x[a-z0-9]*]] 'int' +// CHECK-NEXT: `-ImplicitCastExpr [[ADDR_244:0x[a-z0-9]*]] 'int (*)({{.*}})' +// CHECK-NEXT: `-DeclRefExpr [[ADDR_124]] 'int ({{.*}})' Function [[ADDR_125]] 'overloaded2[implementation={extension(match_none)}]' 'int ({{.*}})' diff --git a/clang/test/OpenMP/declare_variant_ast_print.c b/clang/test/OpenMP/declare_variant_ast_print.c --- a/clang/test/OpenMP/declare_variant_ast_print.c +++ b/clang/test/OpenMP/declare_variant_ast_print.c @@ -14,9 +14,15 @@ #pragma omp declare variant(foo) match(implementation={vendor(score(5): ibm, xxx, ibm)}, device={kind(cpu, nohost)}) #pragma omp declare variant(foo) match(device={kind(host)}) #pragma omp declare variant(foo) match(device={kind(nohost), xxx}) +#pragma omp declare variant(foo) match(implementation={extension(match_all)}) +#pragma omp declare variant(foo) match(implementation={extension(match_any)}) +#pragma omp declare variant(foo) match(implementation={extension(match_none)}) int bar(void); // CHECK: int foo(); +// CHECK-NEXT: #pragma omp declare variant(foo) match(implementation={extension(match_none)}) +// CHECK-NEXT: #pragma omp declare variant(foo) match(implementation={extension(match_any)}) +// CHECK-NEXT: #pragma omp declare variant(foo) match(implementation={extension(match_all)}) // CHECK-NEXT: #pragma omp declare variant(foo) match(device={kind(nohost)}) // CHECK-NEXT: #pragma omp declare variant(foo) match(device={kind(host)}) // CHECK-NEXT: #pragma omp declare variant(foo) match(implementation={vendor(score(5): ibm)}, device={kind(cpu, nohost)}) diff --git a/clang/test/OpenMP/declare_variant_messages.c b/clang/test/OpenMP/declare_variant_messages.c --- a/clang/test/OpenMP/declare_variant_messages.c +++ b/clang/test/OpenMP/declare_variant_messages.c @@ -46,7 +46,7 @@ #pragma omp declare variant(foo) match(device={kind(score(foo()) ibm)}) // expected-warning {{expected '':'' after the score expression; '':'' assumed}} expected-warning {{the context selector 'kind' in the context set 'device' cannot have a score ('foo()'); score ignored}} expected-warning {{'ibm' is not a valid context property for the context selector 'kind' and the context set 'device'; property ignored}} expected-note {{try 'match(implementation={vendor(ibm)})'}} expected-note {{the ignored property spans until here}} #pragma omp declare variant(foo) match(device={kind(score(5): host), kind(llvm)}) // expected-warning {{the context selector 'kind' in the context set 'device' cannot have a score ('5'); score ignored}} expected-warning {{the context selector 'kind' was used already in the same 'omp declare variant' directive; selector ignored}} expected-note {{the previous context selector 'kind' used here}} expected-note {{the ignored selector spans until here}} #pragma omp declare variant(foo) match(device={kind(score(5): nohost), vendor(llvm)}) // expected-warning {{the context selector 'kind' in the context set 'device' cannot have a score ('5'); score ignored}} expected-warning {{the context selector 'vendor' is not valid for the context set 'device'; selector ignored}} expected-note {{the context selector 'vendor' can be nested in the context set 'implementation'; try 'match(implementation={vendor(property)})'}} expected-note {{the ignored selector spans until here}} -#pragma omp declare variant(foo) match(implementation={extension("aaa")}) // expected-warning {{'aaa' is not a valid context property for the context selector 'extension' and the context set 'implementation'; property ignored}} expected-note {{context property options are: }} expected-note {{the ignored property spans until here}} +#pragma omp declare variant(foo) match(implementation={extension("aaa")}) // expected-warning {{'aaa' is not a valid context property for the context selector 'extension' and the context set 'implementation'; property ignored}} expected-note {{context property options are: 'match_all' 'match_any' 'match_none'}} expected-note {{the ignored property spans until here}} int bar(void); #pragma omp declare variant(foo) match(implementation = {vendor(score(foo) :llvm)}) // expected-warning {{score expressions in the OpenMP context selector need to be constant; foo is not and will be ignored}} diff --git a/llvm/include/llvm/Frontend/OpenMP/OMPContext.h b/llvm/include/llvm/Frontend/OpenMP/OMPContext.h --- a/llvm/include/llvm/Frontend/OpenMP/OMPContext.h +++ b/llvm/include/llvm/Frontend/OpenMP/OMPContext.h @@ -154,9 +154,12 @@ }; /// Return true if \p VMI is applicable in \p Ctx, that is, all traits required -/// by \p VMI are available in the OpenMP context \p Ctx. +/// by \p VMI are available in the OpenMP context \p Ctx. If \p DeviceSetOnly is +/// true, only the device selector set, if present, are checked. Note that we +/// still honor extension traits provided by the user. bool isVariantApplicableInContext(const VariantMatchInfo &VMI, - const OMPContext &Ctx); + const OMPContext &Ctx, + bool DeviceSetOnly = false); /// Return the index (into \p VMIs) of the variant with the highest score /// from the ones applicble in \p Ctx. See llvm::isVariantApplicableInContext. diff --git a/llvm/include/llvm/Frontend/OpenMP/OMPKinds.def b/llvm/include/llvm/Frontend/OpenMP/OMPKinds.def --- a/llvm/include/llvm/Frontend/OpenMP/OMPKinds.def +++ b/llvm/include/llvm/Frontend/OpenMP/OMPKinds.def @@ -664,6 +664,9 @@ __OMP_TRAIT_PROPERTY(implementation, vendor, unknown) __OMP_TRAIT_SELECTOR(implementation, extension, true) +__OMP_TRAIT_PROPERTY(implementation, extension, match_all) +__OMP_TRAIT_PROPERTY(implementation, extension, match_any) +__OMP_TRAIT_PROPERTY(implementation, extension, match_none) __OMP_TRAIT_SET(user) diff --git a/llvm/lib/Frontend/OpenMP/OMPContext.cpp b/llvm/lib/Frontend/OpenMP/OMPContext.cpp --- a/llvm/lib/Frontend/OpenMP/OMPContext.cpp +++ b/llvm/lib/Frontend/OpenMP/OMPContext.cpp @@ -137,52 +137,119 @@ static int isVariantApplicableInContextHelper( const VariantMatchInfo &VMI, const OMPContext &Ctx, - SmallVectorImpl *ConstructMatches) { + SmallVectorImpl *ConstructMatches, bool DeviceSetOnly) { + + // The match kind determines if we need to match all traits, any of the + // traits, or none of the traits for it to be an applicable context. + enum MatchKind { MK_ALL, MK_ANY, MK_NONE }; + + MatchKind MK = MK_ALL; + // Determine the match kind the user wants, "all" is the default and provided + // to the user only for completeness. + if (VMI.RequiredTraits.test( + unsigned(TraitProperty::implementation_extension_match_any))) + MK = MK_ANY; + if (VMI.RequiredTraits.test( + unsigned(TraitProperty::implementation_extension_match_none))) + MK = MK_NONE; + + // Helper to deal with a single property that was (not) found in the OpenMP + // context based on the match kind selected by the user via + // `implementation={extensions(match_[all,any,none])}' + auto HandleTrait = [MK](TraitProperty Property, + bool WasFound) -> Optional /* Result */ { + // For kind "any" a single match is enough but we ignore non-matched properties. + if (MK == MK_ANY) { + if (WasFound) + return true; + return None; + } + + // In "all" or "none" mode we accept a matching or non-matching property + // respectively and move on. We are not done yet! + if ((WasFound && MK == MK_ALL) || (!WasFound && MK == MK_NONE)) + return None; + + // We missed a property, provide some debug output and indicate failure. + LLVM_DEBUG({ + if (MK == MK_ALL) + dbgs() << "[" << DEBUG_TYPE << "] Property " + << getOpenMPContextTraitPropertyName(Property) + << " was not in the OpenMP context but match kind is all.\n"; + if (MK == MK_NONE) + dbgs() << "[" << DEBUG_TYPE << "] Property " + << getOpenMPContextTraitPropertyName(Property) + << " was in the OpenMP context but match kind is none.\n"; + }); + return false; + }; for (unsigned Bit : VMI.RequiredTraits.set_bits()) { TraitProperty Property = TraitProperty(Bit); + if (DeviceSetOnly && + getOpenMPContextTraitSetForProperty(Property) != TraitSet::device) + continue; + + // So far all extensions are handled elsewhere, we skip them here as they + // are not part of the OpenMP context. + if (getOpenMPContextTraitSelectorForProperty(Property) == + TraitSelector::implementation_extension) + continue; bool IsActiveTrait = Ctx.ActiveTraits.test(unsigned(Property)); - if (!IsActiveTrait) { - LLVM_DEBUG(dbgs() << "[" << DEBUG_TYPE << "] Property " - << getOpenMPContextTraitPropertyName(Property) - << " was not in the OpenMP context.\n"); - return false; - } + Optional Result = HandleTrait(Property, IsActiveTrait); + if (Result.hasValue()) + return Result.getValue(); } - // We could use isSubset here but we also want to record the match locations. - unsigned ConstructIdx = 0, NoConstructTraits = Ctx.ConstructTraits.size(); - for (TraitProperty Property : VMI.ConstructTraits) { - assert(getOpenMPContextTraitSetForProperty(Property) == - TraitSet::construct && - "Variant context is ill-formed!"); - - // Verify the nesting. - bool FoundInOrder = false; - while (!FoundInOrder && ConstructIdx != NoConstructTraits) - FoundInOrder = (Ctx.ConstructTraits[ConstructIdx++] == Property); - if (ConstructMatches) - ConstructMatches->push_back(ConstructIdx - 1); - - if (!FoundInOrder) { - LLVM_DEBUG(dbgs() << "[" << DEBUG_TYPE << "] Construct property " - << getOpenMPContextTraitPropertyName(Property) - << " was not nested properly.\n"); - return false; + if (!DeviceSetOnly) { + // We could use isSubset here but we also want to record the match + // locations. + unsigned ConstructIdx = 0, NoConstructTraits = Ctx.ConstructTraits.size(); + for (TraitProperty Property : VMI.ConstructTraits) { + assert(getOpenMPContextTraitSetForProperty(Property) == + TraitSet::construct && + "Variant context is ill-formed!"); + + // Verify the nesting. + bool FoundInOrder = false; + while (!FoundInOrder && ConstructIdx != NoConstructTraits) + FoundInOrder = (Ctx.ConstructTraits[ConstructIdx++] == Property); + if (ConstructMatches) + ConstructMatches->push_back(ConstructIdx - 1); + + Optional Result = HandleTrait(Property, FoundInOrder); + if (Result.hasValue()) + return Result.getValue(); + + if (!FoundInOrder) { + LLVM_DEBUG(dbgs() << "[" << DEBUG_TYPE << "] Construct property " + << getOpenMPContextTraitPropertyName(Property) + << " was not nested properly.\n"); + return false; + } + + // TODO: Verify SIMD } - // TODO: Verify SIMD + assert(isSubset(VMI.ConstructTraits, Ctx.ConstructTraits) && + "Broken invariant!"); + } + + if (MK == MK_ANY) { + LLVM_DEBUG(dbgs() << "[" << DEBUG_TYPE + << "] None of the properties was in the OpenMP context " + "but match kind is any.\n"); + return false; } - assert(isSubset(VMI.ConstructTraits, Ctx.ConstructTraits) && - "Broken invariant!"); return true; } bool llvm::omp::isVariantApplicableInContext(const VariantMatchInfo &VMI, - const OMPContext &Ctx) { - return isVariantApplicableInContextHelper(VMI, Ctx, nullptr); + const OMPContext &Ctx, + bool DeviceSetOnly) { + return isVariantApplicableInContextHelper(VMI, Ctx, nullptr, DeviceSetOnly); } static APInt getVariantMatchScore(const VariantMatchInfo &VMI, @@ -267,7 +334,8 @@ SmallVector ConstructMatches; // If the variant is not applicable its not the best. - if (!isVariantApplicableInContextHelper(VMI, Ctx, &ConstructMatches)) + if (!isVariantApplicableInContextHelper(VMI, Ctx, &ConstructMatches, + /* DeviceSetOnly */ false)) continue; // Check if its clearly not the best. APInt Score = getVariantMatchScore(VMI, Ctx, ConstructMatches);