[PATCH] D42441: IRGen: On MSVCRT platforms, implement __builtin_wmemcmp using memcmp.

[Peter Collingbourne via llvm-commits]

Yes, but wmemcmp is defined with an inline implementation in the header,
and the .lib file does not contain a definition for it.

Peter

On Tue, Jan 23, 2018 at 1:50 PM, Adrian McCarthy <amccarth at google.com>
wrote:

> MSVC RT does have wmemcmp, but it's in a different header file:
>
> https://msdn.microsoft.com/en-us/library/zyaebf12.aspx
>
> On Tue, Jan 23, 2018 at 1:43 PM, Peter Collingbourne via Phabricator via
> llvm-commits <llvm-commits at lists.llvm.org> wrote:
>
>> pcc planned changes to this revision.
>> pcc added a comment.
>>
>> Oops, it does indeed. I guess we'll need to emit an inline implementation.
>>
>>
>> https://reviews.llvm.org/D42441
>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>
>
>


-- 
-- 
Peter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180123/61ca1736/attachment.html>