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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
