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

Mon Aug 21 12:55:41 PDT 2023

rnk added a comment.

In D61670#4604486 <https://reviews.llvm.org/D61670#4604486>, @mstorsjo wrote:

> In D61670#4604145 <https://reviews.llvm.org/D61670#4604145>, @rnk wrote:
>
>> cc + at aeubanks @jyknight to consider using the code model for this purpose
>
> Hmm, I don't quite understand this comment; do you suggest that we after all should use the code model for controlling the use of `.refptr` stubs too - didn't we conclude earlier that it isn't a great fit for that?

Sorry, let me elaborate. Since the first round of review, Arthur and James have been looking at x86 code models, in particular the medium code model. I guess what I really want to ask is, Arthur and James, do you agree with our decision that the code model should not control the formation of these COFF refptr stubs?

I expected the answer would be "yes", so I said "lgtm" and then phrased my question very awkwardly.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61670