[PATCH] D84702: [flang] Fix bug with intrinsic in type declaration stmt

[Tim Keith via Phabricator via llvm-commits]

tskeith created this revision.
tskeith added reviewers: klausler, PeteSteinfeld, jeanPerier.
tskeith added a project: Flang.
Herald added a reviewer: jdoerfert.
Herald added a reviewer: DavidTruby.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

When an instrinsic function is declared in a type declaration statement
we need to set the INTRINSIC attribute and (per 8.2(3)) ignore the
specified type.

To simplify the check, add IsIntrinsic utility to BaseVisitor.

Also, intrinsics and external procedures were getting assigned a size
and offset and they shouldn't be.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D84702

Files:
  flang/include/flang/Semantics/symbol.h
  flang/lib/Semantics/compute-offsets.cpp
  flang/lib/Semantics/resolve-names.cpp
  flang/test/Semantics/symbol18.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84702.281048.patch
Type: text/x-patch
Size: 5115 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200727/fa58a8a6/attachment.bin>