[flang-commits] [PATCH] D106693: [flang] Fix IsSimplyContiguous() for the case of a pointer component
Peter Klausler via Phabricator via flang-commits
flang-commits at lists.llvm.org
Mon Aug 2 09:44:01 PDT 2021
This revision was automatically updated to reflect the committed changes.
Closed by commit rGbab86463df07: [flang] Fix IsSimplyContiguous() for the case of a pointer component (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/D106693/new/
https://reviews.llvm.org/D106693
Files:
flang/lib/Evaluate/check-expression.cpp
Index: flang/lib/Evaluate/check-expression.cpp
===================================================================
--- flang/lib/Evaluate/check-expression.cpp
+++ flang/lib/Evaluate/check-expression.cpp
@@ -632,7 +632,7 @@
Result operator()(const ArrayRef &x) const {
const auto &symbol{x.GetLastSymbol()};
- if (!(*this)(symbol)) {
+ if (!(*this)(symbol).has_value()) {
return false;
} else if (auto rank{CheckSubscripts(x.subscript())}) {
// a(:)%b(1,1) is not contiguous; a(1)%b(:,:) is
@@ -645,7 +645,7 @@
return CheckSubscripts(x.subscript()).has_value();
}
Result operator()(const Component &x) const {
- return x.base().Rank() == 0 && (*this)(x.GetLastSymbol());
+ return x.base().Rank() == 0 && (*this)(x.GetLastSymbol()).value_or(false);
}
Result operator()(const ComplexPart &) const { return false; }
Result operator()(const Substring &) const { return false; }
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106693.363502.patch
Type: text/x-patch
Size: 934 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/flang-commits/attachments/20210802/b6da5710/attachment.bin>
More information about the flang-commits
mailing list