[PATCH] D61670: [clang] [MinGW] Add the option -fno-autoimport

Fri Aug 25 14:02:21 PDT 2023

mstorsjo added a comment.

In D61670#4611790 <https://reviews.llvm.org/D61670#4611790>, @aeubanks wrote:

> In D61670#4607313 <https://reviews.llvm.org/D61670#4607313>, @mstorsjo wrote:
>
>>> I expected the answer would be "yes", so I said "lgtm" and then phrased my question very awkwardly.
>>
>> Ah, thanks for the clarification!
>>
>> Any opinion on the name, `-fno-autoimport` vs `-fno-auto-import`, given the existing linker option `--disable-auto-import`?
>
> Is there an official name, "auto-import"/"autoimport"/"auto import"?

Not really, no - it's all very much ad-hoc.

> Since cc1 flags are changeable and this hasn't landed yet, I think it'd be nice to standardize before we commit on a stable name.
> If there isn't already a name, I like the dash in "auto-import".

Ok, sounds reasonable - I'll change it into that form then.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61670