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

[Tim Keith via Phabricator via llvm-commits]

tskeith marked an inline comment as done.
tskeith added inline comments.


================
Comment at: flang/lib/Semantics/resolve-names.cpp:6092
+      symbol->attrs().set(Attr::INTRINSIC);
+      // 8.2(3): ignore type from intrinsic in type-declaration-stmt
+      symbol->get<ProcEntityDetails>().set_interface(ProcInterface{});
----------------
klausler wrote:
> Can you really ignore the type?  Should not `INTEGER :: ASIN` be an error?
It explicitly not an error. 8.2(3) says "Specifying a type for a generic intrinsic function name in a type declaration statement has no effect."

All of the compilers that I tried accepted it without comment except for NAG which warned "Type declaration for generic intrinsic ASIN ignored".


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84702/new/

https://reviews.llvm.org/D84702