[cfe-dev] Any reason we disallow ui64 prefix

[Chris Lattner]

On Jan 22, 2010, at 11:47 AM, Fariborz Jahanian wrote:

>
> On Jan 22, 2010, at 11:41 AM, Chris Lattner wrote:
>
>>
>> On Jan 22, 2010, at 11:40 AM, Fariborz Jahanian wrote:
>>
>>>
>>> Hi,
>>>
>>> In LiteralSupport.cpp we explicitly disallow ui64 suffix in msft
>>> extensions (as in #define ULLONG_MAX 0xffffffffffffffffui64).
>>> This constant is defined by visual studio's "limits.h".
>>
>> Where is it explicitly disallowed?
>
> in: lib/Lex/LiteralSupport.cpp
>
>     case 'u':
>    case 'U':
>      if (isFPConstant) break;  // Error for floating constant.
>      if (isUnsigned) break;    // Cannot be repeated.
>      isUnsigned = true;
>      continue;  // Success.
>
>
> ...
>
> case 'i':
>      if (PP.getLangOptions().Microsoft) {
>        if (isFPConstant || isUnsigned || isLong || isLongLong) break;

Ah, just looks like an oversight,

-Chris