[cfe-commits] r84964 - /cfe/trunk/include/clang/AST/Expr.h

[Douglas Gregor]

On Oct 23, 2009, at 9:45 PM, Chris Lattner wrote:

>
> On Oct 23, 2009, at 1:32 PM, Douglas Gregor wrote:
>
>>
>> On Oct 23, 2009, at 12:38 PM, Sebastian Redl wrote:
>>
>>> Author: cornedbee
>>> Date: Fri Oct 23 14:38:17 2009
>>> New Revision: 84964
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=84964&view=rev
>>> Log:
>>> Fix operator precedence in Doug's most recent commit.
>
>>> -    if (DecoratedD.getInt() & HasQualifierFlag == 0)
>>> +    if ((DecoratedD.getInt() & HasQualifierFlag) == 0)
>>>     return reinterpret_cast<ExplicitTemplateArgumentList *>(this +
>>> 1);
>>>
>>>   return reinterpret_cast<ExplicitTemplateArgumentList *>(
>>
>> Whoops. Thanks for the fix!
>
> Does clang warn about that case?

Nope, but GCC does with -Wall.

>  If not, please file a bugzilla,

Done:

	http://llvm.org/bugs/show_bug.cgi?id=5297

	- Doug