[PATCH] D91851: [compiler-rt] [emutls] Add ifdefs around msvc specific pragmas

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

In D91851#2412367 <https://reviews.llvm.org/D91851#2412367>, @compnerd wrote:

> Would be nice to actually mention that in the commit message and adjust the condition to indicate that (`!defined(__MINGW32__) && !defined(__MINGW64__)`).

I think that's kind of inverts the logic IMO. We have an MSVC specific pragma in win32-generic code, and thus the ifdef should check for presence of `_MSC_VER`, not the absence of something else.

(Also FWIW, in 64 bit cases `__MINGW32__` still defined, just like `_WIN32` is defined on 64 bit platforms as well.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91851