[PATCH] D89717: [flang] Another validity of the TARGET= argument of ASSOCIATED() for objects

Pete Steinfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 19 11:53:01 PDT 2020


PeteSteinfeld added a comment.

In D89717#2339426 <https://reviews.llvm.org/D89717#2339426>, @klausler wrote:

> Why is this a fatal error, rather than a warning?  If the TARGET= argument lacks both POINTER and TARGET attributes, the result will always be false, and we can warn about that.

This same logic applies to all invalid TARGET= arguments.  GNU, pgf90, IBM, Intel, and NAG all consider this an error.  I think that we should. too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89717



More information about the llvm-commits mailing list