[cfe-commits] r137903 - in /cfe/trunk: include/clang/Basic/Builtins.def include/clang/Basic/DiagnosticSemaKinds.td include/clang/Sema/Sema.h lib/Sema/SemaChecking.cpp test/Sema/warn-strlcpycat-size.c

[Ted Kremenek]

On Aug 18, 2011, at 1:22 PM, Eli Friedman wrote:

>>> It's just size_t being int instead of long on windows (and possibly on other platforms)
>>> 
>>> Using a target neutral typedef for size_t (like "__SIZE_TYPE__" or "__typeof__(sizeof(int))")
>>> should make it work on windows.
>>> 
>>> - Ben
>> 
>> After looking at the patch again, I don't think this is the issue.  We are defining these builtins to use size_t.
> 
> The patch is correctly defining size_t; the test is not.

Not enough sleep.  Thanks guys.  I feel really stupid.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110818/e7928e57/attachment.html>