[PATCH] D28383: build: add a heuristic to determine the C++ ABI

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 5 17:02:34 PST 2017


rnk added a comment.

In https://reviews.llvm.org/D28383#637570, @majnemer wrote:

> Why isn't this equivalent to `_MSC_VER` ?


It might also be nice for clang to define some macro to indicate which C++ ABI is in use.

All that said, I think it makes sense to encode this in libc++ include/__config_site because you don't want people to build libc++ for the MS ABI and then try to use it with a mingw triple. It'd be nice if we could diagnose that at compile time instead of link time.


Repository:
  rL LLVM

https://reviews.llvm.org/D28383





More information about the cfe-commits mailing list