[clang] [clang] [NFC] Remove SourceLocation() parameter from Diag.Report() calls in SourceManager, and use the equivalent Report() overload instead (PR #116937)
Boaz Brickner via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 20 00:20:17 PST 2024
https://github.com/bricknerb created https://github.com/llvm/llvm-project/pull/116937
None
>From 376c3582016fb100743a0bbedd329ea480a14a31 Mon Sep 17 00:00:00 2001
From: Boaz Brickner <brickner at google.com>
Date: Wed, 20 Nov 2024 09:19:22 +0100
Subject: [PATCH] [clang] [NFC] Remove SourceLocation() parameter from
Diag.Report() calls in SourceManager, and use the equivalent Report()
overload instead
---
clang/lib/Basic/SourceManager.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/clang/lib/Basic/SourceManager.cpp b/clang/lib/Basic/SourceManager.cpp
index 9b983cefbcf4ca..6e588ce63d813f 100644
--- a/clang/lib/Basic/SourceManager.cpp
+++ b/clang/lib/Basic/SourceManager.cpp
@@ -665,7 +665,7 @@ SourceManager::createExpansionLocImpl(const ExpansionInfo &Info,
LocalSLocEntryTable.push_back(SLocEntry::get(NextLocalOffset, Info));
if (NextLocalOffset + Length + 1 <= NextLocalOffset ||
NextLocalOffset + Length + 1 > CurrentLoadedOffset) {
- Diag.Report(SourceLocation(), diag::err_sloc_space_too_large);
+ Diag.Report(diag::err_sloc_space_too_large);
// FIXME: call `noteSLocAddressSpaceUsage` to report details to users and
// use a source location from `Info` to point at an error.
// Currently, both cause Clang to run indefinitely, this needs to be fixed.
@@ -2295,7 +2295,7 @@ void SourceManager::noteSLocAddressSpaceUsage(
uint64_t LoadedUsage = MaxLoadedOffset - CurrentLoadedOffset;
int UsagePercent = static_cast<int>(100.0 * double(LocalUsage + LoadedUsage) /
MaxLoadedOffset);
- Diag.Report(SourceLocation(), diag::note_total_sloc_usage)
+ Diag.Report(diag::note_total_sloc_usage)
<< LocalUsage << LoadedUsage << (LocalUsage + LoadedUsage)
<< UsagePercent;
@@ -2311,7 +2311,7 @@ void SourceManager::noteSLocAddressSpaceUsage(
// Describe any remaining usage not reported in the per-file usage.
if (ReportedSize != CountedSize) {
- Diag.Report(SourceLocation(), diag::note_file_misc_sloc_usage)
+ Diag.Report(diag::note_file_misc_sloc_usage)
<< (SortedUsage.end() - SortedEnd) << CountedSize - ReportedSize;
}
}
More information about the cfe-commits
mailing list