[cfe-commits] r45976 - in /cfe/trunk: AST/Type.cpp test/Sema/complex-int.c
snaroff at apple.com
Tue Jan 15 06:44:29 PST 2008
On Jan 15, 2008, at 6:27 AM, Neil Booth wrote:
> Steve Naroff wrote:-
>> Just to be clear, my goal isn't to open up endless bugs. I've fixed
>> *many* bugs in clang over the past year and try to be careful when
>> making such changes.
> Of course not, I'm sorry if I phrased my mail badly.
>> Every time anyone touches some code, there is a chance of a
>> (especially since GCC extensions aren't wildly well specified).
>> Apparently I just created a regression. Thanks for bringing it to my
> I just intended to point out that having IsIntegerType mean
> something other than what it means in the standard will result
> in us accepting things we shouldn't. I don't know how to
> construct a vector of int but it seemed from the context of the
> patch that we still report that as an integer type.
I totally agree (isIntegerType needs to reflect the standard).
I have this on my list of things to fix (removing the vector of int
clause from isIntegerType()).
> Perhaps these shouldn't be integer types, but arithmetic types?
> Then they wouldn't go in switch(), nor enum initializers, nor
> integer constant expressions? Complex ints are reasonable in
> arithmetic expressions; I'm less clear where int vectors belong.
Makes sense. I'll consider this...
More information about the cfe-commits