[cfe-commits] r115302 - in /cfe/trunk: include/clang/Basic/Builtins.def include/clang/Basic/BuiltinsX86.def lib/AST/ASTContext.cpp

[Chris Lattner]

On Oct 1, 2010, at 8:47 AM, Eli Friedman wrote:

>> 
>>  // Types may be postfixed with the following modifiers:
>>  // * -> pointer (optionally followed by an address space number)
>> @@ -367,7 +368,7 @@
>>  BUILTIN(__builtin___vprintf_chk, "iicC*a", "FP:1:")
>> 
>>  BUILTIN(__builtin_expect, "LiLiLi"   , "nc")
>> -BUILTIN(__builtin_prefetch, "vvC*.", "nc")
>> +BUILTIN(__builtin_prefetch, "vvC*.", "nIc")
> 
> Err, is this change really intentional?  For functions with variable
> parameters, we should probably just depend on the SemaChecking
> infrastructure to make the appropriate checks.

No, I botched this, will fix thanks!

-Chris