On Feb 23, 2011, at 9:00 PM, Chandler Carruth wrote:

> On Wed, Feb 23, 2011 at 8:57 PM, Chris Lattner <[email protected]> wrote:
> Saying "but 'int' only has 32 bits" seems to do the trick, no?  This is a 
> great example of a case with no promotion involved!
> 
> I'm envisioning users responding to this, "but where the hell did an 'int' 
> type come from?". Basically, it's not at all about promotion, it's about the 
> fact that 10 has a distinct type that the operation takes place within prior 
> to being assigned. Maybe I don't give users enough credit, but it seems nice 
> to remind them that the shift itself has a type, and it's not the one they 
> wrote down.
> 
> I don't feel strongly about this though. I see it as nice to have. If it 
> really bothers you, I'll remove it.

I'd really like to keep the warning terse and to the point.  The essays can 
stay in the static analyzer ;-)

-Chris
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to