[cfe-commits] r108157 - in /cfe/trunk: include/clang/AST/Expr.h lib/Sema/SemaExpr.cpp

Fariborz Jahanian fjahanian at apple.com
Mon Jul 12 11:13:55 PDT 2010


On Jul 12, 2010, at 10:32 AM, Douglas Gregor wrote:

>
> On Jul 12, 2010, at 10:26 AM, Fariborz Jahanian wrote:
>
>>
>
> Thanks, Fariborz! I don't think we even need a parameter for  
> hasDependentType. Why not just check
>
> 	!t.isNull() && t.isDependentType()?
>
> The isDependentType() check is very fast; basically just looking at  
> a bit in the canonical Type structure.

Makes sense and is it now checked in r108166.
- Fariborz

>
>
> 	- Doug




More information about the cfe-commits mailing list