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

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 13:54:25 PST 2018


But is that function exported from their C run-time library? Isn't it just
an inline function definition in that header file?

On Tue, Jan 23, 2018 at 1:50 PM, Adrian McCarthy via llvm-commits <
llvm-commits at lists.llvm.org> 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
>>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180123/e57f08d7/attachment.html>


More information about the llvm-commits mailing list