glandium added a comment.

In D150646#4382763 <https://reviews.llvm.org/D150646#4382763>, @aidengrossman 
wrote:

> Interesting. I would've thought `-fms-extensions` by itself would've declared 
> that macro (which is what I gathered reading what caused the MSVC compatible 
> builtins to get included in clang), but apparently not. For whatever reason, 
> `LangOpts.MicrosoftExt` only seems to be set to true if 
> `-fms-compaitilibty-version` (maybe on top of `-fms-compatibility`) are also 
> passed (not 100% on this though). I'll have to do some more digging soon.

Did you find something?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150646

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D150646: [clang][X8... Mike Hommey via Phabricator via cfe-commits

Reply via email to