[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