[PATCH] D120246: [flang][driver] Add support for `--target`/`--triple`

Eric Schweitz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 23 08:32:32 PST 2022


schweitz added a comment.

> In D120246#3339289 <https://reviews.llvm.org/D120246#3339289>, @tschuett wrote:
>
>> https://reviews.llvm.org/D117809
>>
>> There was a discussion that `-emit-llvm` is considered a mistake.
>
> Which comment are you referring to specifically? Also, `-emit-llvm` is something that Flang inherits from Clang. I'm merely making it available in `flang-new` rather than re-defining it.

Following clang's precedent with regard to spelling the option `-emit-llvm` seems to both follow the principle of least surprise and most obvious for sharing common code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120246



More information about the cfe-commits mailing list