Skip to content

Commit 2a5645a

Browse files
committed
[clang] Rename diag notes that assumed precompiled dependencies are pch's, NFCI (llvm#142161)
(cherry picked from commit 883130e)
1 parent 5444a4a commit 2a5645a

File tree

6 files changed

+14
-12
lines changed

6 files changed

+14
-12
lines changed

clang/include/clang/Basic/DiagnosticSerializationKinds.td

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,9 @@ def err_fe_ast_file_modified : Error<
2424
DefaultFatal;
2525
def err_fe_pch_file_overridden : Error<
2626
"file '%0' from the precompiled header has been overridden">;
27-
def note_pch_required_by : Note<"'%0' required by '%1'">;
28-
def note_pch_rebuild_required : Note<"please rebuild precompiled header '%0'">;
27+
def note_ast_file_required_by : Note<"'%0' required by '%1'">;
28+
def note_ast_file_rebuild_required
29+
: Note<"please rebuild precompiled file '%0'">;
2930
def note_module_cache_path : Note<
3031
"after modifying system headers, please delete the module cache at '%0'">;
3132

clang/lib/Serialization/ASTReader.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2680,25 +2680,25 @@ InputFile ASTReader::getInputFile(ModuleFile &F, unsigned ID, bool Complain) {
26802680
while (!ImportStack.back()->ImportedBy.empty())
26812681
ImportStack.push_back(ImportStack.back()->ImportedBy[0]);
26822682

2683-
// The top-level PCH is stale.
2684-
StringRef TopLevelPCHName(ImportStack.back()->FileName);
2683+
// The top-level AST file is stale.
2684+
StringRef TopLevelASTFileName(ImportStack.back()->FileName);
26852685
Diag(diag::err_fe_ast_file_modified)
26862686
<< *Filename << moduleKindForDiagnostic(ImportStack.back()->Kind)
2687-
<< TopLevelPCHName << FileChange.Kind
2687+
<< TopLevelASTFileName << FileChange.Kind
26882688
<< (FileChange.Old && FileChange.New)
26892689
<< llvm::itostr(FileChange.Old.value_or(0))
26902690
<< llvm::itostr(FileChange.New.value_or(0));
26912691

26922692
// Print the import stack.
26932693
if (ImportStack.size() > 1) {
2694-
Diag(diag::note_pch_required_by)
2694+
Diag(diag::note_ast_file_required_by)
26952695
<< *Filename << ImportStack[0]->FileName;
26962696
for (unsigned I = 1; I < ImportStack.size(); ++I)
2697-
Diag(diag::note_pch_required_by)
2698-
<< ImportStack[I-1]->FileName << ImportStack[I]->FileName;
2697+
Diag(diag::note_ast_file_required_by)
2698+
<< ImportStack[I - 1]->FileName << ImportStack[I]->FileName;
26992699
}
27002700

2701-
Diag(diag::note_pch_rebuild_required) << TopLevelPCHName;
2701+
Diag(diag::note_ast_file_rebuild_required) << TopLevelASTFileName;
27022702
}
27032703

27042704
IsOutOfDate = true;

clang/test/Modules/module-file-modified.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,4 +8,5 @@
88
#include "a.h"
99
int foo = 0; // redefinition of 'foo'
1010
// CHECK: fatal error: file {{.*}} has been modified since the module file {{.*}} was built
11+
// CHECK: note: please rebuild precompiled file
1112
// REQUIRES: shell

clang/test/Modules/validate-file-content.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,5 +29,5 @@
2929
// CHECK: file '[[M_H:.*[/\\]m\.h]]' has been modified since the precompiled header '[[A_PCH:.*/a\.pch]]' was built: content changed
3030
// CHECK: '[[M_H]]' required by '[[M_PCM:.*[/\\]m.*\.pcm]]'
3131
// CHECK: '[[M_PCM]]' required by '[[A_PCH]]'
32-
// CHECK: please rebuild precompiled header '[[A_PCH]]'
32+
// CHECK: please rebuild precompiled file '[[A_PCH]]'
3333
// expected-no-diagnostics

clang/test/PCH/modified-module-dependency.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,4 @@
1717
// CHECK: file '[[TEST_H:.*[/\\]test\.h]]' has been modified since the precompiled header '[[PREFIX_PCH:.*/prefix\.pch]]' was built
1818
// CHECK: '[[TEST_H]]' required by '[[TEST_PCM:.*[/\\]test\.pcm]]'
1919
// CHECK: '[[TEST_PCM]]' required by '[[PREFIX_PCH]]'
20-
// CHECK: please rebuild precompiled header '[[PREFIX_PCH]]'
20+
// CHECK: please rebuild precompiled file '[[PREFIX_PCH]]'

clang/test/PCH/validate-file-content.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,5 +25,5 @@
2525
// RUN: FileCheck %s < %t/stderr
2626
//
2727
// CHECK: file '[[M_H:.*[/\\]m\.h]]' has been modified since the precompiled header '[[A_PCH:.*/a\.pch]]' was built: content changed
28-
// CHECK: please rebuild precompiled header '[[A_PCH]]'
28+
// CHECK: please rebuild precompiled file '[[A_PCH]]'
2929
// expected-no-diagnostics

0 commit comments

Comments
 (0)