[flang-commits] [PATCH] D121124: [flang] Avoid crash case in provenance mapping
Peter Klausler via Phabricator via flang-commits
flang-commits at lists.llvm.org
Mon Mar 7 11:44:02 PST 2022
This revision was not accepted when it landed; it landed in state "Needs Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rGd7ea6068e5ad: [flang] Avoid crash case in provenance mapping (authored by klausler).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121124/new/
https://reviews.llvm.org/D121124
Files:
flang/lib/Parser/provenance.cpp
Index: flang/lib/Parser/provenance.cpp
===================================================================
--- flang/lib/Parser/provenance.cpp
+++ flang/lib/Parser/provenance.cpp
@@ -435,11 +435,15 @@
return std::nullopt;
}
ProvenanceRange first{provenanceMap_.Map(cookedRange.begin() - &data_[0])};
- if (cookedRange.size() <= first.size()) {
+ if (cookedRange.size() <= first.size()) { // always true when empty
return first.Prefix(cookedRange.size());
}
- ProvenanceRange last{provenanceMap_.Map(cookedRange.end() - &data_[0])};
- return {ProvenanceRange{first.start(), last.start() - first.start()}};
+ ProvenanceRange last{provenanceMap_.Map(cookedRange.end() - 1 - &data_[0])};
+ if (first.start() <= last.start()) {
+ return {ProvenanceRange{first.start(), last.start() - first.start() + 1}};
+ } else {
+ return std::nullopt;
+ }
}
std::optional<CharBlock> CookedSource::GetCharBlock(
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121124.413576.patch
Type: text/x-patch
Size: 930 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/flang-commits/attachments/20220307/0ed5f961/attachment.bin>
More information about the flang-commits
mailing list