HomePhabricator

[flang] Improve initializer semantics, esp. for component default values

Authored by klausler on Dec 7 2020, 12:08 PM.

Description

[flang] Improve initializer semantics, esp. for component default values

This patch plugs many holes in static initializer semantics, improves error
messages for default initial values and other component properties in
parameterized derived type instantiations, and cleans up several small
issues noticed during development. We now do proper scalar expansion,
folding, and type, rank, and shape conformance checking for component
default initializers in derived types and PDT instantiations.
The initial values of named constants are now guaranteed to have been folded
when installed in the symbol table, and are no longer folded or
scalar-expanded at each use in expression folding. Semantics documentation
was extended with information about the various kinds of initializations
in Fortran and when each of them are processed in the compiler.

Some necessary concomitant changes have bulked this patch out a bit:

  • contextual messages attachments, which are now produced for parameterized derived type instantiations so that the user can figure out which instance caused a problem with a component, have been added as part of ContextualMessages, and their implementation was debugged
  • several APIs in evaluate::characteristics was changed so that a FoldingContext is passed as an argument rather than just its intrinsic procedure table; this affected client call sites in many files
  • new tools in Evaluate/check-expression.cpp to determine when an Expr actually is a single constant value and to validate a non-pointer variable initializer or object component default value
  • shape conformance checking has additional arguments that control whether scalar expansion is allowed
  • several now-unused functions and data members noticed and removed
  • several crashes and bogus errors exposed by testing this new code were fixed
  • a -fdebug-stack-trace option to enable LLVM's stack tracing on a crash, which might be useful in the future

TL;DR: Initialization processing does more and takes place at the right
times for all of the various kinds of things that can be initialized.

Differential Review: https://reviews.llvm.org/D92783

Details

Committed
klauslerDec 7 2020, 2:40 PM
Parents
rG155fca3cae27: [clang] Fix noderef for array member of deref expr
Branches
Unknown
Tags
Unknown