[PATCH] D94849: [flang] Refine WhyNotModifiable()

Peter Klausler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 15 17:01:58 PST 2021


klausler created this revision.
klausler added a reviewer: PeteSteinfeld.
klausler added a project: Flang.
Herald added a subscriber: jdoerfert.
klausler requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The utility routine WhyNotModifiable() needed to become more
aware of the use of pointers in data-refs; the targets of
pointer components are sometimes modifiable even when the
leftmost ("base") symbol of a data-ref is not.

Added a new unit test for WhyNotModifiable() that uses internal
READ statements (mostly), since I/O semantic checking uses
WhyNotModifiable() for all its definability checking.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D94849

Files:
  flang/include/flang/Evaluate/tools.h
  flang/lib/Evaluate/tools.cpp
  flang/lib/Semantics/check-io.cpp
  flang/lib/Semantics/tools.cpp
  flang/test/Semantics/modifiable01.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D94849.317117.patch
Type: text/x-patch
Size: 9079 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210116/02a86339/attachment.bin>


More information about the llvm-commits mailing list