…uses The Flang implemenation of OpenACC uses a .td file in the llvm/Frontend directory to determine appertainment in 4 categories: -Required: If this list has items in it, the directive requires at least 1 of these be present. -AllowedExclusive: Items on this list are all allowed, but only 1 from the list may be here (That is, they are exclusive of eachother). -AllowedOnce: Items on this list are all allowed, but may not be duplicated. Allowed: Items on this list are allowed. Note th at the actual list of 'allowed' is all 4 of these lists together. This is a draft patch to swtich Clang over to use those tables. Surgery to get this to happen in Clang Sema was somewhat reasonable. However, some gaps in the implementations are obvious, the existing clang implementation disagrees with the Flang interpretation of it. SO, we're keeping a task list here based on what gets discovered. Changes to Clang: - [x] Switch 'directive-kind' enum conversions to use tablegen See ff1a7bddd9435b6ae2890c07eae60bb07898bbf5 - [x] Switch 'clause-kind' enum conversions to use tablegen See ff1a7bddd9435b6ae2890c07eae60bb07898bbf5 - [x] Investigate 'parse' test differences to see if any new disagreements arise. - [x] Clang/Flang disagree as to whether 'collapse' can be multiple times on a loop. Further research showed no prose to limit this, and the comment on the clang implementation said "no good reason to allow", so no standards justification. - [x] Clang/Flang disagree whether 'num_gangs' can appear >1 on a compute/combined construct. This ended up being an unjustified restriction. - [x] Clang/Flang disagree as to the list of required clauses on a 'set' construct. My research shows that Clang mistakenly included 'if' in the list, and that it should be just 'default_async', 'device_num', and 'device_type'. - [x] Order of 'at least one of' diagnostic has changed. Tests were updated. - [x] Ensure we are properly 'de-aliasing' clause names in appertainment checks? - [x] What is 'shortloop'? 'shortloop' seems to be an old non-standard extension that isn't supported by flang, but is parsed for backward compat reasons. Clang won't parse, but we at least have a spot for it in the clause list. - [x] Implemented proposed change for 'routine' gang/worker/vector/seq. (see issue 539) - [x] Implement init/shutdown can only have 1 'if' (see issue 540) - [x] Clang/Flang disagree as to whether 'tile' is permitted more than once on a 'loop' or combined constructs (Flang prohibits >1). I see no justification for this in the standard. EDIT: I found a comment in clang that I did this to make SOMETHING around duplicate checks easier. Discussion showed we should actually have a better behavior around 'device_type' and duplicates, so I've since implemented that. - [x] Clang/Flang disagree whether 'gang', 'worker', or 'vector' may appear on the same construct as a 'seq' on a 'loop' or 'combined'. There is prose for this in 2022: (a gang, worker, or vector clause may not appear if a 'seq' clause appears). EDIT: These don't actually disagree, but aren't in the .td file, so I restored the existing code to do this. - [x] Clang/Flang disagree on whether 'bind' can appear >1 on a 'routine'. I believe line 3096 (A bind clause may not bind to a routine name that has a visible bind clause) makes this limitation (Flang permits >1 bind). we discussed and decided this should have the same rules as worker/vector/etc, except without the 'exactly 1 of' rule (so no dupes in individual sections). - [x] Clang/Flang disagree on whether 'init'/'shutdown' can have multiple 'device_num' clauses. I believe there is no supporting prose for this limitation., We decided that `device_num` should only happen 1x. - [x] Clang/Flang disagree whether 'num_gangs' can appear >1 on a 'kernels' construct. Line 1173 (On a kernels construct, the num_gangs clause must have a single argument) justifies limiting on a per-arguement basis, but doesn't do so for multiple num_gangs clauses. WE decided to do this with the '1-per-device-type' region for num_gangs, num_workers, and vector_length, see openacc bug here: https://github.com/OpenACC/openacc-spec/issues/541 Changes to Flang: - [x] Clang/Flang disgree on whether 'atomic' can take an 'if' clause. This was added in OpenACC3.3_Next See #135451 - [x] Clang/Flang disagree on whether 'finalize' can be allowed >1 times on a 'exit_data' construct. see #135415. - [x] Clang/Flang disagree whether 'if_present' should be allowed >1 times on a 'host_data'/'update' construct. see #135422 - [x] Clang/Flang disagree on whether 'init'/'shutdown' can have multiple 'device_type' clauses. I believe there is no supporting prose for this limitation. - [ ] SEE change for num_gangs/etc above. Changes that need discussion/research:
230 lines
7.0 KiB
C++
230 lines
7.0 KiB
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
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
/// \file
|
|
/// This file implements conversions from the ACC.td from the backend to
|
|
/// determine appertainment, required/etc.
|
|
///
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
#include "clang/Basic/DiagnosticSema.h"
|
|
#include "clang/Sema/SemaOpenACC.h"
|
|
|
|
#include "llvm/ADT/STLExtras.h"
|
|
#include "llvm/ADT/bit.h"
|
|
|
|
using namespace clang;
|
|
|
|
namespace {
|
|
// Implements a simple 'enum-set' which stores enum values in a single 64 bit
|
|
// value. Flang has `EnumSet` which is pretty sizable/has a lot of dependencies,
|
|
// so likely not worth bringing in for this use.
|
|
class AccClauseSet {
|
|
// We're just using a uint64_t as our underlying rep, so if this size ever
|
|
// gets bigger than 64, we probably need a pair of uint64_ts.
|
|
static_assert(static_cast<unsigned>(OpenACCClauseKind::Invalid) < 64);
|
|
uint64_t Data;
|
|
|
|
void setBit(OpenACCClauseKind C) {
|
|
Data |= static_cast<uint64_t>(1) << static_cast<uint64_t>(C);
|
|
}
|
|
|
|
public:
|
|
constexpr AccClauseSet(
|
|
const std::initializer_list<OpenACCClauseKind> &Clauses)
|
|
: Data(0) {
|
|
for (OpenACCClauseKind C : Clauses)
|
|
setBit(C);
|
|
}
|
|
|
|
constexpr bool isSet(OpenACCClauseKind C) const {
|
|
return ((Data >> static_cast<uint64_t>(C)) & 1) != 0;
|
|
}
|
|
|
|
void clearBit(OpenACCClauseKind C) {
|
|
Data &= ~(static_cast<uint64_t>(1) << static_cast<uint64_t>(C));
|
|
}
|
|
|
|
constexpr bool isEmpty() const { return Data == 0; }
|
|
|
|
unsigned popcount() const { return llvm::popcount<uint64_t>(Data); }
|
|
};
|
|
|
|
struct LLVMClauseLists {
|
|
AccClauseSet Allowed;
|
|
AccClauseSet AllowedOnce;
|
|
AccClauseSet AllowedExclusive;
|
|
AccClauseSet Required;
|
|
};
|
|
struct LLVMDirectiveClauseRelationships {
|
|
OpenACCDirectiveKind DirKind;
|
|
LLVMClauseLists Lists;
|
|
};
|
|
|
|
} // namespace
|
|
|
|
// This introduces these in a llvm::acc namespace, so make sure this stays in
|
|
// the global namespace.
|
|
#define GEN_CLANG_DIRECTIVE_CLAUSE_SETS
|
|
#include "llvm/Frontend/OpenACC/ACC.inc"
|
|
|
|
namespace {
|
|
LLVMDirectiveClauseRelationships Relations[] =
|
|
#define GEN_CLANG_DIRECTIVE_CLAUSE_MAP
|
|
#include "llvm/Frontend/OpenACC/ACC.inc"
|
|
;
|
|
|
|
const LLVMClauseLists &getListsForDirective(OpenACCDirectiveKind DK) {
|
|
|
|
auto Res = llvm::find_if(Relations,
|
|
[=](const LLVMDirectiveClauseRelationships &Rel) {
|
|
return Rel.DirKind == DK;
|
|
});
|
|
assert(Res != std::end(Relations) && "Unknown directive kind?");
|
|
|
|
return Res->Lists;
|
|
}
|
|
|
|
std::string getListOfClauses(AccClauseSet Set) {
|
|
// We could probably come up with a better way to do this smuggling, but this
|
|
// is good enough for now.
|
|
std::string Output;
|
|
llvm::raw_string_ostream OS{Output};
|
|
|
|
for (unsigned I = 0; I < static_cast<unsigned>(OpenACCClauseKind::Invalid);
|
|
++I) {
|
|
OpenACCClauseKind CurClause = static_cast<OpenACCClauseKind>(I);
|
|
if (!Set.isSet(CurClause))
|
|
continue;
|
|
|
|
OS << '\'' << CurClause << '\'';
|
|
|
|
Set.clearBit(CurClause);
|
|
|
|
if (Set.isEmpty()) {
|
|
OS.flush();
|
|
return OS.str();
|
|
}
|
|
|
|
OS << ", ";
|
|
|
|
if (Set.popcount() == 1)
|
|
OS << "or ";
|
|
}
|
|
OS.flush();
|
|
return OS.str();
|
|
}
|
|
|
|
OpenACCClauseKind dealiasClauseKind(OpenACCClauseKind CK) {
|
|
switch (CK) {
|
|
default:
|
|
return CK;
|
|
#define VISIT_CLAUSE(NAME)
|
|
#define CLAUSE_ALIAS(ALIAS, NAME, DEPRECATED) \
|
|
case OpenACCClauseKind::ALIAS: \
|
|
return OpenACCClauseKind::NAME;
|
|
#include "clang/Basic/OpenACCClauses.def"
|
|
}
|
|
|
|
return CK;
|
|
}
|
|
} // namespace
|
|
|
|
// Diagnoses if `Clauses` list doesn't have at least one of the required
|
|
// clauses.
|
|
bool SemaOpenACC::DiagnoseRequiredClauses(
|
|
OpenACCDirectiveKind DK, SourceLocation DirectiveLoc,
|
|
ArrayRef<const OpenACCClause *> Clauses) {
|
|
if (DK == OpenACCDirectiveKind::Invalid)
|
|
return false;
|
|
|
|
const LLVMClauseLists &Lists = getListsForDirective(DK);
|
|
|
|
if (Lists.Required.isEmpty())
|
|
return false;
|
|
|
|
for (auto *C : Clauses) {
|
|
if (Lists.Required.isSet(dealiasClauseKind(C->getClauseKind())))
|
|
return false;
|
|
}
|
|
|
|
return Diag(DirectiveLoc, diag::err_acc_construct_one_clause_of)
|
|
<< DK << getListOfClauses(Lists.Required);
|
|
return true;
|
|
}
|
|
|
|
// Diagnoses a 'CK' on a 'DK' present more than once in a clause-list when it
|
|
// isn't allowed.
|
|
bool SemaOpenACC::DiagnoseAllowedOnceClauses(
|
|
OpenACCDirectiveKind DK, OpenACCClauseKind CK, SourceLocation ClauseLoc,
|
|
ArrayRef<const OpenACCClause *> Clauses) {
|
|
if (DK == OpenACCDirectiveKind::Invalid || CK == OpenACCClauseKind::Invalid)
|
|
return false;
|
|
|
|
OpenACCClauseKind Dealiased = dealiasClauseKind(CK);
|
|
|
|
const LLVMClauseLists &Lists = getListsForDirective(DK);
|
|
if (!Lists.AllowedOnce.isSet(CK))
|
|
return false;
|
|
|
|
auto Res = llvm::find_if(Clauses, [=](const OpenACCClause *C) {
|
|
return dealiasClauseKind(C->getClauseKind()) == Dealiased;
|
|
});
|
|
|
|
if (Res == Clauses.end())
|
|
return false;
|
|
|
|
Diag(ClauseLoc, diag::err_acc_duplicate_clause_disallowed) << DK << CK;
|
|
Diag((*Res)->getBeginLoc(), diag::note_acc_previous_clause_here);
|
|
return true;
|
|
}
|
|
|
|
// Diagnoses a 'CK' on a 'DK' being added that isn't allowed to, because another
|
|
// clause in 'Clauses' already exists.
|
|
bool SemaOpenACC::DiagnoseExclusiveClauses(
|
|
OpenACCDirectiveKind DK, OpenACCClauseKind CK, SourceLocation ClauseLoc,
|
|
ArrayRef<const OpenACCClause *> Clauses) {
|
|
if (DK == OpenACCDirectiveKind::Invalid || CK == OpenACCClauseKind::Invalid)
|
|
return false;
|
|
|
|
const LLVMClauseLists &Lists = getListsForDirective(DK);
|
|
OpenACCClauseKind Dealiased = dealiasClauseKind(CK);
|
|
|
|
// If this isn't on the list, this is fine.
|
|
if (!Lists.AllowedExclusive.isSet(Dealiased))
|
|
return false;
|
|
|
|
for (const OpenACCClause *C : Clauses) {
|
|
if (Lists.AllowedExclusive.isSet(dealiasClauseKind(C->getClauseKind()))) {
|
|
Diag(ClauseLoc, diag::err_acc_clause_cannot_combine)
|
|
<< CK << C->getClauseKind() << DK;
|
|
Diag(C->getBeginLoc(), diag::note_acc_previous_clause_here);
|
|
|
|
return true;
|
|
}
|
|
}
|
|
|
|
return false;
|
|
}
|
|
|
|
// Diagnoses if 'CK' is not allowed on a directive of 'DK'.
|
|
bool SemaOpenACC::DiagnoseAllowedClauses(OpenACCDirectiveKind DK,
|
|
OpenACCClauseKind CK,
|
|
SourceLocation ClauseLoc) {
|
|
if (DK == OpenACCDirectiveKind::Invalid || CK == OpenACCClauseKind::Invalid)
|
|
return false;
|
|
const LLVMClauseLists &Lists = getListsForDirective(DK);
|
|
OpenACCClauseKind Dealiased = dealiasClauseKind(CK);
|
|
|
|
if (!Lists.Allowed.isSet(Dealiased) && !Lists.AllowedOnce.isSet(Dealiased) &&
|
|
!Lists.AllowedExclusive.isSet(Dealiased) &&
|
|
!Lists.Required.isSet(Dealiased))
|
|
return Diag(ClauseLoc, diag::err_acc_clause_appertainment) << DK << CK;
|
|
|
|
return false;
|
|
}
|