This is an archive of the discontinued LLVM Phabricator instance.

[flang] Fixed a crash
ClosedPublic

Authored by PeteSteinfeld on May 1 2020, 8:53 PM.

Details

Summary

I found a small test case that caused a crash when derived type
definitions have parameters without definitions.

Diff Detail

Event Timeline

PeteSteinfeld created this revision.May 1 2020, 8:53 PM
Herald added a project: Restricted Project. · View Herald Transcript
PeteSteinfeld added a project: Restricted Project.May 1 2020, 8:55 PM
PeteSteinfeld marked an inline comment as done.May 2 2020, 12:34 PM
PeteSteinfeld added inline comments.
flang/test/Semantics/resolve33.f90
33–38

This test case is not sufficient to trigger the crash. I need to add the declaration of a variable of the type to do that.

  1. Updating D79282: [flang] Fixed a crash #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #
  4. If you intended to create a new revision, use:
  5. $ arc diff --create

Added a declaration to make the test case crash the compiler

The declaration of the type wasn't enough to cause the crash. I needed to
declare an object of that type.

tskeith accepted this revision.May 2 2020, 1:03 PM
This revision is now accepted and ready to land.May 2 2020, 1:03 PM
This revision was automatically updated to reflect the committed changes.