[PATCH] D85363: [flang] Add options to control IMPLICIT NONE

Tim Keith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 5 14:48:56 PDT 2020


tskeith added a comment.

In D85363#2197834 <https://reviews.llvm.org/D85363#2197834>, @klausler wrote:

> In D85363#2197824 <https://reviews.llvm.org/D85363#2197824>, @sscalpone wrote:
>
>> The gfortran options is -fimplicit-none.  It doesn't appear to have an inverse, but following the convention would be -fno-implicit-none.  I think there is value using the same spelling for this option.
>
> But -fno-implicit-none would mean "don't assume IMPLICIT NONE where it doesn't appear", not "ignore IMPLICIT NONE where it does appear."  Not the same thing.

We may also want an option that means "assume IMPLICIT NONE(EXTERNAL)" everywhere. That's why I included "type" in the option name. gfortran's `-fimplicit-none` option may pre-date the addition of `IMPLICIT NONE(EXTERNAL)` to the language.

Also, we can reconsider option names when we have a real driver.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85363



More information about the llvm-commits mailing list