| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370 |
- // Part of the Carbon Language project, under the Apache License v2.0 with LLVM
- // Exceptions. See /LICENSE for license information.
- // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
- #include "toolchain/check/merge.h"
- #include "toolchain/base/kind_switch.h"
- #include "toolchain/check/import_ref.h"
- #include "toolchain/sem_ir/ids.h"
- #include "toolchain/sem_ir/typed_insts.h"
- namespace Carbon::Check {
- CARBON_DIAGNOSTIC(RedeclPrevDecl, Note, "Previously declared here.");
- // Diagnoses a redeclaration which is redundant.
- static auto DiagnoseRedundant(Context& context, Lex::TokenKind decl_kind,
- SemIR::NameId name_id, SemIRLoc new_loc,
- SemIRLoc prev_loc) {
- CARBON_DIAGNOSTIC(RedeclRedundant, Error,
- "Redeclaration of `{0} {1}` is redundant.", Lex::TokenKind,
- SemIR::NameId);
- context.emitter()
- .Build(new_loc, RedeclRedundant, decl_kind, name_id)
- .Note(prev_loc, RedeclPrevDecl)
- .Emit();
- }
- // Diagnoses a redefinition.
- static auto DiagnoseRedef(Context& context, Lex::TokenKind decl_kind,
- SemIR::NameId name_id, SemIRLoc new_loc,
- SemIRLoc prev_loc) {
- CARBON_DIAGNOSTIC(RedeclRedef, Error, "Redefinition of `{0} {1}`.",
- Lex::TokenKind, SemIR::NameId);
- CARBON_DIAGNOSTIC(RedeclPrevDef, Note, "Previously defined here.");
- context.emitter()
- .Build(new_loc, RedeclRedef, decl_kind, name_id)
- .Note(prev_loc, RedeclPrevDef)
- .Emit();
- }
- // Diagnoses an `extern` versus non-`extern` mismatch.
- static auto DiagnoseExternMismatch(Context& context, Lex::TokenKind decl_kind,
- SemIR::NameId name_id, SemIRLoc new_loc,
- SemIRLoc prev_loc) {
- CARBON_DIAGNOSTIC(RedeclExternMismatch, Error,
- "Redeclarations of `{0} {1}` in the same library must "
- "match use of `extern`.",
- Lex::TokenKind, SemIR::NameId);
- context.emitter()
- .Build(new_loc, RedeclExternMismatch, decl_kind, name_id)
- .Note(prev_loc, RedeclPrevDecl)
- .Emit();
- }
- // Diagnoses when multiple non-`extern` declarations are found.
- static auto DiagnoseNonExtern(Context& context, Lex::TokenKind decl_kind,
- SemIR::NameId name_id, SemIRLoc new_loc,
- SemIRLoc prev_loc) {
- CARBON_DIAGNOSTIC(RedeclNonExtern, Error,
- "Only one library can declare `{0} {1}` without `extern`.",
- Lex::TokenKind, SemIR::NameId);
- context.emitter()
- .Build(new_loc, RedeclNonExtern, decl_kind, name_id)
- .Note(prev_loc, RedeclPrevDecl)
- .Emit();
- }
- // Checks to see if a structurally valid redeclaration is allowed in context.
- // These all still merge.
- auto CheckIsAllowedRedecl(Context& context, Lex::TokenKind decl_kind,
- SemIR::NameId name_id, RedeclInfo new_decl,
- RedeclInfo prev_decl, SemIR::ImportIRId import_ir_id)
- -> void {
- if (!import_ir_id.is_valid()) {
- // Check for disallowed redeclarations in the same file.
- if (!new_decl.is_definition) {
- DiagnoseRedundant(context, decl_kind, name_id, new_decl.loc,
- prev_decl.loc);
- return;
- }
- if (prev_decl.is_definition) {
- DiagnoseRedef(context, decl_kind, name_id, new_decl.loc, prev_decl.loc);
- return;
- }
- // `extern` definitions are prevented at creation; this is only
- // checking for a non-`extern` definition after an `extern` declaration.
- if (prev_decl.is_extern) {
- DiagnoseExternMismatch(context, decl_kind, name_id, new_decl.loc,
- prev_decl.loc);
- return;
- }
- return;
- }
- if (import_ir_id == SemIR::ImportIRId::ApiForImpl) {
- // Check for disallowed redeclarations in the same library. Note that a
- // forward declaration in the impl is allowed.
- if (prev_decl.is_definition) {
- if (new_decl.is_definition) {
- DiagnoseRedef(context, decl_kind, name_id, new_decl.loc, prev_decl.loc);
- } else {
- DiagnoseRedundant(context, decl_kind, name_id, new_decl.loc,
- prev_decl.loc);
- }
- return;
- }
- if (prev_decl.is_extern != new_decl.is_extern) {
- DiagnoseExternMismatch(context, decl_kind, name_id, new_decl.loc,
- prev_decl.loc);
- return;
- }
- return;
- }
- // Check for disallowed redeclarations cross-library.
- if (!new_decl.is_extern && !prev_decl.is_extern) {
- DiagnoseNonExtern(context, decl_kind, name_id, new_decl.loc, prev_decl.loc);
- return;
- }
- }
- auto ReplacePrevInstForMerge(Context& context, SemIR::NameScopeId scope_id,
- SemIR::NameId name_id, SemIR::InstId new_inst_id)
- -> void {
- auto& scope = context.name_scopes().Get(scope_id);
- if (auto lookup = scope.name_map.Lookup(name_id)) {
- scope.names[lookup.value()].inst_id = new_inst_id;
- }
- }
- // Returns true if there was an error in declaring the entity, which will have
- // previously been diagnosed.
- static auto EntityHasParamError(Context& context, const DeclParams& info)
- -> bool {
- for (auto param_refs_id : {info.implicit_param_refs_id, info.param_refs_id}) {
- if (param_refs_id.is_valid() &&
- param_refs_id != SemIR::InstBlockId::Empty) {
- for (auto param_id : context.inst_blocks().Get(param_refs_id)) {
- if (context.insts().Get(param_id).type_id() == SemIR::TypeId::Error) {
- return true;
- }
- }
- }
- }
- return false;
- }
- // Returns false if a param differs for a redeclaration. The caller is expected
- // to provide a diagnostic.
- static auto CheckRedeclParam(Context& context,
- llvm::StringLiteral param_diag_label,
- int32_t param_index,
- SemIR::InstId new_param_ref_id,
- SemIR::InstId prev_param_ref_id,
- SemIR::SpecificId prev_specific_id) -> bool {
- // TODO: Consider differentiating between type and name mistakes. For now,
- // taking the simpler approach because I also think we may want to refactor
- // params.
- auto diagnose = [&]() {
- CARBON_DIAGNOSTIC(RedeclParamDiffers, Error,
- "Redeclaration differs at {0}parameter {1}.",
- llvm::StringLiteral, int32_t);
- CARBON_DIAGNOSTIC(RedeclParamPrevious, Note,
- "Previous declaration's corresponding {0}parameter here.",
- llvm::StringLiteral);
- context.emitter()
- .Build(new_param_ref_id, RedeclParamDiffers, param_diag_label,
- param_index + 1)
- .Note(prev_param_ref_id, RedeclParamPrevious, param_diag_label)
- .Emit();
- };
- auto new_param_ref = context.insts().Get(new_param_ref_id);
- auto prev_param_ref = context.insts().Get(prev_param_ref_id);
- if (new_param_ref.kind() != prev_param_ref.kind() ||
- !context.types().AreEqualAcrossDeclarations(
- new_param_ref.type_id(),
- SemIR::GetTypeInSpecific(context.sem_ir(), prev_specific_id,
- prev_param_ref.type_id()))) {
- diagnose();
- return false;
- }
- if (new_param_ref.Is<SemIR::AddrPattern>()) {
- new_param_ref =
- context.insts().Get(new_param_ref.As<SemIR::AddrPattern>().inner_id);
- prev_param_ref =
- context.insts().Get(prev_param_ref.As<SemIR::AddrPattern>().inner_id);
- if (new_param_ref.kind() != prev_param_ref.kind()) {
- diagnose();
- return false;
- }
- }
- if (new_param_ref.Is<SemIR::AnyBindName>()) {
- new_param_ref =
- context.insts().Get(new_param_ref.As<SemIR::AnyBindName>().value_id);
- prev_param_ref =
- context.insts().Get(prev_param_ref.As<SemIR::AnyBindName>().value_id);
- }
- auto new_param = new_param_ref.As<SemIR::Param>();
- auto prev_param = prev_param_ref.As<SemIR::Param>();
- if (new_param.name_id != prev_param.name_id) {
- diagnose();
- return false;
- }
- return true;
- }
- // Returns false if the param refs differ for a redeclaration.
- static auto CheckRedeclParams(Context& context, SemIRLoc new_decl_loc,
- SemIR::InstBlockId new_param_refs_id,
- SemIRLoc prev_decl_loc,
- SemIR::InstBlockId prev_param_refs_id,
- llvm::StringLiteral param_diag_label,
- SemIR::SpecificId prev_specific_id) -> bool {
- // This will often occur for empty params.
- if (new_param_refs_id == prev_param_refs_id) {
- return true;
- }
- // If exactly one of the parameter lists was present, they differ.
- if (new_param_refs_id.is_valid() != prev_param_refs_id.is_valid()) {
- CARBON_DIAGNOSTIC(RedeclParamListDiffers, Error,
- "Redeclaration differs because of {1}{0}parameter list.",
- llvm::StringLiteral, llvm::StringLiteral);
- CARBON_DIAGNOSTIC(RedeclParamListPrevious, Note,
- "Previously declared with{1} {0}parameter list.",
- llvm::StringLiteral, llvm::StringLiteral);
- context.emitter()
- .Build(
- new_decl_loc, RedeclParamListDiffers, param_diag_label,
- new_param_refs_id.is_valid() ? llvm::StringLiteral("") : "missing ")
- .Note(prev_decl_loc, RedeclParamListPrevious, param_diag_label,
- prev_param_refs_id.is_valid() ? llvm::StringLiteral("") : "out")
- .Emit();
- return false;
- }
- CARBON_CHECK(new_param_refs_id.is_valid() && prev_param_refs_id.is_valid());
- const auto new_param_ref_ids = context.inst_blocks().Get(new_param_refs_id);
- const auto prev_param_ref_ids = context.inst_blocks().Get(prev_param_refs_id);
- if (new_param_ref_ids.size() != prev_param_ref_ids.size()) {
- CARBON_DIAGNOSTIC(
- RedeclParamCountDiffers, Error,
- "Redeclaration differs because of {0}parameter count of {1}.",
- llvm::StringLiteral, int32_t);
- CARBON_DIAGNOSTIC(RedeclParamCountPrevious, Note,
- "Previously declared with {0}parameter count of {1}.",
- llvm::StringLiteral, int32_t);
- context.emitter()
- .Build(new_decl_loc, RedeclParamCountDiffers, param_diag_label,
- new_param_ref_ids.size())
- .Note(prev_decl_loc, RedeclParamCountPrevious, param_diag_label,
- prev_param_ref_ids.size())
- .Emit();
- return false;
- }
- for (auto [index, new_param_ref_id, prev_param_ref_id] :
- llvm::enumerate(new_param_ref_ids, prev_param_ref_ids)) {
- if (!CheckRedeclParam(context, param_diag_label, index, new_param_ref_id,
- prev_param_ref_id, prev_specific_id)) {
- return false;
- }
- }
- return true;
- }
- // Returns true if the two nodes represent the same syntax.
- // TODO: Detect raw identifiers (will require token changes).
- static auto IsNodeSyntaxEqual(Context& context, Parse::NodeId new_node_id,
- Parse::NodeId prev_node_id) -> bool {
- if (context.parse_tree().node_kind(new_node_id) !=
- context.parse_tree().node_kind(prev_node_id)) {
- return false;
- }
- // TODO: Should there be a trivial way to check if we need to check spellings?
- // Identifiers and literals need their text checked for cross-file matching,
- // but not intra-file. Keywords and operators shouldn't need the token text
- // examined at all.
- auto new_spelling = context.tokens().GetTokenText(
- context.parse_tree().node_token(new_node_id));
- auto prev_spelling = context.tokens().GetTokenText(
- context.parse_tree().node_token(prev_node_id));
- return new_spelling == prev_spelling;
- }
- // Returns false if redeclaration parameter syntax doesn't match.
- static auto CheckRedeclParamSyntax(Context& context,
- Parse::NodeId new_first_param_node_id,
- Parse::NodeId new_last_param_node_id,
- Parse::NodeId prev_first_param_node_id,
- Parse::NodeId prev_last_param_node_id)
- -> bool {
- // Parse nodes may not always be available to compare.
- // TODO: Support cross-file syntax checks. Right now imports provide invalid
- // nodes, and we'll need to follow the declaration to its original file to
- // get the parse tree.
- if (!new_first_param_node_id.is_valid() ||
- !prev_first_param_node_id.is_valid()) {
- return true;
- }
- CARBON_CHECK(new_last_param_node_id.is_valid())
- << "new_last_param_node_id.is_valid should match "
- "new_first_param_node_id.is_valid";
- CARBON_CHECK(prev_last_param_node_id.is_valid())
- << "prev_last_param_node_id.is_valid should match "
- "prev_first_param_node_id.is_valid";
- auto new_range = Parse::Tree::PostorderIterator::MakeRange(
- new_first_param_node_id, new_last_param_node_id);
- auto prev_range = Parse::Tree::PostorderIterator::MakeRange(
- prev_first_param_node_id, prev_last_param_node_id);
- // zip is using the shortest range. If they differ in length, there should be
- // some difference inside the range because the range includes parameter
- // brackets. As a consequence, we don't explicitly handle different range
- // sizes here.
- for (auto [new_node_id, prev_node_id] : llvm::zip(new_range, prev_range)) {
- if (!IsNodeSyntaxEqual(context, new_node_id, prev_node_id)) {
- CARBON_DIAGNOSTIC(RedeclParamSyntaxDiffers, Error,
- "Redeclaration syntax differs here.");
- CARBON_DIAGNOSTIC(RedeclParamSyntaxPrevious, Note,
- "Comparing with previous declaration here.");
- context.emitter()
- .Build(new_node_id, RedeclParamSyntaxDiffers)
- .Note(prev_node_id, RedeclParamSyntaxPrevious)
- .Emit();
- return false;
- }
- }
- return true;
- }
- auto CheckRedeclParamsMatch(Context& context, const DeclParams& new_entity,
- const DeclParams& prev_entity,
- SemIR::SpecificId prev_specific_id,
- bool check_syntax) -> bool {
- if (EntityHasParamError(context, new_entity) ||
- EntityHasParamError(context, prev_entity)) {
- return false;
- }
- if (!CheckRedeclParams(context, new_entity.loc,
- new_entity.implicit_param_refs_id, prev_entity.loc,
- prev_entity.implicit_param_refs_id, "implicit ",
- prev_specific_id)) {
- return false;
- }
- if (!CheckRedeclParams(context, new_entity.loc, new_entity.param_refs_id,
- prev_entity.loc, prev_entity.param_refs_id, "",
- prev_specific_id)) {
- return false;
- }
- if (check_syntax &&
- !CheckRedeclParamSyntax(context, new_entity.first_param_node_id,
- new_entity.last_param_node_id,
- prev_entity.first_param_node_id,
- prev_entity.last_param_node_id)) {
- return false;
- }
- return true;
- }
- } // namespace Carbon::Check
|