[PATCH] D99353: [driver] Make `clang` warn rather then error on `flang` options

Andrzej Warzynski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 7 08:53:46 PDT 2021


awarzynski added a comment.

In D99353#2672503 <https://reviews.llvm.org/D99353#2672503>, @protze.joachim wrote:

> In D99353#2669046 <https://reviews.llvm.org/D99353#2669046>, @awarzynski wrote:
>
>> Btw, how important are these aliases for you?
>
> It's just not obvious from `flang --help` that this is an alias for `-ffixed-line-length-132` (or the other way around). I only learned that by looking at LLVM source.

It's the other way round :) `-ffixed-line-length-132` is an alias for `-ffixed-line-length=132`. If you run `flang-new --help`, then you will only see non-aliases. Hopefully that will help navigating this.

I don't have a solution for aliases, I'd need more time for this. Long term we should definitely aim for something more general, but for now I'd like to focus on making sure that:

- you are no longer blocked, and
- we are not introducing anything that could block/disable other important use cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99353



More information about the cfe-commits mailing list