[PATCH] D143301: Emit warning for unsupported gfortran flags

Tue Feb 14 11:31:39 PST 2023

awarzynski added a comment.

In D143301#4126855 <https://reviews.llvm.org/D143301#4126855>, @jdoerfert wrote:

> In D143301#4126712 <https://reviews.llvm.org/D143301#4126712>, @awarzynski wrote:
>
>>> I think the -W stuff can go in, it has tests and is reasonable.
>>
>> I'd like for us to rely on a flag from Options.td for this instead. Something similar to clang_ignored_f_Group <https://github.com/llvm/llvm-project/blob/630266aed342797d6413a69b8792567fc6263501/clang/include/clang/Driver/Options.td#L233-L234>. I would probably call it `flang_ignored_w_Group` :)
>
> For the -W stuff? You want to remove the explicit warning then (which is generally fine too)?

I had something like this in mind: https://github.com/llvm/llvm-project/blob/7301a7ce196e217c077b2b68f58366be48664223/clang/lib/Driver/ToolChains/Clang.cpp#L7448. Also, the whole logic could be moved to the compiler driver (i.e. Flang.cpp) 🤔 . Emitting a warning makes sense, but do we care about the frontend driver (i.e. "CompilerInvocation.cpp")? (which is intended for developers familiar with the implementation?)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143301