[libcxx-commits] [PATCH] D91413: [libcxx] Provide __int128_t builtins needed for filesystem on MSVC

Nico Weber via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Nov 13 08:32:56 PST 2020


thakis added a comment.

Hrm, this seems like a fix at the wrong layer to me.

Since __int128_t creates compiler_rt calls, it'd seem reasonable if we made __uint128_t error out when targeting windows in the compiler, or if we changed codegen to call _mul128() on windows, or some such. This change here would mean that making such a change in clang would break libcxx, which seems a bit unfortunate.

Maybe we should just teach clang to use _mul128() when targeting windows?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91413



More information about the libcxx-commits mailing list