Skip to content

Commit

Permalink
Remove leftover debugging code/comments
Browse files Browse the repository at this point in the history
Signed-off-by: Anna Rift <anna.rift@hpe.com>
  • Loading branch information
riftEmber committed May 14, 2024
1 parent 177e6a4 commit 3fc1146
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 17 deletions.
2 changes: 0 additions & 2 deletions frontend/lib/parsing/parsing-queries.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -945,8 +945,6 @@ static const AstNode* const& astForIdQuery(Context* context, ID id) {
}

const AstNode* idToAst(Context* context, ID id) {
if (id.symbolName(context).str() == "_tuple") gdbShouldBreakHere();

if (id.isEmpty()) {
CHPL_ASSERT(false && "bad query of uAST for empty ID");
return nullptr;
Expand Down
14 changes: 0 additions & 14 deletions frontend/lib/resolution/Resolver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -602,8 +602,6 @@ Resolver::gatherReceiverAndParentScopesForType(Context* context,
const types::Type* thisType) {
ReceiverScopesVec scopes;

/* gdbShouldBreakHere(); */

if (thisType != nullptr) {
if (const CompositeType* ct = thisType->getCompositeType()) {
// add the scope declaring the type
Expand Down Expand Up @@ -1022,7 +1020,6 @@ getTypeWithCustomInfer(Context* context, const Type* type) {
// TODO 2023-05: This might return null for a type like 'string' which
// we force to exist whether the module is present or not. Is this
// necessary in the long-term?
// maybe this is the problem? first hit of idtoast for _tuple, and it's null here
if (const auto node = parsing::idToAst(context, rec->id())) {
if (const auto attr = node->attributeGroup()) {
if (attr->hasPragma(PRAGMA_INFER_CUSTOM_TYPE)) {
Expand Down Expand Up @@ -2664,17 +2661,6 @@ Resolver::lookupIdentifier(const Identifier* ident,
auto pair = namesWithErrorsEmitted.insert(ident->name());
if (pair.second) {
// insertion took place so emit the error
gdbShouldBreakHere();
auto rcv = methodReceiverScopes(true);
// receiverscopes is wrong, does root scope instead of scope of _tuple
/* for (auto rcvScope : receiverScopes) { */
/* for (const auto& [key, value] : rcvScope->declared()) { */
/* printf("%s\n", key.c_str()); */
/* } */
/* } */
auto vec = lookupNameInScopeWithWarnings(context, scope, receiverScopes,
ident->name(), config,
ident->id());
bool mentionedMoreThanOnce = identHasMoreMentions(ident);
CHPL_REPORT(context, UnknownIdentifier, ident, mentionedMoreThanOnce);
}
Expand Down
1 change: 0 additions & 1 deletion frontend/lib/types/TupleType.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,6 @@ TupleType::getReferentialTuple(Context* context,

const TupleType*
TupleType::getGenericTupleType(Context* context) {
// attempting wiring up here and in other get*Tuple* functions
auto symbolPath = UniqueString::get(context, "ChapelTuple._tuple");
auto name = UniqueString::get(context, "_tuple");
auto id = ID(symbolPath, -1, 0);
Expand Down

0 comments on commit 3fc1146

Please sign in to comment.