[libcxx-commits] [PATCH] D91139: [7/N] [libcxx] Don't use __int128 for msvc targets

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Dec 3 08:19:32 PST 2020

ldionne added a comment.

In D91139#2429595 <https://reviews.llvm.org/D91139#2429595>, @rnk wrote:

> After having dealt with an increasing number of user complaints about lack of i128 support, I am of the opinion that we should start shipping, auto-linking, and requiring clang_rt.builtins in clang-cl.
> However, it's a bit of a project to make this work out of the box, since clang's runtime libraries are not on the linker search path. We'd need to update our msbuild integration bits and some documentation. But it's time to do that work already.

So, if I understand correctly, you're suggesting that the `int128` builtins should be provided on Windows, and this workaround shouldn't be necessary once that's done? In that case, unless @mstorsjo  disagrees, I think we're better holding off with this patch.

  rG LLVM Github Monorepo



More information about the libcxx-commits mailing list